-
Notifications
You must be signed in to change notification settings - Fork 3
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
[FEATURE] Ne pas passer par la page /locale-choice
pour choisir sa locale
#459
[FEATURE] Ne pas passer par la page /locale-choice
pour choisir sa locale
#459
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants : Les variables d'environnement seront accessibles ici. |
2156f9c
to
f1fdd5c
Compare
f6ed61e
to
25ac905
Compare
44be081
to
52975b4
Compare
ddd6d3a
to
38a4fe3
Compare
38a4fe3
to
b96a3f1
Compare
f1fdd5c
to
c35a05e
Compare
83701ae
to
6a9e0cc
Compare
6a9e0cc
to
bf15969
Compare
/locale-choice
/locale-choice
pour choisir sa locale
6457cc1
to
eb48807
Compare
eb48807
to
7daee2b
Compare
Détecté à ce moment mais était déjà le cas depuis la #455, le stockage de préférence "International FR" ne fonctionne pas sur la home (seulement cette locale) car elle correspond à la Sinon fonctionnellement, c'est ✅ |
🎄 Problème
La page de sélection de locale est accessible via
/locale-choice
ce qu'on souhaite éviter, car ça nous oblige à devoir toujours le supporter à l'avenir. Cette page est d'ailleurs exposée sur pix.fr (alors qu'elle n'y sert à rien). Elle avait été ajoutée pour fonctionner avec lerootRedirect
de nuxt-i18n mais cette piste a ensuite été abandonnée, ce n'est donc plus qu'un reliquat de cette lointaine époque.On constate aussi que le
$router.push
ajoute une entrée dans l'historique de l'utilisateur, ce qui fait qu'on ne peut pas utiliser le bouton précédent du navigateur qui nous redirige automatiquement sur la page courante.🎁 Proposition
Ayant maintenant une page
index
différente de la home des locales (locale-home
), on peut directement y rendre le sélecteur de locale.Le layout avait besoin d'être allégé car il se chargeait tout de même avant l'apparition de la page.
On remplace aussi le
$router.push
par un $router.replace` pour ne pas rentre inutilisable la navigation dans l'historique avec une boucle.🌟 Remarques
Impact en perfs ?
🎅 Pour tester
Procédure à suivre :
locale
qui pourrait être dans le navigateur/locale-choice
, on y choisit sa locale/fr
,/en-gb
ou/fr-be
. Constater dans le navigateur l'ajout d'un cookielocale
ayant pour valeur l'identifiant de la locale sélectionnée,fr
,en-gb
oufr-be
XXX/fr-be/les-tests
,/fr/pix-et-unesco
,XXX/en-gb/the-tests
, etc. etc.) et constater qu'aucune redirection n'a lieu