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

[TECH] Petit refacto du mail-service pour la gestion de la langue (PIX-12193). #8827

Conversation

mariannebost
Copy link
Contributor

@mariannebost mariannebost commented May 2, 2024

🦄 Problème

Lors de l'ajout d'une nouvelle langue, les modifications à apporter sur le mail-service sont nombreuses et éparpillées sur les différentes fonctions de création de mail.

🤖 Proposition

Factoriser la gestion des langues pour rendre plus simple l'ajout ou la modification d'une langue dans le fichier mail-service

🌈 Remarques

  • Les 5 premiers commits permettent d'extraire les variables nécessaires pour chaque templates et le 6e commit les ajoute dans une fonction.
  • Ajout des traductions email-sender-name pour Orga et Certif

💯 Pour tester (préparez vous un petit café/thé, ca va etre long ☕ )

  • Vérifier que les tests passent ✅

Pour sendAccountCreationEmail :

  • Se créer un compte sur Pix app avec votre adresse email
  • Vérifier la bonne réception du mail de confirmation avec toutes les informations bien renseignées

Pour sendResetPasswordDemandEmail :

  • Aller sur la page de connection de Pix App et cliquer sur le lien "Mot de passe oublié ?"
  • Renseigner votre mail déjà utiliser pour un compte Pix
  • Vérifier la bonne réception du mail de confirmation avec toutes les informations bien renseignées

Pour sendOrganizationInvitationEmail :

  • Se connecter à Pix Admin avec le compte superadmin
  • Aller sur une organisation puis dans l'onglet "Invitations"
  • Remplir le champ "Adresse e-mail du membre à inviter" avec son adresse mail
  • Vérifier la bonne réception du mail de confirmation avec toutes les informations bien renseignées Capture d’écran 2024-05-10 à 18 28 14

Pour sendScoOrganizationInvitationEmail :

  • Se connecter à Pix Admin avec le compte superadmin
  • Aller sur une organisation sco ex: "Lycée Joséphine Baker"
  • Modifier les informations et mettre votre mail dans le champ "Adresse e-mail d'activation SCO"
  • Aller sur Pix Orga avec le compte allorga et bien sélectionner le lycée en haut à droite
  • S'ajouter en tant que membre admin dans l'organisation
  • Aller dans vos mail et confirmer l'invitation
  • Aller sur cette url : https://orga-pr8827.review.pix.org/demande-administration-sco
  • Et renseigner l'UAI de l'organisation ("ACCESS_SCO_BAUDELAIRE") et votre nom/prénom
  • Vérifier la bonne réception du mail de confirmation avec toutes les informations bien renseignées Capture d’écran 2024-05-10 à 18 09 16

Pour sendCertificationCenterInvitationEmail :

  • Se connecter à Pix Admin avec le compte superadmin
  • Aller sur un centre de certification puis dans l'onglet "Invitations"
  • Remplir le champ "Adresse e-mail du membre à inviter" avec son adresse mail
  • Vérifier la bonne réception du mail de confirmation avec toutes les informations bien renseignées Capture d’écran 2024-05-10 à 18 17 52

Pour sendAccountRecoveryEmail :

  • Se rendre sur l'url : https://app-pr8827.review.pix.fr/recuperer-mon-compte
  • Récupérer un élève avec un INE en base et remplir le formulaire avec ses info (ex: 123456789NR ; Nico ; Robin)
  • Vérifier la bonne réception du mail de confirmation avec toutes les informations bien renseignées Capture d’écran 2024-05-14 à 16 15 19

Pour sendVerificationCodeEmail :

  • Se connecter à Pix App et aller sur "Mon compte"
  • Dans l'onglet "Mes méthodes de connections" cliquer sur "Modifier" à coté de "Adresse e-mail"
  • Remplir le formulaire avec une nouvelle adresse (ou avec un alias ex: test+email)
  • Vérifier la bonne réception du mail de confirmation avec toutes les informations bien renseignées Capture d’écran 2024-05-10 à 18 33 02

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

@mariannebost mariannebost force-pushed the pix-12193-refactor-language-management-in-mail-service branch 5 times, most recently from 842e73f to e817790 Compare May 7, 2024 13:52
@mariannebost mariannebost marked this pull request as ready for review May 7, 2024 14:02
@mariannebost mariannebost force-pushed the pix-12193-refactor-language-management-in-mail-service branch from e817790 to 90bb616 Compare May 7, 2024 14:05
api/lib/domain/services/mail-service.js Outdated Show resolved Hide resolved
api/lib/domain/services/mail-service.js Outdated Show resolved Hide resolved
api/translations/es.json Outdated Show resolved Hide resolved
@mariannebost mariannebost force-pushed the pix-12193-refactor-language-management-in-mail-service branch from 90bb616 to e6cc665 Compare May 10, 2024 08:57
api/lib/domain/services/mail-service.js Outdated Show resolved Hide resolved
api/lib/domain/services/mail-service.js Outdated Show resolved Hide resolved
api/lib/domain/services/mail-service.js Outdated Show resolved Hide resolved
@mariannebost mariannebost force-pushed the pix-12193-refactor-language-management-in-mail-service branch from f5a036c to bcbbff7 Compare May 14, 2024 09:10
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 sur Firefox en RA ✅

// INTERNATIONAL
const PIX_HOME_NAME_INTERNATIONAL = `pix${config.domain.tldOrg}`;
const PIX_HOME_URL_INTERNATIONAL = {
en: `${config.domain.pix + config.domain.tldOrg}/en-gb/`,
Copy link
Contributor

Choose a reason for hiding this comment

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

(hors scope de la PR) Il faudra penser plus tard à repasser sur les liens de https://pix.org/en-gb pour les changer en https://pix.org/en.
On utilise la locale en maintenant au lieu de en-gb même si une redirection est faite de https://pix.org/en-gb => /en pour la rétrocompatibilité.

Je me dis que garder ce lien dans le code peut porter à confusion par rapport à ça 🤔

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12193-refactor-language-management-in-mail-service branch 2 times, most recently from 8a0af3e to 0afdc26 Compare May 15, 2024 09:26
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12193-refactor-language-management-in-mail-service branch from 0afdc26 to dbca21e Compare May 15, 2024 09:32
@pix-service-auto-merge pix-service-auto-merge merged commit e7b9db2 into dev May 15, 2024
4 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12193-refactor-language-management-in-mail-service branch May 15, 2024 09:37
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

5 participants