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

Chinese language support #19

Closed
pirishd opened this issue Sep 13, 2017 · 4 comments
Closed

Chinese language support #19

pirishd opened this issue Sep 13, 2017 · 4 comments

Comments

@pirishd
Copy link
Contributor

pirishd commented Sep 13, 2017

There does not seem to be Chinese language support in the library contrary to what the documentation says: https://github.com/eggheadgames/Siren#localization.

Am I right? Is the support of this language planned?

@mikemee
Copy link
Collaborator

mikemee commented Sep 13, 2017

Hi @pirishd , we should have all the languages that the iOS Siren supports, but perhaps we missed some, or they were added between when we created the code and copied and modified the documentation.

If you can tell me which ones from here: https://github.com/ArtSabintsev/Siren/tree/master/Sources/Siren.bundle that you need that are missing from our list here https://github.com/eggheadgames/Siren/tree/develop/library/src/main/res, we can add them. If you know the equivalent Android language code(s) too, that will speed things up.

Or, of course, feel free to give us a PR 👍

Thanks!

@pirishd
Copy link
Contributor Author

pirishd commented Sep 13, 2017

Hi @mikemee, I created a PR to add the support of the Chinese language (based on the translations found in https://github.com/ArtSabintsev/Siren/blob/master/Sources/Siren.bundle/zh-Hans.lproj/SirenLocalizable.strings).

Would you please review that?

@mikemee
Copy link
Collaborator

mikemee commented Sep 13, 2017

@pirishd Thanks! Let's continue in the PR, where I have a question.

@mikemee
Copy link
Collaborator

mikemee commented Sep 14, 2017

@pirishd Many thanks for the issue and PR. This was fixed in release 1.4.8.

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

No branches or pull requests

2 participants