Skip to content

Support des translation sur tout le plugin - #1382

Merged
iambibi merged 12 commits into
ServerOpenMC:masterfrom
iambibi:change/translations-component
Jul 29, 2026
Merged

Support des translation sur tout le plugin#1382
iambibi merged 12 commits into
ServerOpenMC:masterfrom
iambibi:change/translations-component

Conversation

@iambibi

@iambibi iambibi commented Jul 27, 2026

Copy link
Copy Markdown
Member

Petit résumé de la PR:

A ma découverte j'ai vu que meme si on mets les translation component sur les items, ça n'impactais pas de bug ou autre.
Alors je me lance dans une rapide conversion des DreamItem qui ont du lore custom ou meme les CustomItem en général

  • Suppression du TranslationManager.translationString, utilisez le PlainTextSerializer si vrm vous pouvez pas faire autrement (ex NotationDialog)

É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
  • Les Issues corrigée(s) en commun :

Decrivez vos changements

@iambibi iambibi added this to the 2.5.0-beta-2 milestone Jul 27, 2026
@iambibi iambibi self-assigned this Jul 27, 2026
@iambibi iambibi added the 🆙 Amélioration Une amélioration sur quelque chose label Jul 27, 2026
@iambibi

iambibi commented Jul 27, 2026

Copy link
Copy Markdown
Member Author

pas a merge mtn, y'a encore un check a faire si tt est bien supporté, je la merge dans la 2.5.0-beta-2

@iambibi

iambibi commented Jul 27, 2026

Copy link
Copy Markdown
Member Author

(du moins, dans max 40 min apres je pars)

@iambibi

iambibi commented Jul 27, 2026

Copy link
Copy Markdown
Member Author

plus le temps

@iambibi iambibi modified the milestones: 2.5.0-beta-2, 2.5.0-beta-3 Jul 27, 2026
@iambibi iambibi linked an issue Jul 28, 2026 that may be closed by this pull request
@iambibi
iambibi marked this pull request as draft July 28, 2026 09:17
@iambibi
iambibi marked this pull request as ready for review July 28, 2026 18:54
@iambibi

iambibi commented Jul 28, 2026

Copy link
Copy Markdown
Member Author

je pense que c'est bon, a review, surtout les trucs en anglais mais je pense avoir bien fait le tour

@iambibi
iambibi merged commit 606f36a into ServerOpenMC:master Jul 29, 2026
2 of 3 checks passed
@iambibi
iambibi deleted the change/translations-component branch July 29, 2026 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🆙 Amélioration Une amélioration sur quelque chose

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] argument message translation

2 participants