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] Corriger le problème des locales disponibles non-identiques entre pix-site à pix-pro (PIX-14143) #718

Conversation

lego-technix
Copy link
Contributor

@lego-technix lego-technix commented Oct 2, 2024

🦄 Problème

Quand on a un cookie de locale positionné par Pix Site avec une locale non-disponible sur Pix Pro, on a une erreur « error-content » quand on va sur Pix Pro.

Dans tous les pix-sites il manque un fallback lorsque la locale spécifiée dans le cookie locale n’existe pas dans le pix-site en question.

🤖 Proposition

Implémenter une fonction getBestMatchingLocaleName pour présenter la meilleure page localisée à l'utilisateur en fonction des locales disponibles sur le pix-site visité.

🌈 Remarques

RAS

💯 Pour tester

Il y a une différence de comportement entre le comportement des pix-sites entre la production, en local et dans les RA. En effet les RA des pix-site et des pix-pro ne sont pas sur le même domaine.

Donc pour ce ticket aucun test n'est probant dans les RA. Il faut donc faire des tests en local.

  1. cd pix-site/
  2. Exécuter la commande : npm run dev:site:org
  3. Aller sur http://localhost:7001/ et cliquer sur België (Nederlands)
  4. Constater le positionnement d'un cookie locale avec la valeur nl-BE
  5. Arrêter la commande npm run dev:site:org
  6. cd pix-pro/
  7. Exécuter la commande : npm run dev:site:org
  8. Aller sur http://localhost:7001/ (et pas sur http://localhost:7001/nl-be)
  9. Constater que l'utilisateur est redirigé vers http://localhost:7001/en/ mais que son cookie locale a toujours la valeur nl-BE

Refaire la même opération avec Belgique (Français) (fr-BE) et constater que l'utilisateur est redirigé vers http://localhost:7001/en/

@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 sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-site-review-pr718/environment

@lego-technix lego-technix force-pushed the pix-14143-fix-locale-cookie-unavailability-with-best-match branch from 71adf42 to 8d5ba84 Compare October 2, 2024 13:50
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Review OK.

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