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] Mettre à jour le contenu de l'email "Création de compte" (PIX-12924) #9417

Merged

Conversation

er-lim
Copy link
Contributor

@er-lim er-lim commented Jul 2, 2024

🤖 Contexte

Inciter l’utilisateur ayant reçu cet email à cliquer sur le bouton pour valider son adresse email lorsqu'il reçoit le mail de création de son compte.

  • Après le merge de cette PR, synchroniser les nouvelles clés dans Phrase (ici pix-account-creation-email.params.subtitleDescription) et faire la traduction en nl et es.
  • Modification et validation du template Brevo en production ?

🤖 Proposition

  1. Modification des textes fr, en, nl et es pour les paramètres du template d'email de création de compte.
  2. Modification de la clé pix-account-creation-email.params.title pour avoir un clé dynamique avec le prénom de l'utilisateur.
  • nécessite le passage du prénom de l'utilisateur
  • nécessite l'utilisation de i18n

🌈 Remarques

  • Le template Brevo pour la création de compte a été modifié pour ajouter un champ subtitleDescription correspondant à la clé de traduction pix-account-creation-email.params.subtitleDescription

🫡 Scout rules

En voulant comprendre la configuration et l'intégration i18n dans HAPI, je me suis rendu compte qu'il n'y avait aucun tests dessus, j'ai donc ajouté des tests d'intégration de la config i18n et du plugin hapi-i18n.

💯 Pour tester

  • Aller sur la RA de Pix App .org
  • S'inscrire en saisissant une adresse email auquelle vous avez accès
  • Vérifier dans le mail de création de compte que le contenu est bien à jour.
  • Répéter ces étapes en changeant la locale, avec le language switcher, afin de tester le contenu en anglais, espagnol et néerlandais.

image

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

@er-lim er-lim force-pushed the pix-12924-update-pix-account-creation-email-content branch from 959e7af to fd57837 Compare July 2, 2024 15:28
@bpetetot bpetetot force-pushed the pix-12924-update-pix-account-creation-email-content branch 2 times, most recently from 647d111 to a617691 Compare July 9, 2024 12:18
@bpetetot bpetetot self-assigned this Jul 9, 2024
@bpetetot bpetetot force-pushed the pix-12924-update-pix-account-creation-email-content branch 4 times, most recently from 4c053ce to c075def Compare July 9, 2024 15:37
@bpetetot bpetetot marked this pull request as ready for review July 9, 2024 16:25
@bpetetot bpetetot requested a review from a team as a code owner July 9, 2024 16:25
Copy link
Contributor

@mariannebost mariannebost 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é sur Chrome 👌

"helpdeskLinkLabel": "here",
"moreOn": "More on",
"pixPresentation": "Pix is the online public service to assess, develop and certify your digital skills.",
"subtitle": "You can now start testing your skills."
"subtitle": "Thank you for creating your account. There is one last step to secure your account: validating your email address.",
"subtitleDescription": "It is very simple, please click on the button below and login to your Pix account."
Copy link
Contributor

Choose a reason for hiding this comment

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

C"est juste un détail mais pourquoi pas juste description je trouve ca un peu bizarre que se soit la description du sous titre 🤔

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 en RA

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12924-update-pix-account-creation-email-content branch from 7cd42ce to 0d8d6a1 Compare July 11, 2024 15:05
@pix-service-auto-merge pix-service-auto-merge merged commit e876876 into dev Jul 11, 2024
4 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12924-update-pix-account-creation-email-content branch July 11, 2024 15:11
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.

None yet

6 participants