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 to add arabic to langs.json #45

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

rex07
Copy link

@rex07 rex07 commented May 23, 2024

Add Arabic

Add Arabic
@MSOB7YY
Copy link
Member

MSOB7YY commented May 23, 2024

the translation on weblate has many defects, feels like machine generated. there are already 2 earlier pr with the same issues

@rex07
Copy link
Author

rex07 commented May 23, 2024

If you know the issues on weblate you can point it out so I can fix it.

@rex07
Copy link
Author

rex07 commented May 24, 2024

i have gone through all the strings and I fix the one's that I found problematic.

@rex07 rex07 changed the title Update langs.json Update to add arabic to langs.json Jun 19, 2024
@rex07
Copy link
Author

rex07 commented Jun 19, 2024

@MSOB7YY can you merge the pull request?
If not why I told you I fix the problems that I found and you chosen to ignore this.

@MSOB7YY
Copy link
Member

MSOB7YY commented Jun 20, 2024

sorry i didn't ignore, i just dont have time to review the full translation, i will surely by the next update so dw

@rex07
Copy link
Author

rex07 commented Nov 24, 2024

@MSOB7YY up

@MSOB7YY
Copy link
Member

MSOB7YY commented Nov 26, 2024

@rex07 every time i check it i get cringed out tbh
feels so machiney

@rex07
Copy link
Author

rex07 commented Nov 27, 2024

@MSOB7YY I don't want to be rude, but if you don't explain to me what's wrong with my translation and evaluate my translation based on your feelings, I don't think Arabic has a place in your project. That's okay, I won't force you to do that.

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