Skip to content

Complete Arabic translation #51899

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

shahar1
Copy link
Contributor

@shahar1 shahar1 commented Jun 18, 2025


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added area:translations area:UI Related to UI/UX. For Frontend Developers. translation:ar labels Jun 18, 2025
@shahar1 shahar1 changed the title Complete common.json in Arabic [WIP] Complete common.json in Arabic Jun 18, 2025
@shahar1 shahar1 force-pushed the i18n-arabic-common branch 3 times, most recently from b7d5480 to b032369 Compare June 18, 2025 18:29
@shahar1 shahar1 changed the title [WIP] Complete common.json in Arabic Complete common.json in Arabic Jun 18, 2025
@shahar1 shahar1 force-pushed the i18n-arabic-common branch from b032369 to e651c8e Compare June 19, 2025 15:52
@shahar1 shahar1 marked this pull request as ready for review June 19, 2025 15:52
@shahar1 shahar1 requested a review from hussein-awala as a code owner June 19, 2025 15:52
@shahar1
Copy link
Contributor Author

shahar1 commented Jun 19, 2025

cc: @ahmadtfarhan

@shahar1 shahar1 force-pushed the i18n-arabic-common branch 3 times, most recently from 9880fc4 to bfaad10 Compare June 20, 2025 06:53
@shahar1 shahar1 force-pushed the i18n-arabic-common branch from bfaad10 to be93b20 Compare June 29, 2025 18:10
@shahar1 shahar1 changed the title Complete common.json in Arabic Complete Arabic translation Jun 29, 2025
"connection_one": "موَّصل",
"connection_other": "موَّصلات",
"connection_two": "موَّصلان",
Copy link
Contributor

Choose a reason for hiding this comment

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

is this just for arabic language?

Copy link
Member

Choose a reason for hiding this comment

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

Yeah. Arabic has 6(!) different plurals. It's a winner ove Polish as we have just 4 :).

You can play with it here - to see how they map to numbers:

https://jsfiddle.net/6bpxsgd4

Copy link
Member

Choose a reason for hiding this comment

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

Actually arabic rules seem to be way more logical than Polish > 0ne, zero, two, few, many, other = a lot (100+)

Copy link
Member

@potiuk potiuk Jun 29, 2025

Choose a reason for hiding this comment

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

For example In Polish:

1000 -> many (we have no zero - zero is the same as many (!))
1001 -> many (not one)
1002 -> still many
1003 -> still many
1004 -> still many
1005 -> few (!!!!!)

@@ -75,8 +90,12 @@
"columns": {
"source": "المصدر"
},
"importError_few": "خطاء في استيراد المكوّن الإضافي",
"importError_many": "أخطاء في استيراد المكوّن الإضافي",
Copy link
Contributor

Choose a reason for hiding this comment

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

what is the difference between many and other?

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

I do not pretend I can read it though :)

Copy link
Contributor Author

@shahar1 shahar1 Jun 29, 2025

Choose a reason for hiding this comment

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

I didn't find any essential difference, so I just made them duplicates for now

@shahar1 shahar1 force-pushed the i18n-arabic-common branch from be93b20 to ba1d45a Compare June 29, 2025 19:04
@shahar1 shahar1 requested a review from ahmadtfarhan June 29, 2025 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:translations area:UI Related to UI/UX. For Frontend Developers. translation:ar
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants