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] Changer des messages sur les pages de connexion et réinitialisation (PIX-15996) #11088

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

theotime2005
Copy link
Contributor

@theotime2005 theotime2005 commented Jan 13, 2025

🎄 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

  • Les traductions en es et nl n'ont pas été modifiée.
  • Des liens sont écrit en dur dans les sections de traduction concerné,
  • Les liens de la traduction française redirige vers ppix.fr systématiquement.

🎅 Pour tester

Pour la page de réinitialisation de mot de passe

  • Se rendre sur cette page,
  • Constater en français puis en anglais que les messages correspondent à ce qui était attendu, soit:
    • “Un problème pour vous connecter à Pix ? Consultez le centre d’aide”,
    • “Having trouble connecting to Pix ? Consult the help center”.

Pour le message d'erreur

  • En local, modifier la route de connexion pour renvoyer systématiquement une erreur,
  • Constater la présence en anglais et français des nouveaux messages, soit:
    • “Impossible de se connecter. Veuillez réessayer dans quelques instants. Si le problème persiste, contactez-nous via le centre d’aide”,
    • “‘Unable to connect. Please try again in a few moments. If the problem persists, please contact us via the help center’

@theotime2005 theotime2005 self-assigned this Jan 13, 2025
@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 :

@EmmanuelleBonnemay
Copy link
Contributor

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 ?

@theotime2005 theotime2005 force-pushed the pix-15996-change-message-for-login-pages branch from 53cb2f5 to 35496b5 Compare January 15, 2025 09:56
@theotime2005
Copy link
Contributor Author

Pour l'instant les liens sont écrit en dur dans les testes. Doit-on les modifier dans cette PR?

@theotime2005 theotime2005 force-pushed the pix-15996-change-message-for-login-pages branch from 35496b5 to e68c3c4 Compare January 15, 2025 10:21
@EmmanuelleBonnemay
Copy link
Contributor

EmmanuelleBonnemay commented Jan 16, 2025

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, 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.

@theotime2005
Copy link
Contributor Author

theotime2005 commented Jan 16, 2025 via email

@@ -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"
Copy link
Contributor

@lego-technix lego-technix Jan 16, 2025

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"
Copy link
Contributor

@lego-technix lego-technix Jan 16, 2025

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.

@theotime2005 theotime2005 force-pushed the pix-15996-change-message-for-login-pages branch from e68c3c4 to e1650e6 Compare January 17, 2025 15:25
@theotime2005
Copy link
Contributor Author

La page d'inscription est également touchée par le message d'erreur lors de la tentative de conexion.

@theotime2005 theotime2005 force-pushed the pix-15996-change-message-for-login-pages branch from e1650e6 to 39deb6a Compare January 17, 2025 15:49
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.

4 participants