Skip to content

Add French translation, new game tags and credits - #211

Open
voltek-laruelle wants to merge 6 commits into
eklonofficial:mainfrom
voltek-laruelle:main
Open

voltek-laruelle wants to merge 6 commits into
eklonofficial:mainfrom
voltek-laruelle:main

Conversation

@voltek-laruelle

@voltek-laruelle voltek-laruelle commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor
  • Add French translation (fr.json)
  • Add game
  • Update credits

@voltek-laruelle

Copy link
Copy Markdown
Contributor Author

@eklonofficial désolé pour le silence de ces dernières semaines, j'avais fait une pause et j'ai arrêté les tags. Je vais repartir d'une base propre et refaire une PR.

@voltek-laruelle voltek-laruelle changed the title add fr Add French translation, new game tags and credits Sep 20, 2026
@eklonofficial

Copy link
Copy Markdown
Owner

Merci pour la traduction, elle est vraiment complète. J'ai tout vérifié en local sur main à jour :

  • 95 % des chaînes traduites, et tous les {placeholders} correspondent à l'anglais, aucune clé en trop. Le reste retombe sur l'anglais, c'est prévu pour.
  • OpenFront : j'ai vérifié que steam_app_3560670 est bien OpenFront sur Steam.
  • La suite de tests passe sur le résultat de la fusion.

Deux petites choses avant que je fusionne :

  1. La branche est en conflit, mais uniquement sur le bundle généré, parce que main a bougé depuis. Il suffit de fusionner main et de reconstruire :
git merge main
npm run build
git add vice/ui/scripts/app.js vice/ui/styles/app.css && git commit
  1. Tu peux retirer le changement de package-lock.json ? Il ne vient pas de la traduction, c'est juste une autre version de npm qui a réécrit le fichier (version du paquet et champs libc). git checkout main -- package-lock.json suffit.

Pas de souci pour le silence, et pour les tags, repartir d'une base propre me va très bien.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants