[TECH] Utiliser les termes "locales" au lieu de "languages" (PIX-6772) #464
Conversation
yannbertrand
added
👀 Tech Review Needed
👀 Func Review Needed
team-evaluation
|
Une fois les applications déployées, elles seront accessibles via les liens suivants : Les variables d'environnement seront accessibles ici. |
| @@ -19,7 +19,7 @@ | |||
|
|
|||
| <script> | |||
| import { getAbsoluteUrlIfSwitchWebsite } from '~/services/get-absolute-url-if-switch-website' | |||
| import { language } from '~/config/language' | |||
| import { localization } from '~/config/localization' | |||
There was a problem hiding this comment.
On ne souhaite pas mettre un s ? 🤔
There was a problem hiding this comment.
Plutôt pour ne pas mettre de « s », car il me semble qu'il s'agit du paramétrage de la localization, et non des différentes locales. Mais ça ne serait pas très grave avec un « s » 🙂
There was a problem hiding this comment.
J'ai l'impression que location conviendrait mieux :)
There was a problem hiding this comment.
location en français c'est « lieu », « localité », là on est dans un contexte plus large Internationalization/localization i18n/L10n https://en.wikipedia.org/wiki/Internationalization_and_localization
yannbertrand
changed the title
yannbertrand
added
Func Review OK
Tech Review OK
🚀 Ready to Merge
and removed
👀 Tech Review Needed
👀 Func Review Needed
labels
🎄 Problème
On souhaite implémenter une partie des choix de 1024pix/pix#5341. Dans cet ADR on aborde l'utilisation de "locales" plutôt que de "langues". Le code était assez peu clair à ce niveau jusqu'à présent.
🎁 Proposition
Renommer :
langdevientlocaleCode(les classes CSS sont mises à jour également)/config/languagesdevient/config/localizationlangdevienttranslationset est déplacé dans/config/localizationlangdeviennentlocale.🌟 Remarques
L'ADR préconise le format
BCP 47(fr-BE), qui n'est pas celui choisi par notre CMS Prismic, on décide de ne pas changer notre manière de faire actuelle pour éviter tout risque. Leur API manipule d'ailleurs deslangplutôt que deslocale, il reste donc quelques occurrences dans le code lorsque l'on communique avec Prismic.🎅 Pour tester
Vérifier le bon fonctionnement des 4 sites.