python-holidays icon indicating copy to clipboard operation
python-holidays copied to clipboard
verified publisher icon vacanza

Add Arabic language localisation

Open abh31000 opened this issue 2 years ago • 8 comments

Hi. I saw that your library doesn't support Arabic localisation. As a native speaker, I'd like to contribute to add support for it. Thanks.

abh31000 avatar May 14 '23 17:05 abh31000

Hi @abh31000,

it's great to hear that you're willing to add Arabic language support for PH!

The l10n process is not hard but not ideal yet. A (non language specific) help is available from other contributors. I suggest starting here. And here you can find several PRs you could use as an example.

Let me know in case you have l10n related questions.

Thanks again, we're very lucky to have a native Arabic speaker in our l10n team!

arkid15r avatar May 15 '23 15:05 arkid15r

Hi @abh31000,

it's great to hear that you're willing to add Arabic language support for PH!

The l10n process is not hard but not ideal yet. A (non language specific) help is available from other contributors. I suggest starting here. And here you can find several PRs you could use as an example.

Let me know in case you have l10n related questions.

Thanks again, we're very lucky to have a native Arabic speaker in our l10n team!

I'll start working on it right away! If I encounter any difficulties or get stuck on something, I'll make sure to get back to you!

abh31000 avatar May 15 '23 16:05 abh31000

Hi @arkid15r, There are some updates that should be made to the countries that need Arabic localization. Should I include them in one PR or work on them on a separate one? Thank you!

abh31000 avatar May 16 '23 17:05 abh31000

Hey @abh31000,

it's okay to combine multi-country changes if they are related. So a single PR would be fine.

arkid15r avatar May 16 '23 17:05 arkid15r

Hi @arkid15r,

This is actually my first l10n work. I implented the arabic localization for some countries. And they all work as expected.

Since all Islamic holidays are estimated. Please if you can, tell me how do I add localization for the estimated_label attributes in IslamicHolidays class.

Thank you.

abh31000 avatar May 17 '23 21:05 abh31000

Hi @abh31000, that's a totally valid point which requires some attention. I expect the issue to be addressed in https://github.com/dr-prodigy/python-holidays/pull/1140 end of the week -- early next week. I hope this won't block you from submitting your PR

arkid15r avatar May 18 '23 03:05 arkid15r

Hey @abh31000, how is it going with your first l10n attempt? Do you need help with anything (just want to make sure you aren't stuck with some technical difficulty)?

arkid15r avatar Jun 09 '23 00:06 arkid15r

Hey @abh31000, how is it going with your first l10n attempt? Do you need help with anything (just want to make sure you aren't stuck with some technical difficulty)?

Hey! It's just that I have been very busy lately. I will continue working on it when I find some free time. Thank you.

abh31000 avatar Jun 12 '23 20:06 abh31000