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

Update Malayalam translations #1813

Merged

Conversation

anoopmsivadas
Copy link
Contributor

@anoopmsivadas anoopmsivadas commented Oct 1, 2021

What does this PR do?

Updates Malayalam translations in ml.json.

Test Plan

Used the glossary of Microsoft and Mozilla for cross checking the words I used for translation. The tranlated strings are based on the English translations from en.json. And also have run the e2e testcases for Locale.

Related PRs and Issues

#1702

Have you read the Contributing Guidelines on issues?

yes

@christyjacob4
Copy link
Member

Please use the correct PR template. You are missing some items in there

Copy link
Member

@christyjacob4 christyjacob4 left a comment

Choose a reason for hiding this comment

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

Please address the comments

@anoopmsivadas
Copy link
Contributor Author

anoopmsivadas commented Oct 4, 2021

Please address the comments

Hey @christyjacob4 , I have fixed the issues mentioned. Please review at your convenience. Thanks

@christyjacob4
Copy link
Member

Awesome . Looks great 👍

@christyjacob4 christyjacob4 added the hacktoberfest-accepted Accepted for Hacktoberfest, will be merged later label Oct 4, 2021
@christyjacob4 christyjacob4 merged commit e300194 into appwrite:master Oct 5, 2021
@christyjacob4
Copy link
Member

THANK YOU so much! All changes merged 🥳
Please reach out to me on our Discord server if you would like to claim your Appwrite swag!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Accepted for Hacktoberfest, will be merged later
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants