Skip to content

Rewrite du TranslationManager + Impl adminshop, commands, api#1161

Open
iambibi wants to merge 25 commits intoServerOpenMC:v2.5from
iambibi:chore/messages
Open

Rewrite du TranslationManager + Impl adminshop, commands, api#1161
iambibi wants to merge 25 commits intoServerOpenMC:v2.5from
iambibi:chore/messages

Conversation

@iambibi
Copy link
Member

@iambibi iambibi commented Feb 24, 2026

Petit résumé de la PR:

Rewrite de la base des translations, tout les messages seront dans un fichier (ressources/translations/lang_fr_FR.properties par ex).

A partir que cette PR passe, toute nouvelle PR devra mettre ses messages dans le fichier lang

Choses non traduites :

  • Phrase dans les YAML (ex catégorie de l'admin shop, messages boss bar, et j'en oublie peut etre)

Étape nécessaire afin que la PR soit fini (si PR en draft)

  • Suivre le Code de Conduite
  • Enlever tous les imports non utilisés
  • Bien documenter la feature
  • Fournir un profileur (si besoin/demandé par un admin)
  • Avoir une milestone associée à la PR
  • Valider tout les checks
  • Tester et valider la feature/changement

Decrivez vos changements

  • Plus de ChatColor d'autorisé
  • traduction de tout les messages dans des autres langues possible
  • Tout en MiniMessage

@iambibi iambibi added this to the 2.5.0 milestone Feb 24, 2026
@iambibi iambibi self-assigned this Feb 24, 2026
@iambibi iambibi added the 📦 Feature Ajout d'une fonctionnalité label Feb 24, 2026
@iambibi iambibi marked this pull request as draft February 24, 2026 15:40
@iambibi
Copy link
Member Author

iambibi commented Mar 3, 2026

apres plusieures batailles avec les components et les lois de la translation des messages, je pense avoir trouvé une bonne solution afin de combiner la lisibilité et le pratique

@iambibi
Copy link
Member Author

iambibi commented Mar 7, 2026

je suis deja a 164 fichiers changé, m'en voulez pas, je viens a peine de finir une feature mdrr.

PS: y'a des endroits ou j'ai faillit mourrir sur le code de l'admin shop

@iambibi
Copy link
Member Author

iambibi commented Mar 7, 2026

prochaine feature : Les Villes 🤯 💥

@iambibi
Copy link
Member Author

iambibi commented Mar 7, 2026

vous voulez que je fasse une PR pour l'implémentation du systeme de translation + traduction adminshop et api, et commandes et une PR par feature traduite?
OU tout dans une ?

(si premiere choisie, alors faudra vite la passer, je peux finir la world gen de la dimension des reves mais ça durera pas 10 ans)

@iambibi iambibi changed the title Rewrite du TranslationManager Rewrite du TranslationManager + Impl adminshop, commands, api Mar 7, 2026
@iambibi iambibi marked this pull request as ready for review March 7, 2026 11:56
@iambibi iambibi linked an issue Mar 7, 2026 that may be closed by this pull request
@iambibi
Copy link
Member Author

iambibi commented Mar 7, 2026

a review, assez rapidement (~1-2 semaines)

@iambibi iambibi requested review from AxenoDev, PuppyTransGirl and ltuffery and removed request for AxenoDev March 9, 2026 13:56
@iambibi
Copy link
Member Author

iambibi commented Mar 9, 2026

up

@iambibi
Copy link
Member Author

iambibi commented Mar 11, 2026

vous voulez que je fasse une PR pour l'implémentation du systeme de translation + traduction adminshop et api, et commandes et une PR par feature traduite?
OU tout dans une ?

(si premiere choisie, alors faudra vite la passer, je peux finir la world gen de la dimension des reves mais ça durera pas 10 ans)

Juste ici

@PuppyTransGirl
Copy link
Member

Sinon juste player.sendTitle(Title.title(...)) au lieu de title part

@PuppyTransGirl
Copy link
Member

c'est pour l'optimisation 🤓☝️

@iambibi
Copy link
Member Author

iambibi commented Mar 11, 2026

Sinon juste player.sendTitle(Title.title(...)) au lieu de title part

'sendTitle(com.destroystokyo.paper.Title)' is deprecated

@iambibi
Copy link
Member Author

iambibi commented Mar 11, 2026

preuve que ça optimise rien xD

@iambibi
Copy link
Member Author

iambibi commented Mar 11, 2026

fin si on est dans l'objectif la refaire completer un serveur de zéro avec nos features nativements compatible bedrock ect ect

@PuppyTransGirl
Copy link
Member

Sinon juste player.sendTitle(Title.title(...)) au lieu de title part

'sendTitle(com.destroystokyo.paper.Title)' is deprecated

https://jd.advntr.dev/api/4.26.1/net/kyori/adventure/audience/Audience.html#showTitle(net.kyori.adventure.title.Title)

@PuppyTransGirl
Copy link
Member

C'est celui de Adventure et pas Paper qu'il faut utiliser

@PuppyTransGirl
Copy link
Member

ah oui je suis débile c'est pas sendTitle c'est showTitle j'oublie à chaque fois

@iambibi
Copy link
Member Author

iambibi commented Mar 11, 2026

oui oui puis je suis un peu bete au lieu d'ecrire juste title, je mettais send donc moi je voyais que la méthode la et non pas show

@iambibi
Copy link
Member Author

iambibi commented Mar 14, 2026

PR peut etre merge dans v2.5 :
=> aucun conflit avec la PR de gab (c'est sur la branche v2.5, je ferais les conflits lorsque j'updaterais la branche v2.5

Copy link
Member

@PuppyTransGirl PuppyTransGirl left a comment

Choose a reason for hiding this comment

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

:3

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

Labels

📦 Feature Ajout d'une fonctionnalité

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Les Translations

3 participants