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 Arabic translations in ar.js #565

Merged
merged 6 commits into from
Jul 8, 2024
Merged

Update Arabic translations in ar.js #565

merged 6 commits into from
Jul 8, 2024

Conversation

medelm1
Copy link
Contributor

@medelm1 medelm1 commented Jul 8, 2024

This pull request updates the Arabic localization file (ar.js) to ensure it aligns with the latest English translations in en.js. The goal was to address discrepancies such as missing translations, outdated entries, and ensure cultural appropriateness and accurate meanings were maintained throughout.

Changes Made

  • Added missing translations where applicable.
  • Updated outdated or incorrect translations to match the current English version.
  • Ensured all translations are culturally appropriate and maintain the intended meaning of the English text.

Additional Notes

  • Transliteration or translator notes were used where necessary to maintain fidelity to the original text.
  • Verified each change against the source file (en.js) to guarantee consistency.

Related Files

  • src/gui/src/i18n/translations/ar.js: Updated Arabic translations.
  • src/gui/src/i18n/translations/en.js: Reference file for English translations.

This update enhances the Arabic localization of the project, ensuring a more consistent and accurate user experience for Arabic-speaking users.

…match English version in src/gui/src/i18n/translations/en.js
@CLAassistant
Copy link

CLAassistant commented Jul 8, 2024

CLA assistant check
All committers have signed the CLA.

@medelm1 medelm1 closed this Jul 8, 2024
@jelveh
Copy link
Contributor

jelveh commented Jul 8, 2024

Hi @medelm1, thank you very much for your contribution! Merged 🎉

@jelveh
Copy link
Contributor

jelveh commented Jul 8, 2024

oh just as I was about merge. Any reason you closed it?

@medelm1
Copy link
Contributor Author

medelm1 commented Jul 8, 2024

I accidentally closed this request; I will reopen it now.

@medelm1 medelm1 reopened this Jul 8, 2024
@jelveh jelveh merged commit 8b9d57d into HeyPuter:main Jul 8, 2024
4 checks passed
@jelveh
Copy link
Contributor

jelveh commented Jul 8, 2024

Awesome! Thanks again for your contribution.

@medelm1
Copy link
Contributor Author

medelm1 commented Jul 8, 2024

Thank you! I'm glad to contribute. Let me know if there's anything else I can help with!

@jelveh
Copy link
Contributor

jelveh commented Jul 8, 2024

We definitely appreciate any help. Whether it's code, documentation, or user support. All contributions are welcome 😇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants