[TECH] Remplacer la locale "en-gb" en "en" (PIX-7514) #538

er-lim · 2023-05-19T11:54:58Z

☑️ Reste à faire

Mettre à jour les liens dans les documents Prismic en remplaçant en-gb par en (edit: peut être fait après le merge)

🦄 Problème

Aujourd’hui, la locale International English est gérée sur le site vitrine avec le code en-gb. Or ce code représente l’anglais britannique et non l’anglais international qui a pour code en. Cela pose aussi d’autres problèmes de cohérence par rapport aux Pix apps qui elles utilisent la valeur en pour la lang des utilisateurs.

🤖 Proposition

Remplacer l’utilisation de la valeur en-gb dans le site vitrine par la valeur en.

🌈 Remarques

Une précédente PR a déjà mis en place le remplacement dans le cookie locale des valeurs de locales en-GB et en-gb par en : [FEATURE] Remplacer la locale en-GB par en dans le cookie (PIX-7474) #522
La façon de récupérer le document Prismic pour la home de Pix Pro a été modifiée car ne fonctionne plus avec la locale en. Cela fonctionnait pour la locale en-gb car celui-ici contenait avant l'uid decouvrir-pix-pro et Prismic conserve les anciens uid du document. Ce mécanisme va être amélioré dans un prochain ticket.
La rétrocompatilité a été mise en place pour pouvoir lire les cookies avec une valeur en-gb (afin de renvoyer sur le /en du site) 😄

💯 Pour tester

Pix Site

Aller sur la RA .org https://site-pr538.review.pix.org/
Si vous avez déjà une locale en cookie, cliquez sur International English dans le Locale Switcher. Sinon cliquez sur International English
Vérifier que le lien est bien https://site-pr538.review.pix.org/en et que la page s'affiche bien en anglais
Aller sur https://site-pr538.review.pix.org/en-gb et constater que la redirection se fait bien en https://site-pr538.review.pix.org/en.
Changer la valeur du cookie locale en en-gb.
Rafraîchir la page et constater que ça fonctionne et que la valeur du cookie locale a été remis en en
Parcourir les liens du menu (Discover Pix, The tests...).
Constater que les liens ont, pour le moment, toujours un path en /en-gb
Cliquer sur les liens et voir que l'url affichée est bien en /en.

Pix Pro

Aller sur la RA .org https://pro-pr538.review.pix.org/
Si vous avez déjà une locale en cookie, cliquez sur International English dans le Locale Switcher. Sinon cliquez sur International English
Vérifier que le lien est bien https://pro-pr538.review.pix.org/en et que la page s'affiche bien en anglais
Aller sur https://pro-pr538.review.pix.org/en-gb et constater que la redirection se fait bien en https://pro-pr538.review.pix.org/en.
Changer la valeur du cookie locale en en-gb.
Rafraîchir la page et constater que ça fonctionne et que la valeur du cookie locale a été remis en en
Parcourir les liens du menu (Discover Pix Pro, Business...).
Constater que les liens ont, pour le moment, toujours un path en /en-gb
Cliquer sur les liens et voir que l'url affichée est bien en /en.

pix-bot-github · 2023-05-19T11:55:27Z

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

Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-site-review-pr538/environment

reibecca · 2023-05-23T09:34:02Z

Tests fonctionnels pix-site & pix-pro sur Chrome :

✅ Aller sur /en et vérifier que la page s'affiche bien en anglais
✅ Aller sur /en-gb et constater que la redirection se fait bien en /en
- ⚠️ Lorsque l'on passe /en-GB sur pix-site on est redirigé sur une page d'erreur
- ⚠️ Lorsque l'on passe /en-GB sur pix-pro on est redirigé sur https://pro-pr538.review.pix.org/fr/en-GB et la page "Oups" apparaît
❌ Changer la valeur du cookie locale en en-gb. Rafraîchir la page et constater que ça fonctionne et que la valeur du cookie locale a été remis en en
- ➡️ après avoir rafraichi la page, le cookie garde toujours la valeur en-gb
Parcourir les liens du menu (Discover Pix, The tests...) :
- ✅ Constater que les liens ont, pour le moment, toujours un path en /en-gb
- ✅ Cliquer sur les liens et voir que l'url affichée est bien en /en.

pages/pix-pro/locale-home.vue

lego-technix · 2023-05-24T07:17:57Z

Lorsque l'on passe /en-GB sur pix-site on est redirigé sur une page d'erreur

warning Lorsque l'on passe /en-GB sur pix-pro on est redirigé sur https://pro-pr538.review.pix.org/fr/en-GB et la page "Oups" apparaît

Je ne pense pas que devoir gérer des URL de la forme /en-GB soit à faire car ces URL n'ont jamais existé il me semble.

lego-technix

Comme l'a déjà noté @reibecca, si on est sur une page (comme http://dev.pix.org/en/the-tests/) et qu'on recharge cette page ou bien qu'on navigue sur une autre page, un cookie avec la valeur en-gb garde cette valeur au lieu d'avoir sa valeur modifiée en en.

À part ce point, le reste est super 😃

er-lim · 2023-05-25T07:16:32Z

@reibecca @lego-technix J'ai ajouté, dans les derniers commits, la gestion du cookie locale avec une valeur en-GB. Lorsque qu'un visiteur arrive sur la page et qu'il avait déjà un cookie avec cette valeur là, le cookie sera bien mis à jour avec une valeur en.

Je vous laisse tester 😄

services/cookie.js

lego-technix · 2023-05-25T08:44:30Z

✅ Testé fonctionnellement avec succès avec Firefox

reibecca · 2023-05-25T09:33:48Z

Lorsque l'on passe /en-GB sur pix-site on est redirigé sur une page d'erreur

warning Lorsque l'on passe /en-GB sur pix-pro on est redirigé sur https://pro-pr538.review.pix.org/fr/en-GB et la page "Oups" apparaît

Je ne pense pas que devoir gérer des URL de la forme /en-GB soit à faire car ces URL n'ont jamais existé il me semble.

Actuellement en production si tu vas sur pix-site ou pix-pro en .org et que tu sélectionnes la locale International English tu tombe sur un URL avec /en-gb. Et si tu la modifies en écrivant /en-GB tu ne tombes pas sur une page d'erreur, tu restes bien sur la page en anglais. Donc j'ai l'impression qu'il faut gérer ce cas ici.

reibecca · 2023-05-25T09:34:47Z

✅ Testé fonctionnellement à nouveau sur Chrome et c'est bon pour le cookie 😄

lego-technix · 2023-05-25T09:37:43Z

Actuellement en production si tu vas sur pix-site ou pix-pro en .org et que tu sélectionnes la locale International English tu tombe sur un URL avec /en-gb. Et si tu le modifies en écrivant /en-GB tu ne tombes pas sur page d'erreur, tu restes bien sur la page en anglais. Donc j'ai l'impression qu'il faut gérer ce cas ici.

Personne n'est censé modifier un URL en /en-gb pour le changer en /en-GB. Et il n'y a jamais eu de contenus publiés sur Prismic dans une locale avec la valeur en-GB (c'est d'ailleurs un, tout petit, problème de leur implémentation : Prismic n'utilise pas la valeur canonique recommandée par les RFC).

Alors que pour les cookies c'est différent : les différentes valeurs en-gb et en-GB se sont retrouvées dans les cookies des visiteurs au fur et à mesure des évolutions que nous avons faites.

er-lim · 2023-05-25T10:08:48Z

Actuellement en production si tu vas sur pix-site ou pix-pro en .org et que tu sélectionnes la locale International English tu tombe sur un URL avec /en-gb. Et si tu le modifies en écrivant /en-GB tu ne tombes pas sur page d'erreur, tu restes bien sur la page en anglais. Donc j'ai l'impression qu'il faut gérer ce cas ici.

Personne n'est censé modifier un URL en /en-gb pour le changer en /en-GB. Et il n'y a jamais eu de contenus publiés sur Prismic dans une locale avec la valeur en-GB (c'est d'ailleurs un, tout petit, problème de leur implémentation : Prismic n'utilise pas la valeur canonique recommandée par les RFC).

Alors que pour les cookies c'est différent : les différentes valeurs en-gb et en-GB se sont retrouvées dans les cookies des visiteurs au fur et à mesure des évolutions que nous avons faites.

La remarque de @reibecca est intéressante car en effet si https://pix.org/en-gb et https://pix.org/en-GB sont accessibles et donc référencées, il faut aussi rediriger le trafic de /en-GB vers /en nn ? 🤔
Je parles surtout du point de vue SEO 😄

lego-technix · 2023-05-25T10:11:56Z

La remarque de @reibecca est intéressante car en effet si https://pix.org/en-gb et https://pix.org/en-GB sont accessibles et donc référencées (possiblement contenu dupliqué d'ailleurs...) , il faut aussi rediriger le trafic de /en-GB vers /en nn ? thinking
Je parles surtout du point de vue SEO

Je répondais précisément à ce point : il n'y a jamais jamais jamais eu de contenu en ligne commençant par l'URL https://pix.org/en-GB. Et c'est une certitude car Prismic ~~ne le permet pas~~ n'a pas la valeur de la locale sous forme canonique.

Évidemment si de tels URL avaient existé nous aurions fait les redirections pour ces URL aussi, de manière à être le mieux possible en terme de SEO.

lego-technix · 2023-05-25T10:16:39Z

Et je dirais même : effectuer des redirections pour des URL commençant par /en-GB (par exemple https://pix.org/en-GB, etc.) pourrait induire en erreur les développeurs faisant des évolutions/maintenance sur le projet en leur laissant penser qu'il y a eu un jour des contenus avec en-GB dans l'URL et qu'il y a eu un jour des contenus avec la locale en-GB dans Prismic. Ce qui n'a jamais été le cas.

er-lim · 2023-05-25T10:27:11Z

Et je dirais même : effectuer des redirections pour des URL commençant par /en-GB (par exemple https://pix.org/en-GB, etc.) pourrait induire en erreur les développeurs faisant des évolutions/maintenance sur le projet en leur laissant penser qu'il y a eu des contenus avec en-GB dans l'URL et qu'il y a eu des contenus avec la locale en-GB dans Prismic. Ce qui n'a jamais été le cas.

En effet, on tombe sur une 404 ce qui valide ton hypothèse. L'url /en-GB n'est pas indexé 👍
Complètement d'accord pour ne pas traiter ce cas là 😄

Co-authored-by: LEGO Technix <109212476+lego-technix@users.noreply.github.com

er-lim added 🚧 Development in progress team-acces labels May 19, 2023

er-lim self-assigned this May 19, 2023

er-lim changed the title ~~Pix 7514 replace en gb locale with en~~ [TECH] Remplacer la locale "en-gb" en "en" (PIX-7514) May 19, 2023

er-lim force-pushed the pix-7514-replace-en-gb-locale-with-en branch from aff5050 to 6c91a9d Compare May 22, 2023 15:04

er-lim marked this pull request as ready for review May 22, 2023 15:29

er-lim added 👀 Tech Review Needed 👀 Func Review Needed and removed 🚧 Development in progress labels May 23, 2023

lego-technix reviewed May 24, 2023

View reviewed changes

pages/pix-pro/locale-home.vue Outdated Show resolved Hide resolved

lego-technix requested changes May 24, 2023

View reviewed changes

er-lim force-pushed the pix-7514-replace-en-gb-locale-with-en branch from 70eebca to 823079c Compare May 25, 2023 08:39

lego-technix approved these changes May 25, 2023

View reviewed changes

services/cookie.js Outdated Show resolved Hide resolved

reibecca approved these changes May 25, 2023

View reviewed changes

er-lim force-pushed the pix-7514-replace-en-gb-locale-with-en branch 2 times, most recently from 483ba5a to a6a45e0 Compare May 26, 2023 07:04

er-lim removed 👀 Tech Review Needed 👀 Func Review Needed labels May 26, 2023

er-lim added Func Review OK Tech Review OK 🚀 Ready to Merge labels May 26, 2023

er-lim added 11 commits May 26, 2023 07:45

build: add /en-gb to en permanent redirection

7107bd2

Co-authored-by: LEGO Technix <109212476+lego-technix@users.noreply.github.com

feat: rename en-gb locale with en in config directory

831eabf

chore: remove en-gb condition in _setLocaleCookie

21057e0

feat: handle en-gb cookie value

23398d1

chore: rename locale en-gb with en in components

70d2c5c

build: add test for redirection en-gb to en

b8343f8

fix: correct uid for pix-pro home document

6dbb966

refactor: improve constants naming in pix-pro locale-home

c01a2fa

feat: add cookie service

f0ad0ad

refactor: use cookie service in project

6a71155

feat: replace locale cookie value with en when getting en-gb

169ab40

pix-service-auto-merge force-pushed the pix-7514-replace-en-gb-locale-with-en branch 2 times, most recently from 63e3dcb to 169ab40 Compare May 26, 2023 07:45

pix-service-auto-merge merged commit 0c8aad0 into dev May 26, 2023

pix-service-auto-merge deleted the pix-7514-replace-en-gb-locale-with-en branch May 26, 2023 07:46

lego-technix mentioned this pull request Jul 16, 2024

[TECH] Remplacer la locale "en-gb" obsolète par "en" (PIX-13458) 1024pix/pix#9538

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[TECH] Remplacer la locale "en-gb" en "en" (PIX-7514) #538

[TECH] Remplacer la locale "en-gb" en "en" (PIX-7514) #538

er-lim commented May 19, 2023 •

edited by lego-technix

Loading

pix-bot-github commented May 19, 2023

reibecca commented May 23, 2023 •

edited

Loading

lego-technix commented May 24, 2023

lego-technix left a comment •

edited

Loading

er-lim commented May 25, 2023

lego-technix commented May 25, 2023

reibecca commented May 25, 2023 •

edited

Loading

reibecca commented May 25, 2023

lego-technix commented May 25, 2023 •

edited

Loading

er-lim commented May 25, 2023 •

edited

Loading

lego-technix commented May 25, 2023 •

edited

Loading

lego-technix commented May 25, 2023 •

edited

Loading

er-lim commented May 25, 2023 •

edited

Loading

[TECH] Remplacer la locale "en-gb" en "en" (PIX-7514) #538

[TECH] Remplacer la locale "en-gb" en "en" (PIX-7514) #538

Conversation

er-lim commented May 19, 2023 • edited by lego-technix Loading

☑️ Reste à faire

🦄 Problème

🤖 Proposition

🌈 Remarques

💯 Pour tester

Pix Site

Pix Pro

pix-bot-github commented May 19, 2023

reibecca commented May 23, 2023 • edited Loading

lego-technix commented May 24, 2023

lego-technix left a comment • edited Loading

Choose a reason for hiding this comment

er-lim commented May 25, 2023

lego-technix commented May 25, 2023

reibecca commented May 25, 2023 • edited Loading

reibecca commented May 25, 2023

lego-technix commented May 25, 2023 • edited Loading

er-lim commented May 25, 2023 • edited Loading

lego-technix commented May 25, 2023 • edited Loading

lego-technix commented May 25, 2023 • edited Loading

er-lim commented May 25, 2023 • edited Loading

er-lim commented May 19, 2023 •

edited by lego-technix

Loading

reibecca commented May 23, 2023 •

edited

Loading

lego-technix left a comment •

edited

Loading

reibecca commented May 25, 2023 •

edited

Loading

lego-technix commented May 25, 2023 •

edited

Loading

er-lim commented May 25, 2023 •

edited

Loading

lego-technix commented May 25, 2023 •

edited

Loading

lego-technix commented May 25, 2023 •

edited

Loading

er-lim commented May 25, 2023 •

edited

Loading