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

[FEATURE] Afficher la bonne réception de demande de réinitialisation de mot de passe (PIX-14112) #10402

Conversation

lego-technix
Copy link
Contributor

@lego-technix lego-technix commented Oct 23, 2024

🦄 Problème

Il faut afficher la bonne réception de demande de réinitialisation de mot de passe.

Desktop - Mot de passe oublié - 2

🤖 Proposition

Afficher une zone d'information dans le même composant plutôt que de créer une nouvelle page + un nouveau composant.

Cette PR contient également la modification de ne plus afficher qu'un compte n'existe pas si l'utilisateur fournit une adresse email ne correspondant à aucun compte existant, même si pour l'instant l'API continue de renvoyer une réponse 404.

🌈 Remarques

RAS

💯 Pour tester

  1. Activer le feature flag FT_NEW_AUTHENTICATION_DESIGN_ENABLED=true
  2. Aller sur https://app.dev.pix.fr/mot-de-passe-oublie
  3. Vérifier qu'on ne peut envoyer le formulaire qu'avec une adresse email valide
  4. Vérifier qu'en envoyant le formulaire avec une adresse email présente dans la table users, le formulaire affiche une zone d'information avec le message « Si votre adresse est bien associée à un compte Pix, les instructions pour réinitialiser votre mot de passe vous ont été envoyées par e-mail. »
  5. Vérifier qu'en envoyant le formulaire avec une adresse email NON présente dans la table users, le formulaire affiche une zone d'information avec le message « Si votre adresse est bien associée à un compte Pix, les instructions pour réinitialiser votre mot de passe vous ont été envoyées par e-mail. »
  6. Cliquer sur le lien « Renvoyer l’e-mail de réinitialisation » et vérifier qu'il renvoie bien sur le formulaire de mot de passe oublié
  7. Aller sur https://app.dev.pix.org/mot-de-passe-oublie et sélectionner différentes langues avec le LanguageSwitcher pour vérifier les traductions de la zone d'information
  8. Arrêter l'API et vérifier que le formulaire affiche une erreur Une erreur interne est survenue, nos équipes sont en train de résoudre le problème. Veuillez réessayer ultérieurement. (il faut généralement attendre environ 20s ou 30s pour que la connexion arrive en timeout avant qu'une erreur soit déclenchée)

@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 :

@lego-technix lego-technix force-pushed the pix-14112-new-component-password-reset-demand-sending branch from e87aef8 to efe4713 Compare October 23, 2024 17:43
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

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

✅ Testé et approuvé.
Juste 2 petites suggestions.

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-14112-new-component-password-reset-demand-sending branch from 005caea to 30c98b9 Compare October 24, 2024 08:02
@pix-service-auto-merge pix-service-auto-merge merged commit ac02370 into dev Oct 24, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-14112-new-component-password-reset-demand-sending branch October 24, 2024 08:08
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.

4 participants