Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add more localization support, this time with lower_with_under #2214

Draft
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

TwistedGiraffe
Copy link
Contributor

@TwistedGiraffe TwistedGiraffe commented Sep 7, 2023

This PR currently adds localization support for:

  • RP-1 contract agencies (e.g., FAI)
  • Craft files (e.g., Jet Trainer)
  • KCT tags
  • Program descriptions
  • Leader names, biography, and effects.
  • Milestone newspaper articles
  • Tech tree
  • Upgrades (except ProcAvionics)

Future additions include but are not limited to:

  • ProcAvionics (currently using hard-coded upgrade description)
  • Science parts and experiments
  • Contracts

I will also change the #rp0's into #rp1's in the source code. However, a complete localization of RP-1 GUI is probably best suited in another PR.

As discussed in the last PR (#2154), this time the tags follow lower_with_under and are all in a single file.

@TwistedGiraffe TwistedGiraffe marked this pull request as draft September 7, 2023 22:10
@NathanKell
Copy link
Member

Yay it's back! :)

@NathanKell
Copy link
Member

What's the status of this? We really need to get this merged so that it doesn't keep conflicting every time we make loc updates to master.

@NathanKell
Copy link
Member

@TwistedGiraffe poke?

@TwistedGiraffe
Copy link
Contributor Author

Oh sorry NK, didn't see the reply earlier.
I stopped working on this PR when the Big Balance became a thing, thinking the refactor would jumble up localization strings in ddls. While BB is merged now, I have just been assigned a new research project, so I will be very busy at least until this Thursday. Then I should be able to make it to a mergeable state by 10/23.
Given all the changes in Big Balance, I will likely close this and open a new PR (again).

@NathanKell
Copy link
Member

That works! Just let's merge it immediately once you're done with that. :D

@NathanKell
Copy link
Member

@TwistedGiraffe any progress here?

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.

2 participants