Repository navigation
feat: module translations (spec 019 T090, T091) - #51
ValgulNecron wants to merge 3 commits into
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> build.sh adds one layer per locales/<tag>.json (en.json excluded) with media type application/vnd.gameplane.module.locale.v1+json. CI runs i18n-check --module on every module that has a locales/ folder; that step needs GameplanePanel/Gameplane#668 merged first. Signed-off-by: Claude <noreply@anthropic.com>
…Valheim Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Signed-off-by: Claude <noreply@anthropic.com>
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Generated by Claude Code |
The locale check needs GameplanePanel/Gameplane#668 on master; running it last keeps the build.sh and validate.py tests visible meanwhile. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Signed-off-by: Claude <noreply@anthropic.com>
Before: module bundles carry only the template text, so the dashboard can show module names, settings and actions only in English.
After: a module can ship
locales/<tag>.json.build.shpushes each file as its own OCI layer, CI checks every module that has locales, and Minecraft Java, Terraria and Valheim ship a generatedlocales/en.jsonthat translators start from.How:
build.shadds one layer perlocales/<tag>.jsonexcepten.json, with media typeapplication/vnd.gameplane.module.locale.v1+jsonand the file name as its title, so the operator finds it. The validate workflow runsi18n-check --modulefrom the main repo on every module with alocales/folder. Theen.jsonfiles were generated withgp-module i18n extract, not written by hand. A new root README explains how to add a language.Merge order: this needs GameplanePanel/Gameplane#668 merged into master first, because the CI step installs
i18n-checkfrom master. The rootmodulespointer is bumped only after this PR merges.🤖 Generated with Claude Code
Generated by Claude Code