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

[ADD] base: add Arabic languge #28640

Closed
wants to merge 1 commit into from

Conversation

mart-e
Copy link
Contributor

@mart-e mart-e commented Nov 13, 2018

Odoo only has ar_SY language but not a generic Arabic language.

cc @odony to be sure it is a stable change

@sswapnesh
Copy link
Contributor

May be better to add in master this csv will be loaded when someone upgrade base module.

@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Nov 13, 2018
@mart-e
Copy link
Contributor Author

mart-e commented Nov 13, 2018

Well as odoo finally works well with Arabic languages, it is the right time to have a proper Arabic language.

@C3POdoo C3POdoo added the RD research & development, internal work label Nov 14, 2018
@mart-e mart-e force-pushed the 12.0-arabic-language-code-mat branch from 7666ea5 to 224fda1 Compare February 18, 2019 08:31
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Feb 18, 2019
@mart-e
Copy link
Contributor Author

mart-e commented Feb 18, 2019

robodoo r+

@mart-e mart-e mentioned this pull request Feb 18, 2019
@mart-e mart-e force-pushed the 12.0-arabic-language-code-mat branch from 224fda1 to f900b70 Compare February 18, 2019 08:59
@robodoo robodoo removed the r+ 👌 label Feb 18, 2019
@mart-e mart-e force-pushed the 12.0-arabic-language-code-mat branch from f900b70 to 13c200d Compare February 18, 2019 09:01
@@ -1,8 +1,10 @@
"id","name","code","iso_code","direction","grouping","decimal_point","thousands_sep","date_format","time_format","translatable","week_start"
"base.lang_en","English","en_US","en","Left-to-Right","[3,0]",".",",","%m/%d/%Y","%H:%M:%S","True","7"
"base.lang_am_ET","Amharic / አምሃርኛ","am_ET","am_ET","Left-to-Right","[3,0]",".",",","%d/%m/%Y","%I:%M:%S","True","7"
"base.lang_ar_SY","Arabic / الْعَرَبيّة","ar_SY","ar","Right-to-Left","[3,0]",".",",","%d %b, %Y","%I:%M:%S","True","6"
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's use ar_AA (Arabic "Unitag") as a countryless ISO code for arabic :-)

Now that we have proper support for RTL language, add an Arabic language with
code "ar" (while the only possiblity before was ar_SY).
Use ar_AA code as Arabic unitag.
In master, the ar_SY could be removed as no need to have both.

Add Bengali language
Fixes odoo#31165
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Feb 18, 2019
@mart-e mart-e force-pushed the 12.0-arabic-language-code-mat branch from 13c200d to 89dabeb Compare February 18, 2019 10:00
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Feb 18, 2019
@mart-e
Copy link
Contributor Author

mart-e commented Feb 18, 2019

robodoo r+

robodoo pushed a commit that referenced this pull request Feb 18, 2019
Now that we have proper support for RTL language, add an Arabic language with
code "ar" (while the only possiblity before was ar_SY).
Use ar_AA code as Arabic unitag.
In master, the ar_SY could be removed as no need to have both.

Add Bengali language
Fixes #31165

closes #28640
@robodoo
Copy link
Contributor

robodoo commented Feb 18, 2019

Merged, thanks!

@robodoo robodoo closed this Feb 18, 2019
@mart-e mart-e deleted the 12.0-arabic-language-code-mat branch April 2, 2019 11:16
@KangOl
Copy link
Contributor

KangOl commented Apr 3, 2019

This patch wasn't stable due to uniqness of lang name.

('name_uniq', 'unique(name)', 'The name of the language must be unique !'),


Actually works because loading of csv files update existing row before creating new one.
Was failing on my test database due to lang_ar_SY being in noupdate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants