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] Stocker la préférence de locale "International Français" #463

Merged
merged 2 commits into from
Jan 6, 2023

Conversation

yannbertrand
Copy link
Member

@yannbertrand yannbertrand commented Jan 6, 2023

🎄 Problème

Sur la page de sélection de locale préférée, le clic sur "International Français" ne déclenche pas l’événement i18n.onBeforeLanguageSwitch car c'est la locale par défaut. À cause de ça, le cookie locale=fr n'est pas positionné et l'utilisateur n'est ensuite pas redirigé automatiquement sur sa préférence.

De plus, au build Pix Pro, nous n'avons plus d'index.html dans /dist/pix.org depuis la #455 (pages/pix-pro/index.vue). Donc Nginx renvoie une 403 avant de nous rediriger côté client par la page d'erreur. On constate donc un clignotement de la page d'erreur.

🎁 Proposition

Ne plus déclarer de defaultLocale car on souhaite ne pas avoir de locale par défaut :).

Cette solution a un impact identifié : les pages accessibles sans préfixe de locale /*** ne sont plus redirigées vers /fr/***. À ma connaissance, la seule page concernée est la page d'erreur où on retrouve donc la clé de traduction "error-content" affichée telle quelle. Pas d'impact sur les assets static accessibles via /images/background-comment-ca-marche.jpg (je suppose que nuxt-i18n ne passe pas dessus).

La config i18n, en particulier fallbackLocale devrait corriger ce problème mais semble ne pas fonctionner malheureusement. Ce sera à creuser par la suite.

Sur Pix Pro, on souhaite toujours une defaultLocale pour rediriger automatiquement l'utilisateur vers /fr. On en profite pour renommer le fichier locale-home.vue en index.vue pour créer un index.html à la racine de pro.pix.org. On a testé une autre approche en créant un fichier /pages/pix-pro/index.vue minimal (avec un i18n à false) qu'on pensait être suffisant mais qui n'effectuait finalement pas de redirection donc on arrivait sur une page blanche...

🌟 Remarques

Tout est dis ?

🎅 Pour tester

Vérifier que le stockage de la locale=fr au clic sur "International Français" est corrigé.

Vérifier que les assets statics (dossier assets et static) sont toujours accessibles.

Constater que la page d'erreur non localisée contient seulement le message "error-content".

@pix-service
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles ici.

Base automatically changed from pix-6506-save-locale-preferences-without-locale-choice-page to dev January 6, 2023 15:27
@yannbertrand yannbertrand force-pushed the fix-international-fr-preferred-locale branch from 86f283a to 42c2590 Compare January 6, 2023 15:31
@yannbertrand yannbertrand force-pushed the fix-international-fr-preferred-locale branch from 42c2590 to bb90764 Compare January 6, 2023 15:35
@yannbertrand
Copy link
Member Author

Le retour en arrière du navigateur re déclenche le i18n.onBeforeLanguageSwitch, donc faire 2 changements de locale puis "Précédant" change la valeur du cookie.

@yannbertrand yannbertrand force-pushed the fix-international-fr-preferred-locale branch from e88574e to 03cbe52 Compare January 6, 2023 16:18
@yannbertrand yannbertrand force-pushed the fix-international-fr-preferred-locale branch from 03cbe52 to 02431e8 Compare January 6, 2023 16:19
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.

6 participants