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

support i18n #782

Closed
wants to merge 4 commits into from
Closed

Conversation

skipness
Copy link
Contributor

@skipness skipness commented May 24, 2023

resolve #734
This draft PR add basic support for i18n and using splash screen and the first step of "create a new account" as demo, you can't change locale manually and only support english now.

To add other language you will need to:

  1. create a folder in locale naming with language tag (e.g. en-US)
  2. copy those 3 json files in en folder to the folder above
  3. do your translation work
  4. add entry in resources.ts

Not fully functional, still need polishing.

@skipness skipness mentioned this pull request May 24, 2023
@skipness skipness marked this pull request as draft May 25, 2023 03:32
@pfrazee pfrazee added the x:on-the-roadmap We're planning to do this but it may be a bit label May 30, 2023
@pfrazee
Copy link
Collaborator

pfrazee commented May 30, 2023

I like where this is headed, but localization is a pretty huge undertaking and we're just super not ready for this right now. We're barely keeping up with the existing task load.

@pfrazee pfrazee added x:putting-this-off To be honest, we just can't deal with this right now. and removed x:on-the-roadmap We're planning to do this but it may be a bit labels May 31, 2023
@skipness
Copy link
Contributor Author

skipness commented May 31, 2023

I like where this is headed, but localization is a pretty huge undertaking and we're just super not ready for this right now. We're barely keeping up with the existing task load.

@pfrazee I understand it's huge, I will just leave this PR here in case you want to start working on it. Keep up on other tasks😁

@ansh
Copy link
Contributor

ansh commented Nov 16, 2023

Closing this in favor of #1822

@ansh ansh closed this Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:putting-this-off To be honest, we just can't deal with this right now.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support i18n
3 participants