Skip to content

Conversation

@Polypheides
Copy link
Contributor

@Polypheides Polypheides commented Mar 21, 2025

Here is the requested Ukrainian addition together with a Missing? Arabic entry in Patch104pZH/ModBundleOptionalLanguageItems.json

Ukrainian will not build yet unless added to the gametextcompiler this will be fixed later

My changes for gametextcompiler
gametextcompiler code Ukrainian UA
gametextcompiler code Ukrainian UK
gametextcompiler code Swedish SV and Ukrainian UK
gametextcompiler code Swedish SV, Ukrainian UK, Azerbaijani AZ Turkish TR
gametextcompiler code Now adds almost all the languages in the world

I have also added _comments for each community-made language

Reason for the Swedish entries seen in the files changed is because Ukrainian entries comes after Swedish alphabetically in the JSON and that the Swedish PR #2715 is not merged yet.

@Polypheides Polypheides changed the title Adding _comments, Ukrainian entries in various json and a missing Arabic entry Fix: Adding missing Arabic, Ukrainian and _comment in various JSON Mar 25, 2025
@Polypheides Polypheides force-pushed the Adding-UK-entries-in-json branch from 047106c to fc89129 Compare March 25, 2025 11:28
@DevGeniusCode
Copy link
Collaborator

@Polypheides is this ready to review?

@Polypheides
Copy link
Contributor Author

Polypheides commented Mar 25, 2025

is this ready to review?

All my pull request are always ready for scrutiny.
So yes you may review it but we still miss critical fixes for the changes to work 'gametextcompiler'.

and #2715 comes first in the commit chain that needs reviewing and approval adding Swedish.

my latest iteration of gametextcompiler works with both SV and UK Polypheides/Thyme@95138dc but contains comments in code that we might not want in the 'gametextcompiler' main branch.

@Polypheides
Copy link
Contributor Author

Polypheides commented Mar 30, 2025

"I have a commit for this pull request waiting to be pushed, removing _comment. xezon argued against adding comments to these files (#2715). We'll see what he decides."

I think this comment is not necessary here. There already is a wiki page somewhere which could list the original game languages, which then implies which ones are community made.

Here is the wiki page
https://github.com/TheSuperHackers/GeneralsGamePatch/wiki/localization_contribution

@Polypheides Polypheides changed the title Fix: Adding missing Arabic, Ukrainian and _comment in various JSON Fix: Adding missing Arabic, Ukrainian in various JSON Mar 30, 2025
@Polypheides Polypheides changed the title Fix: Adding missing Arabic, Ukrainian in various JSON Fix: Adding missing Arabic and Ukrainian in various JSON Mar 30, 2025
@Polypheides Polypheides force-pushed the Adding-UK-entries-in-json branch from 71dc699 to ade2602 Compare June 10, 2025 08:45
Update ModBundleOptionalLanguageItems.json

adding swedish entries in .json
Update ModBundleFullPacks.json

Adding _comment and a "missing Arabic option?"

Squashed commit of the following:

commit d9b0c10
Author: Polypheides <[email protected]>
Date:   Fri Mar 21 20:08:03 2025 +0100

    fix

commit b8c2ddd
Author: Polypheides <[email protected]>
Date:   Fri Mar 21 20:01:54 2025 +0100

    Update ModBundleFullPacks.json

commit 6e6c383
Author: Polypheides <[email protected]>
Date:   Fri Mar 21 19:56:25 2025 +0100

    Update ModBundleCoreLanguageItems.json

update alphabetic order

Update ModBundleCoreLanguageItems.json

_comment": "This Swedish localization is community-made"

An example comment in JSON

Update ModBundleCoreLanguageItems.json
@Polypheides Polypheides force-pushed the Adding-UK-entries-in-json branch from ade2602 to bc9e441 Compare June 11, 2025 20:24
@Polypheides Polypheides changed the title Fix: Adding missing Arabic and Ukrainian in various JSON Fix Adding missing Arabic and Ukrainian in various JSON Jun 13, 2025
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.

Ukrainian localization is not available in Mod Builder

2 participants