Mindustry icon indicating copy to clipboard operation
Mindustry copied to clipboard

Update bundle_fr.properties

Open AyuKo-o opened this issue 2 years ago • 7 comments

French Translation is done and ready to be used

AyuKo-o avatar Nov 11 '22 22:11 AyuKo-o

:pray: Thanks mate

phinner avatar Nov 11 '22 22:11 phinner

I translated what was left to translate, I PR it on your branch I let you update!

JojoFR1 avatar Nov 12 '22 00:11 JojoFR1

Added a french translation for those following descriptions:

  • The core
  • All the defense
  • All the production
  • All the drills
  • All the walls
  • All the duct/conduit
  • All energy related stuff
  • All the units stuff
  • and the other stuff

TLDR: What was missing to translate.

JojoFR1 avatar Nov 12 '22 00:11 JojoFR1

Aussi, pour pas te casser la tête a implémenter les changements toi même comme tout à l'heure, clique sur ce bouton. Cela va automatiquement merge la suggestion. exemple

phinner avatar Nov 12 '22 02:11 phinner

La traduction est bien dans son ensemble, mais parfois, on a l'impression que vous avez juste utilisé un outil de traduction sans bien vérifier car :

  • Il y a de la ponctuation manquante.
  • Quelques traduction incomplètes.
  • Des tournures de phrase bizarres.
  • etc...

C'est bon on a régler ce que t'a suggérer de modifier, normalement tout est traduit !

JojoFR1 avatar Nov 12 '22 11:11 JojoFR1

Excellent, mais il manque 2 suggestions :

  • https://github.com/Anuken/Mindustry/pull/7883/files#r1020631761

  • https://github.com/Anuken/Mindustry/pull/7883/files#r1020632388

phinner avatar Nov 12 '22 12:11 phinner

Excellent, mais il manque 2 suggestions :

  • https://github.com/Anuken/Mindustry/pull/7883/files#r1020631761

  • https://github.com/Anuken/Mindustry/pull/7883/files#r1020632388

Ah oui j'avais pas vu, my bad

JojoFR1 avatar Nov 12 '22 16:11 JojoFR1

Line 159, add the new translation (la traduction ça doit être ça, regardez le dernier "Bundle update")

mod.requiresversion = Requiert la version: [red]{0}

JojoFR1 avatar Nov 14 '22 09:11 JojoFR1

Line 159, add the new translation (la traduction ça doit être ça, regardez le dernier "Bundle update")

mod.requiresversion = Requiert la version: [red]{0}

Il faudrait que ayuko synchronize son fork pour avoir les changements.

phinner avatar Nov 14 '22 09:11 phinner

image Clique-là

phinner avatar Nov 14 '22 09:11 phinner

https://github.com/Anuken/Mindustry/pull/7883/commits/469335e99753010de7f9a88b7e93c2020ade6605

Bruh, tu l'as rajouté manuellement, tu as créé un conflit...

phinner avatar Nov 14 '22 09:11 phinner

AAAAAAAAAAAAAAAAAAAAAAH, QU'AS-TU FAIT !!! Tu aurais pu reverser ton commit, synchronizer le fork et enfin ajouter la traduction...

phinner avatar Nov 14 '22 09:11 phinner

AAAAAAAAAAAAAAAAAAAAAAH, QU'AS-TU FAIT !!! Tu aurais pu reverser ton commit, synchronizer le fork et enfin ajouter la traduction...

T'as répondu qu'il fallait résoudre conflits après que je devais sync Mais sinon il n'y a pas un moyen de re-ouvrir le pull request? sinon on peut en faire un autre je suppose

AyuKo-o avatar Nov 14 '22 09:11 AyuKo-o

Il y aurait jamais eu de conflit si tu n'avais pas ajouter la ligne manuellement. Je vais réouvrir ce PR moi même...

phinner avatar Nov 14 '22 09:11 phinner

Il y aurait jamais eu de conflit si tu n'avais pas ajouter la ligne manuellement. Je vais réouvrir ce PR moi même...

J'avais aucun moyen de merge ou autre... S'il ne fallait pas le faire dans ce cas il ne fallait pas faire la demande mais bon pas grave

AyuKo-o avatar Nov 14 '22 09:11 AyuKo-o

👌

phinner avatar Nov 14 '22 10:11 phinner