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] Utiliser la locale sur la demande de reset de password (PIX-15065) #10497

Merged

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Nov 6, 2024

🍂 Problème

Lors de la demande de reset de mot de passe (sur le nouveau design uniquement), l'utilisateur reçoit un email avec un lien vers une redirection sur pix.org.

🌰 Proposition

La locale n'était pas envoyée dans le header car on utilise fetch qui ne reprends pas tous les headers nécessaires aux requête vers l'API.

Pour éviter cette erreur dans le futur sur l'utilisation de fetch, un nouveau service a été ajouté utilisant le mécanisme de RequestManager d'ember-data.

Ce nouveau service basé sur RequestManager fonctionne avec fetch et inclu par défaut les headers suivants:

  • Authorization: si l'utilisateur est authentifié
  • Accept-Language: la locale de l'utilisateur
  • X-App-Version: la version de l'application
  • Accept: application/json
  • Content-Type: application/json

De plus, le service sérialise automatiquement le body des requêtes.

Exemple:

@service requestManager;

const { response } = await this.requestManager.request({
  url: `${ENV.APP.API_HOST}/api/password-reset-demands`,
  method: 'POST',
  body: { email },
});

🎃 Remarques

Note

Il y a quelques différences avec fetch

  • La fonction request() retourne un objet de type:
interface StructuredDocument<T> {
  request: RequestInfo;
  response: ResponseInfo | null;
  content?: T;
}
  • En cas d'erreur (status !== 2XX), une exception est jetée.

🪵 Pour tester

Sur pix.fr

Aller sur la page de réinitialisation de mot de passe.

  1. Saisir un email existant: [email protected]
  • Affiche l'écran de succès d'envoie de l'email.
  • Vérifier l'email reçu, il doit être en Français.
  • Cliquer dans le lien de l'email, il doit rediriger sur pix.fr avec la langue fr.
  1. Saisir un email qui n'existe pas: [email protected]
  • Affiche l'écran de succès d'envoie de l'email.
  • Aucun email n'est envoyé.

Sur pix.org en FR

Aller sur la page de réinitialisation de mot de passe.

  1. Saisir un email existant: [email protected]
  • Affiche l'écran de succès d'envoie de l'email.
  • Vérifier l'email reçu, il doit être en Français.
  • Cliquer dans le lien de l'email, il doit rediriger sur pix.org avec la langue fr.

Sur pix.org en NL

  1. Changer la langue en NL
  2. Saisir un email existant: [email protected]
  • Affiche l'écran de succès d'envoie de l'email.
  • Vérifier l'email reçu, il doit être en Néerlandais.
  • Cliquer dans le lien de l'email, il doit rediriger sur pix.org avec la langue NL.

@bpetetot bpetetot self-assigned this Nov 6, 2024
@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 :

@bpetetot bpetetot force-pushed the pix-15065-fix-lang-on-password-reset-demand branch from cf19fd1 to fecac63 Compare November 6, 2024 13:15
@bpetetot bpetetot marked this pull request as ready for review November 6, 2024 13:49
@bpetetot bpetetot changed the title [BUG] Utiliser la locale sur la demande de reset de password (PIX-15065) [BUGFIX] Utiliser la locale sur la demande de reset de password (PIX-15065) Nov 6, 2024
@EmmanuelleBonnemay
Copy link
Contributor

J'ai modifié la description de la PR : l'utilisateur reçoit un email en anglais avec un lien > l'utilisateur reçoit un email avec un lien...
(L'email est en français).

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

@bpetetot bpetetot force-pushed the pix-15065-fix-lang-on-password-reset-demand branch from fecac63 to 29d8628 Compare November 8, 2024 09:39
Copy link
Contributor

@er-lim er-lim left a comment

Choose a reason for hiding this comment

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

Test en RA sur firefox ✅

@bpetetot bpetetot force-pushed the pix-15065-fix-lang-on-password-reset-demand branch from 29d8628 to 9165b88 Compare November 8, 2024 11:24
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15065-fix-lang-on-password-reset-demand branch from 69acc45 to 9b03bcb Compare November 8, 2024 13:45
@pix-service-auto-merge pix-service-auto-merge merged commit ac821b2 into dev Nov 8, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15065-fix-lang-on-password-reset-demand branch November 8, 2024 13:52
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