[FEATURE] Gérer la locale fr-fr (fr-FR) sur pix.org sous forme de préfixe (PIX-13385) #683
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lego-technix
added
🚧 Development in progress
cross-team
|
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-pr683/environment |
lego-technix
force-pushed
the
pix-13385-handle-fr-fr-locale-in-pix-org
branch
2 times, most recently
from
45d7303
to
e4057f9
Compare
|
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-pr683/environment |
clemlatz
force-pushed
the
pix-13385-handle-fr-fr-locale-in-pix-org
branch
2 times, most recently
from
ab9c5a0
to
d493974
Compare
lego-technix
force-pushed
the
pix-13385-handle-fr-fr-locale-in-pix-org
branch
2 times, most recently
from
b8e6632
to
b2a80e5
Compare
clemlatz
force-pushed
the
pix-13385-handle-fr-fr-locale-in-pix-org
branch
from
b2a80e5
to
b28cfb1
Compare
lego-technix
force-pushed
the
pix-13385-handle-fr-fr-locale-in-pix-org
branch
13 times, most recently
from
d28e283
to
7e02147
Compare
lego-technix
changed the title
lego-technix
force-pushed
the
pix-13385-handle-fr-fr-locale-in-pix-org
branch
6 times, most recently
from
b50a292
to
c24e4e5
Compare
lego-technix
force-pushed
the
pix-13385-handle-fr-fr-locale-in-pix-org
branch
3 times, most recently
from
c855e0f
to
d9629b8
Compare
Co-authored-by: Clément Latzarus <[email protected]> Co-authored-by: Bérengere Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
Co-authored-by: Clément Latzarus <[email protected]> Co-authored-by: Bérengère Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
Co-authored-by: Clément Latzarus <[email protected]> Co-authored-by: Bérengère Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
Co-authored-by: Bérengère Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
Co-authored-by: Clément Latzarus <[email protected]> Co-authored-by: Bérengère Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
Co-authored-by: Clément Latzarus <[email protected]> Co-authored-by: Bérengère Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
Co-authored-by: Bérengère Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
Co-authored-by: Clément Latzarus <[email protected]> Co-authored-by: Bérengère Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
Co-authored-by: Clément Latzarus <[email protected]> Co-authored-by: Bérengère Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
Co-authored-by: Clément Latzarus <[email protected]> Co-authored-by: Bérengère Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
Co-authored-by: Clément Latzarus <[email protected]> Co-authored-by: Bérengère Claudeau <[email protected]> Co-authored-by: LEGO Technix <[email protected]>
lego-technix
force-pushed
the
pix-13385-handle-fr-fr-locale-in-pix-org
branch
from
d9629b8
to
56d46c9
Compare
lego-technix
force-pushed
the
pix-13385-handle-fr-fr-locale-in-pix-org
branch
from
56d46c9
to
d6ddf6e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🦄 Problème
Actuellement gérer les 2 sites distincts https://pix.org/ et https://pix.fr/, mais qui sont extrêmement proches, apporte beaucoup de complexité et de problèmes, alors qu'on pourrait fournir les mêmes services et la même qualité de service avec un seul site web.
Idem pour https://pro.pix.org/ et https://pro.pix.fr/
🤖 Proposition
On propose, sous forme de ce prototype, de ne plus gérer le contenu de https://pix.fr/ comme une exception, mais plutôt de le gérer dans https://pix.org/ comme un cas générique suivant la locale
fr-fr.🌈 Remarques
🧹 Si ce prototype est validé (cela ne sert à rien de le faire avant), supprimer les derniers bouts de code et de build qui assurent la gestion spécifique de https//pix.fr/
ℹ️ Dans cette PR c'est la valeur de locale
fr-frqui est utilisée au lieu de la valeur de locale sous sa forme canoniquefr-FR. Évidemment il faudrait utiliser la forme canoniquefr-FRmais cela ferait une PR plus grosse et plus difficile à appréhender alors que cette modification pourra être réalisée juste après.ℹ️ Et on cite enfin toujours en référence l'ADR https://github.com/1024pix/pix/blob/dev/docs/adr/0040-locales-languages.md
💯 Pour tester
Redirections
Chaque URL en
pix.fr(dit autrement, chaque adresse enpix.fr) doit être redirigée vers la ressource correspondante surpix.org, pour que les visiteurs ne voient aucune différence et pour maintenir le bon référencement (SEO).Pour tester cela :
Locale
fr-frexpliciteLa locale
fr-frdoit être gérée explicitement comme toutes les autres locales (fr-be,fr,en, etc.)Pour tester cela :
Franceon reste bien sur le même si web https://site-pr683.review.pix.org/ (on n'est pas redirigé vers un site en.fr) et que les URL commencent parhttps://site-pr683.review.pix.org/fr-fr/Franceon reste bien sur le même si web https://pro-pr683.review.pix.org/ (on n'est pas redirigé vers un site en.fr) et que les URL commencent parhttps://pro-pr683.review.pix.org/fr-fr/Bon enchainement vers Pix App sur le bon domaine et avec la bonne langue
Les sites vitrines doivent pointer vers Pix App suivant un bon URL (autrement dit, suivant une bonne adresse) qui va permettre à Pix App de savoir quelle est la langue de l'utilisateur et quelles épreuves du référentiel sélectionner.
Pour tester cela :
Connexionpointe vers https://app.pix.fr/connexionConnexionpointe vers https://app.pix.org/connexionConnexionpointe vers https://app.pix.org/connexion?lang=en