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
[TECH] Petit refacto du mail-service pour la gestion de la langue (PIX-12193). #8827
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants : Les variables d'environnement seront accessibles via les liens suivants : |
842e73f
to
e817790
Compare
e817790
to
90bb616
Compare
90bb616
to
e6cc665
Compare
f5a036c
to
bcbbff7
Compare
There was a problem hiding this 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/`, |
There was a problem hiding this comment.
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 🤔
8a0af3e
to
0afdc26
Compare
0afdc26
to
dbca21e
Compare
🦄 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
email-sender-name
pour Orga et Certif💯 Pour tester (préparez vous un petit café/thé, ca va etre long ☕ )
Pour
sendAccountCreationEmail
:Pour
sendResetPasswordDemandEmail
:Pour
sendOrganizationInvitationEmail
:superadmin
Pour
sendScoOrganizationInvitationEmail
:superadmin
Pour
sendCertificationCenterInvitationEmail
:superadmin
Pour
sendAccountRecoveryEmail
:Pour
sendVerificationCodeEmail
: