-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 Chinese localization #247
Conversation
Pull Request Test Coverage Report for Build 1977574982
💛 - Coveralls |
Structurally this looks fine, although I can't read Chinese. |
dc90842
to
2fb038b
Compare
Addresses #134 (comment)
2fb038b
to
37c6201
Compare
Hi @shamoon I built it myself and tested the simplified chinese translation, it looks great! But I found some untranslated strings that are not on crowdin either. |
Thanks for verifying @FaintGhost (and for translating of course)! Hmm that’s the Django admin, I’ll take a look |
@paperless-ngx/backend do any of you Django folks know why #247 (comment) ? I checked with other languages and they seem to translate those strings just fine but none of the languages include those strings in the .po file, I think the translations are automatically generated by Django, I just dont know why not for Chinese Simplified. |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns. |
Addresses #134 (comment)
It looks like there have been some changes to Chinese Simplified locale string and other things so would be good if other(s) could test to make sure this is OK. Obviously I tested (screenshot) this, fyi I created a config like: