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

[BUGFIX] Si l'utilisateur a un identifiant renseigné alors le champ "e-mail" ne doit pas être obligatoire dans Pix Admin (PIX-15540) #11099

Conversation

P-Jeremy
Copy link
Contributor

@P-Jeremy P-Jeremy commented Jan 13, 2025

🎄 Problème

Lorsqu’un agent PIX tente de modifier une fiche utilisateur dans Pix Admin (par exemple, en changeant le nom) pour un utilisateur disposant d’un identifiant comme seule méthode de connexion alors la modification est bloquée car Pix Admin exige une adresse e-mail alors que cela ne devrait pas l’être dans ce contexte.

🎁 Proposition

Ne pas rendre obligatoire l'email si le user a une méthode de connexion par identifiant

🧦 Remarques

🎅 Pour tester

Depuis Orga:

  • Modifier un élève en cliquant sur les 3 points > Gerer le compte > Ajouter l'identifiant
    image
    image

Depuis Admin:

  • Retirer la méthode de connection par mail de cet utilisateur
  • Cliquer sur Modifier pour mettre à jour son prénom par exemple
  • Constater que le formulaire ne bloque pas l'appel même avec un email vide

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@P-Jeremy P-Jeremy force-pushed the pix-15540-allow-email-null-when-editing-a-user-with-userName-connection branch 2 times, most recently from 2fac36f to 1f4b03b Compare January 13, 2025 16:19
@P-Jeremy P-Jeremy closed this Jan 14, 2025
@P-Jeremy P-Jeremy reopened this Jan 14, 2025
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@P-Jeremy P-Jeremy force-pushed the pix-15540-allow-email-null-when-editing-a-user-with-userName-connection branch from 8870b9c to 333075e Compare January 14, 2025 15:36
Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lu et testé avec succès

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15540-allow-email-null-when-editing-a-user-with-userName-connection branch from 333075e to 5573e5c Compare January 15, 2025 10:03
@pix-service-auto-merge pix-service-auto-merge merged commit 80cc393 into dev Jan 15, 2025
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15540-allow-email-null-when-editing-a-user-with-userName-connection branch January 15, 2025 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants