[FEATURE] Afficher le message d'erreur "adresse email invalide ou déjà utilisée" (pix-14689) #10546

EmmanuelleBonnemay · 2024-11-14T13:36:42Z

🍂 Problème

Pour des questions de gestion d'informations personnelles, il ne faut pas divulguer l'information d'un email déjà utilisé.
Contextes concernés :
Contextes

Lors de l’inscription

Lors de la modification d’une adresse e-mail dans Mon Compte

Lors d’une invitation à rejoindre un centre de certification

Lors d’une invitation à rejoindre une orga

🌰 Proposition

Modifier chaque message d’erreur par
🇫🇷 "Adresse e-mail invalide ou déjà utilisée"

🇬🇧 “Invalid or already used e-mail address”

🎃 Remarques

Ne pas toucher au contexte sco
Un autre scénario a été identifié : celui de la récupération de compte après sortie du sco. Le message d'erreur a été modifié également dans ce scénario. Choix à discuter.
Les clés de traduction précédemment utilisées ont été retirées dans Pix Certif mais pas dans Pix App, où elles sont susceptibles d'être encore utilisées.

🪵 Pour tester

Reprendre chacun des scénarios et vérifier l'affichage du message attendu :

inscription sur pix app
update d'email sur pix app
creation de compte après réception d'une invitation sur pix orga
création de compte après réception d'une invitation dur pix certif

Reprendre les deux scénarios sco et vérifier que le message d'erreur est différent du 1.

du learner vers le user :
On attend ce message "Cette adresse e-mail est déjà enregistrée, connectez-vous."
Pour tester ce scénario voir [FEATURE] Envoi de l'email de creation de compte en asynchrone (PIX-15112) #10509, partie "création de compte avec réconciliation"
du user vers le learner :
On attend cette modale :

Pour tester ce scénario :
-- afficher dans pix orga (connecté avec [email protected]) la liste des élèves du collège House of the Dragon
-- se connecter sur pix app avec un compte random (par exemple [email protected])
-- cliquer sur "J'ai un code"
-- choisir pour la réconciliation un élève qui a déjà une méthode de connexion email dans la liste des élèves du collège
-- constater l'affichage de la modale

Tester aussi le scénario supplémentaire de récupération de compte après sortie de sco (?)

https://1024pix.atlassian.net/wiki/x/AYApw

pix-bot-github · 2024-11-14T13:37:34Z

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

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

lego-technix · 2024-11-15T10:28:40Z

api/src/identity-access-management/domain/errors.js

+class InvalidOrAlreadyUsedEmailError extends DomainError {
+  constructor(message = 'Cette adresse e-mail est invalide ou déjà utilisée.', code = 'INVALID_OR_ALREADY_USED_EMAIL') {
+    super(message, code);
+  }
+}


Il ne faut pas mettre de message en français dans les erreurs, cf. https://github.com/1024pix/pix/blob/dev/docs/adr/0044-gestion-erreurs-i18n-reference.md

De nombreuses erreurs ne suivent pas encore l'ADR, mais il faut suivre l'ADR pour la création de toute nouvelle erreur.

Et par ailleurs il n'y a pas besoin de rendre le « message d'erreur en anglais concis » et le code d'ereur configurables, aussi je suggère le code ci-dessous :

Suggested change

class InvalidOrAlreadyUsedEmailError extends DomainError {

constructor(message = 'Cette adresse e-mail est invalide ou déjà utilisée.', code = 'INVALID_OR_ALREADY_USED_EMAIL') {

super(message, code);

}

}

class InvalidOrAlreadyUsedEmailError extends DomainError {

constructor() {

super('Invalid or already used e-mail address', 'INVALID_OR_ALREADY_USED_EMAIL');

}

}

lego-technix · 2024-11-15T10:43:06Z

api/translations/en.json

@@ -295,6 +295,7 @@
    "EMPTY_LAST_NAME": "Please enter a last name.",
    "EMPTY_USERNAME": "Please enter a username.",
    "FILL_USERNAME_OR_EMAIL": "Please enter an email address and/or a username.",
+    "INVALID_OR_ALREADY_USED_EMAIL" : "Invalid or already used e-mail address",


Il faut saisir l'occasion de cette PR pour effectuer les modifications nécessaires pour supprimer l'utilisation de cette clé de traduction dans l'API. En effet les traductions doivent être gérées dans les apps Front, conformément à https://github.com/1024pix/pix/blob/dev/docs/adr/0044-gestion-erreurs-i18n-reference.md

lego-technix · 2024-11-15T10:43:13Z

api/translations/es.json

@@ -307,6 +307,7 @@
    "EMPTY_LAST_NAME": "No se ha introducido su nombre.",
    "EMPTY_USERNAME": "Su nombre de usuario no se ha rellenado.",
    "FILL_USERNAME_OR_EMAIL": "Debe introducir una dirección de correo electrónico y/o un nombre de usuario.",
+    "INVALID_OR_ALREADY_USED_EMAIL" : "Invalid or already used e-mail address",


Il faut saisir l'occasion de cette PR pour effectuer les modifications nécessaires pour supprimer l'utilisation de cette clé de traduction dans l'API. En effet les traductions doivent être gérées dans les apps Front, conformément à https://github.com/1024pix/pix/blob/dev/docs/adr/0044-gestion-erreurs-i18n-reference.md

lego-technix · 2024-11-15T10:43:33Z

api/translations/fr.json

@@ -309,6 +309,7 @@
    "EMPTY_LAST_NAME": "Votre nom n’est pas renseignée.",
    "EMPTY_USERNAME": "Votre identifiant n’est pas renseigné.",
    "FILL_USERNAME_OR_EMAIL": "Vous devez renseigner une adresse e-mail et/ou un identifiant.",
+    "INVALID_OR_ALREADY_USED_EMAIL" : "Adresse e-mail invalide ou déjà utilisée",


Il faut saisir l'occasion de cette PR pour effectuer les modifications nécessaires pour supprimer l'utilisation de cette clé de traduction dans l'API. En effet les traductions doivent être gérées dans les apps Front, conformément à https://github.com/1024pix/pix/blob/dev/docs/adr/0044-gestion-erreurs-i18n-reference.md

lego-technix · 2024-11-15T10:43:40Z

api/translations/nl.json

@@ -307,6 +307,7 @@
    "EMPTY_LAST_NAME": "Je naam is niet ingevuld.",
    "EMPTY_USERNAME": "Uw login is niet ingevuld.",
    "FILL_USERNAME_OR_EMAIL": "Je moet een e-mailadres en/of een gebruikersnaam invoeren.",
+    "INVALID_OR_ALREADY_USED_EMAIL" : "Invalid or already used e-mail address",


Il faut saisir l'occasion de cette PR pour effectuer les modifications nécessaires pour supprimer l'utilisation de cette clé de traduction dans l'API. En effet les traductions doivent être gérées dans les apps Front, conformément à https://github.com/1024pix/pix/blob/dev/docs/adr/0044-gestion-erreurs-i18n-reference.md

…itory

…p and email update

…center sign up

…ontext

er-lim · 2024-11-15T13:42:25Z

api/lib/domain/usecases/create-and-reconcile-user-to-organization-learner.js

+  if (error instanceof InvalidOrAlreadyUsedEmailError) {
+    error = new AlreadyRegisteredEmailError();
+  }


Remarque: J'ai l'impression qu'on peut enlever le if nn ?
Dans tous les cas, on veut que error soit du type AlreadyRegisteredEmailError;

Suggested change

if (error instanceof InvalidOrAlreadyUsedEmailError) {

error = new AlreadyRegisteredEmailError();

}

error = new AlreadyRegisteredEmailError();

er-lim · 2024-11-15T14:19:54Z

Test en RA sur firefox fait :

Invitation Pix Orga / Certif ✅
Inscription et modification de l'adresse e-mail sur mon-pix ✅
Rejoindre une campagne avec le lien ou avec le bouton J'ai un code ✅
Sortie SCO ✅

Une remarque pour la sortie SCO : l'erreur s'affiche dans un bloc bleu plutôt que rouge comme sur la capture 🤔

EmmanuelleBonnemay added the team-acces label Nov 14, 2024

EmmanuelleBonnemay self-assigned this Nov 14, 2024

EmmanuelleBonnemay requested a review from a team as a code owner November 14, 2024 13:36

EmmanuelleBonnemay force-pushed the pix-14689-change-error-message branch 3 times, most recently from 60b8f71 to cc7e521 Compare November 14, 2024 17:08

EmmanuelleBonnemay changed the title ~~[REFACTOR] Afficher le message d'erreur "adresse email invalide ou déjà utilisée" (pix-14689)~~ [FEATURE] Afficher le message d'erreur "adresse email invalide ou déjà utilisée" (pix-14689) Nov 14, 2024

EmmanuelleBonnemay force-pushed the pix-14689-change-error-message branch 3 times, most recently from 7fc9f72 to 4299192 Compare November 15, 2024 08:45

lego-technix reviewed Nov 15, 2024

View reviewed changes

EmmanuelleBonnemay force-pushed the pix-14689-change-error-message branch 2 times, most recently from 2ca29e1 to 0568a01 Compare November 15, 2024 13:03

EmmanuelleBonnemay added 4 commits November 15, 2024 14:26

refactor(api): add Invalid or already used email error for user repos…

78ac08b

…itory

refactor(mon-pix): add Invalid or already used email error for sign u…

83dc72d

…p and email update

refactor(certif): add Invalid or already used email in certification …

ebb13e8

…center sign up

refactor(api): keep error type "AlreadyRegisteredEmailError" in sco c…

ca19a9a

…ontext

er-lim reviewed Nov 15, 2024

View reviewed changes

EmmanuelleBonnemay force-pushed the pix-14689-change-error-message branch from 0568a01 to 5c9036f Compare November 15, 2024 14:33

refactor(api): handle account recovery case

1be67b3

EmmanuelleBonnemay force-pushed the pix-14689-change-error-message branch from 5c9036f to 1be67b3 Compare November 15, 2024 14:43

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[FEATURE] Afficher le message d'erreur "adresse email invalide ou déjà utilisée" (pix-14689) #10546

[FEATURE] Afficher le message d'erreur "adresse email invalide ou déjà utilisée" (pix-14689) #10546

EmmanuelleBonnemay commented Nov 14, 2024 •

edited

Loading

pix-bot-github commented Nov 14, 2024

lego-technix Nov 15, 2024 •

edited

Loading

lego-technix Nov 15, 2024 •

edited

Loading

lego-technix Nov 15, 2024 •

edited

Loading

lego-technix Nov 15, 2024 •

edited

Loading

lego-technix Nov 15, 2024 •

edited

Loading

er-lim Nov 15, 2024

er-lim commented Nov 15, 2024

[FEATURE] Afficher le message d'erreur "adresse email invalide ou déjà utilisée" (pix-14689) #10546

Are you sure you want to change the base?

[FEATURE] Afficher le message d'erreur "adresse email invalide ou déjà utilisée" (pix-14689) #10546

Conversation

EmmanuelleBonnemay commented Nov 14, 2024 • edited Loading

🍂 Problème

🌰 Proposition

🎃 Remarques

🪵 Pour tester

pix-bot-github commented Nov 14, 2024

lego-technix Nov 15, 2024 • edited Loading

Choose a reason for hiding this comment

lego-technix Nov 15, 2024 • edited Loading

Choose a reason for hiding this comment

lego-technix Nov 15, 2024 • edited Loading

Choose a reason for hiding this comment

lego-technix Nov 15, 2024 • edited Loading

Choose a reason for hiding this comment

lego-technix Nov 15, 2024 • edited Loading

Choose a reason for hiding this comment

er-lim Nov 15, 2024

Choose a reason for hiding this comment

er-lim commented Nov 15, 2024

EmmanuelleBonnemay commented Nov 14, 2024 •

edited

Loading

lego-technix Nov 15, 2024 •

edited

Loading

lego-technix Nov 15, 2024 •

edited

Loading

lego-technix Nov 15, 2024 •

edited

Loading

lego-technix Nov 15, 2024 •

edited

Loading

lego-technix Nov 15, 2024 •

edited

Loading