-
Notifications
You must be signed in to change notification settings - Fork 43
adding localisation support - merging into 23.0 release branch #254
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
Conversation
6407e34
to
d2a3476
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much @naren2605. This is a big chunk of work you have done in this PR.
Please address some of the minor misses I have commented on inline. Thanks.
thanks for reviewing @Achal1607 & @sid-srini will take a look and address these comments |
bb5645f
to
4f29479
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @naren2605. LGTM.
4f29479
to
ac4a12c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks @naren2605
localisation enhancement -#253
for simplified chinese and japanese