Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Améliorer la gestion de l'unicité des login/email sur les personnes physiques #1125

Open
cmourizard opened this issue Jan 9, 2022 · 1 comment
Assignees

Comments

@cmourizard
Copy link
Contributor

Sur l'espace membre, lorsqu'on modifie son login, nous n'avons pas de notification si la nouvelle valeur est un doublon par rapport aux autres membres.

Dans le BO, il y a bien une vérification par contre, on n'a pas un "joli" message d'erreur sur le formulaire, mais juste un message comme quoi il y eut une erreur. Il faudrait à la fois changer l'unicité en base du login, et améliorer le message dans le BO (et voir les impacts du changement d'unicité sur les autres formulaires comme le formulaire d'inscription)

@agallou
Copy link
Member

agallou commented Mar 13, 2022

Espace membre :

Admin :

Base :

  • Dans la table afup_personnes_physiques, on a déjà une unicité sur le mail : UNIQUE KEY idx_email_unique (email),
  • ajouter via une migration une unicité sur le champ login de afup_personnes_physiques.

Nous n'avons pas deux fois le mêm login en base donc, il n'y aura pas de rattrapage à faire là dessus :


MariaDB [afup]> select count(*) from  afup_personnes_physiques group by login having count(*) > 1;
Empty set (0.00 sec)

Adhésion personne physique

Adhésion personne morale

Invitation sur personne morale

@agallou agallou moved this to Todo in Backlog global Mar 13, 2022
@agallou agallou changed the title Espace membre > Changement de login/email - Notification en cas de doublon Améliorer la gestion de l'unicité des login/email sur les personnes physiques Mar 13, 2022
@stakovicz stakovicz self-assigned this Nov 14, 2024
stakovicz added a commit to stakovicz/web that referenced this issue Nov 15, 2024
@stakovicz stakovicz moved this from Todo to In Progress in Backlog global Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

No branches or pull requests

3 participants