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] Sur Pix App .org corriger les liens vers le support qui ne sont pas bons sur les pages de réinitialisation du mot de passe (PIX-16033) #11065

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

Conversation

lego-technix
Copy link
Contributor

@lego-technix lego-technix commented Jan 10, 2025

🎄 Problème

Sur Pix App .org, sur les pages de réinitialisation du mot de passe, les liens vers le support (les liens avec le texte « Contactez-nous ») ne sont pas les bons.

On note notamment les cas problématiques suivants :

🎁 Proposition

Le problème vient que les liens vers le support étaient écrits en dur dans les fichiers de traduction, au lieu d'être calculés plus intelligemment et de manière centralisée dans un service adapté.

La proposition est, pour le formulaire de demande de réinitialisation de mot de passe (password-reset-demand-form.gjs), de récupérer l'URL du support en le demandant à l’urlService.

On profite de cette PR pour faire évoluer un peu l'urlService pour qu'il gère plus génériquement les locales à l'aide du localeService au lieu de faire du spécifique pour chacune de ses fonctions.

🧦 Remarques

À noter que le lien vers le support fournit par le code de cette PR est https://pix.org/nl-BE/support et non https://pix.org/nl-be/support car cette PR s'aligne au maximum sur l’ADR Logique et stratégie de gestion des paramètres régionaux et des langues (locales & languages)

Il faudrait utiliser systématiquement l’urlService pour récupérer des URL et ne jamais en récupérer dans des fichiers de traduction.

🎅 Pour tester

Pix App .org

Pix App .fr

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

@lego-technix lego-technix force-pushed the pix-16033-fix-pix-app-support-links branch from 5b08e6a to a38a216 Compare January 10, 2025 12: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.

2 participants