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 simplified Chinese language translations #3784

Merged
merged 1 commit into from Oct 1, 2020
Merged

Add simplified Chinese language translations #3784

merged 1 commit into from Oct 1, 2020

Conversation

EHfive
Copy link
Contributor

@EHfive EHfive commented Sep 29, 2020

Generally, we use language code 'zh-cn' to represent simplified Chinese, 'zh-tw' in the case of traditional Chinese.
Due to the limitation of https://github.com/usrz/bootstrap-languages used by mailcow UI, there is only code 'zh' avaliable for Chinese. That's why I use 'zh' instead.

Maybe we should switch from bootstrap-languages in future so that we can add support for traditional Chinese.

@MAGICCC
Copy link
Member

MAGICCC commented Sep 29, 2020

Wew nice!
You can also add some strings here:
https://github.com/mailcow/mailcow-dockerized/blob/master/data/conf/dovecot/dovecot.conf#L90

@EHfive
Copy link
Contributor Author

EHfive commented Sep 29, 2020

Oh, I missed that.

@EHfive
Copy link
Contributor Author

EHfive commented Sep 29, 2020

Wew nice!
You can also add some strings here:
https://github.com/mailcow/mailcow-dockerized/blob/master/data/conf/dovecot/dovecot.conf#L90

Added.

@EHfive
Copy link
Contributor Author

EHfive commented Sep 30, 2020

I think it's ready to be reviewed.

@MAGICCC
Copy link
Member

MAGICCC commented Sep 30, 2020

LGTM @andryyy

@andryyy
Copy link
Contributor

andryyy commented Sep 30, 2020

Very cool. Thank you!
Did you check Dovecot is ok with the chinese chars in the config file?

@MAGICCC
Copy link
Member

MAGICCC commented Sep 30, 2020

It's starting and does not crash for me

@EHfive
Copy link
Contributor Author

EHfive commented Sep 30, 2020

Very cool. Thank you!
Did you check Dovecot is ok with the chinese chars in the config file?

I don't think it would be a problem because dovecot.conf stored as 'UTF-8'.
And, yes, I did't observed any errors.

@andryyy
Copy link
Contributor

andryyy commented Sep 30, 2020

Let me check that too and then merge it. :)

@andryyy
Copy link
Contributor

andryyy commented Oct 1, 2020

Thank you!!!

@andryyy andryyy merged commit b5ee399 into mailcow:master Oct 1, 2020
@EHfive EHfive deleted the lang-zh branch October 1, 2020 23:52
Hexaris pushed a commit to Hexaris/mailcow-dockerized that referenced this pull request Oct 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants