Skip to content

Update localisation for objects#242

Merged
ZehMatt merged 1 commit intoOpenRCT2:masterfrom
ZehMatt:merge-localisation
May 5, 2023
Merged

Update localisation for objects#242
ZehMatt merged 1 commit intoOpenRCT2:masterfrom
ZehMatt:merge-localisation

Conversation

@ZehMatt
Copy link
Contributor

@ZehMatt ZehMatt commented Apr 30, 2023

I tried my best to keep the diff minimal but the inconsistency is all over the place. I wrote some scripts using NodeJS to also help improve the other repo, not sure yet how the code is best shared among the two.

Diff is now quite simple

@ZehMatt ZehMatt force-pushed the merge-localisation branch from 3c1f81c to 7ce41c9 Compare May 4, 2023 19:28
@ZehMatt ZehMatt marked this pull request as ready for review May 4, 2023 19:29
Copy link
Member

@tupaschoal tupaschoal left a comment

Choose a reason for hiding this comment

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

I just find the removals of en-US weird

@duncanspumpkin
Copy link
Contributor

en-us has been removed if identical to en-gb.

@ZehMatt
Copy link
Contributor Author

ZehMatt commented May 5, 2023

I just find the removals of en-US weird

I did that to keep it consistent, sometimes it was present with identical translation to en-gb and sometimes it wasn't, I think due to the way we use en-gb as fallback we can omit them. I can also enforce the other way around and always have en-us there.

@ZehMatt ZehMatt merged commit e71cbcb into OpenRCT2:master May 5, 2023
@ZehMatt ZehMatt deleted the merge-localisation branch May 5, 2023 19:01
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.

3 participants