Skip to content

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

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

naren2605
Copy link
Member

localisation enhancement -#253
for simplified chinese and japanese

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Sep 4, 2024
@Achal1607 Achal1607 requested review from lahodaj and sid-srini and removed request for lahodaj September 4, 2024 12:33
Copy link
Member

@Achal1607 Achal1607 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@sid-srini sid-srini left a 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.

@Achal1607 Achal1607 linked an issue Sep 4, 2024 that may be closed by this pull request
@naren2605
Copy link
Member Author

thanks for reviewing @Achal1607 & @sid-srini will take a look and address these comments

@naren2605 naren2605 force-pushed the localisation-23.0 branch 3 times, most recently from bb5645f to 4f29479 Compare September 6, 2024 10:40
@sid-srini sid-srini self-requested a review September 6, 2024 11:31
Copy link
Member

@sid-srini sid-srini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @naren2605. LGTM.

Copy link
Member

@Achal1607 Achal1607 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @naren2605

@Achal1607 Achal1607 merged commit b124094 into oracle:23.0.0 Sep 6, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add localisation support for simplified chinese and japanese
3 participants