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 translation.json #1432

Merged
merged 1 commit into from
Jan 30, 2025
Merged

Update translation.json #1432

merged 1 commit into from
Jan 30, 2025

Conversation

7ROBE
Copy link
Contributor

@7ROBE 7ROBE commented Jan 30, 2025

When submitting this pull request, I confirm the following (please check the boxes):

  • I have read and understood the Contributor Guidelines.
  • I have checked that there are no duplicate pull requests related to this request.
  • I have considered, and confirm that this submission is valuable to others.
  • I accept that this submission may not be used and the pull request may be closed at the discretion of the maintainers.

Fill in the PR content:

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Updated Arabic translation file to improve readability by removing diacritical marks (tashkeel) and aligning with modern Arabic writing conventions.

  • Removed tashkeel marks from existing translations in /src/locales/ar/translation.json for better clarity
  • Added new translation strings for achievements, permissions, and subscription features
  • Improved natural language flow while maintaining accurate UI translations
  • Made text more consistent with contemporary Arabic writing standards

💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!

1 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

src/locales/ar/translation.json Show resolved Hide resolved
@zamitto zamitto merged commit 7c7f621 into hydralauncher:main Jan 30, 2025
4 checks passed
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.

2 participants