search

luuppiry / luuppi-next

🔁 The backbone of the luuppi.fi website (new weba) built using the latest bleeding-edge technologies, utilizing Next.js and the headless content management system, Strapi.
https://luuppi.fi
MIT License
9 stars 1 forks source link

Kotikunta pakolliseks #148

Open matskuman5 opened 2 weeks ago

matskuman5 commented 2 weeks ago

Description

Käyttäjän tiedoissa kotikuntaa ei oo merkattu pakolliseks (ei oo tähtee) vaikka yhdistyslain vaatima tieto

Voi pistää siihe jonku infon et "kunta jossa asut tällä hetkellä" ni ei mee porukka shokkii et mitä toi tarkottaa

Reproduce

  1. ylläpito -> käyttäjähallinta -> eti käyttäjä -> perustiedot -> kotikunnassa pitäs olla tähti et on pakollinen

Screenshots

No response

Area

Frontend (next)

KasperiP commented 2 weeks ago

Tämä ei oikeastaan ole bugi. Käyttäjähallinnan puolella mikään kenttä ei ole pakollinen (paitsi sähköposti, joka on kaikilla käyttäjillä ja jota ei voi muokata, sekä pääaine, joka oletuksena on "OTHER").

Jos käyttäjä on jäsen ja muokkaa omia tietojaan itse profiilin kautta, silloin kaikki yhdistyslain vaatimat tiedot ovat pakollisia. En kuitenkaan näe järkeä rajoittaa admin-puolen hallintaa samalla tavalla, koska voi olla tilanteita, joissa käyttäjän tietoja täytyy muokata, vaikka kaikkia tietoja ei olisikaan tiedossa. Tällöin pakollisten tietojen syöttö estäisi tietojen tallentamisen.