-
Notifications
You must be signed in to change notification settings - Fork 56
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] Changer des messages sur les pages de connexion et réinitialisation (PIX-15996) #11088
base: dev
Are you sure you want to change the base?
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 : |
ef2a18f
to
53cb2f5
Compare
Deux petites remarques : il y a deux tests à corriger ; et aussi : en anglais, conserve-t-on l'espace entre le mot et le point d'interrogation ? |
53cb2f5
to
35496b5
Compare
Pour l'instant les liens sont écrit en dur dans les testes. Doit-on les modifier dans cette PR? |
35496b5
to
e68c3c4
Compare
Oui, c'est préférable, ainsi les changements de wording ultérieurs n'affecteront pas les tests ; par ailleurs, dans ce fichier de test, presque tous les textes sont en clés de traduction. |
Sauf que si on les modifie dans cette PR, il faut modifier les traductions certes, mais aussi modifier les composants en eux-même, ce qui change le sujet non?
… Le 16 janv. 2025 à 09:10, EmmanuelleBonnemay ***@***.***> a écrit :
Pour l'instant les liens sont écrit en dur dans les testes. Doit-on les modifier dans cette PR?
Oui, c'est préférable
—
Reply to this email directly, view it on GitHub <#11088 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BC2PMRQAWELWOU5ZDOO5FY32K5SQRAVCNFSM6AAAAABVCJ246GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOJUG44DENBXGA>.
You are receiving this because you were assigned.
|
@@ -184,7 +184,8 @@ | |||
"receive-reset-button": "Receive a reset link" | |||
}, | |||
"contact-us-link": { | |||
"link-text": "Contact us" | |||
"link-text": "Consult the help center", | |||
"link-url": "https://pix.org/en/support" |
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.
Ne pas mettre d'URL dans les fichiers de traduction et utiliser l'urlService pour obtenir le bon URL.
@@ -184,7 +184,8 @@ | |||
"receive-reset-button": "Recevoir un lien de réinitialisation" | |||
}, | |||
"contact-us-link": { | |||
"link-text": "Contactez-nous" | |||
"link-text": "Consultez le centre d'aide", | |||
"link-url": "https://pix.fr/support" |
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.
Ne pas mettre d'URL dans les fichiers de traduction et utiliser l'urlService pour obtenir le bon URL.
e68c3c4
to
e1650e6
Compare
La page d'inscription est également touchée par le message d'erreur lors de la tentative de conexion. |
e1650e6
to
39deb6a
Compare
🎄 Problème
Dans la page de réinitialisation du mot de passe, ainsi que lors d'une erreur de connexion, les messages mentionnaient un contact directt avec Pix alors qu'il s'agit plutôt d'accéder au centre d'aide.
🎁 Proposition
Modifier les traductions anglaises et françaises pour les ajuster en conséquence.
🧦 Remarques
🎅 Pour tester
Pour la page de réinitialisation de mot de passe
Pour le message d'erreur