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 Hausa locale information #512

Closed
wants to merge 4 commits into from

Conversation

petyaraykovska
Copy link
Contributor

@petyaraykovska petyaraykovska commented Jul 15, 2016

$hau->lang_code_iso_639_1 = 'ha';
$hau->lang_code_iso_639_2 = 'hau';
$hau->lang_code_iso_639_3 = 'hau';
$hau->slug = 'hau';
Copy link
Contributor

Choose a reason for hiding this comment

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

Any particular reason for the slug change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We're using ISO 639-3 codes for new WordPress locales and where possible adapting old ones as well (https://make.wordpress.org/polyglots/handbook/translating/requesting-a-new-locale/#find-your-language-code). Hausa has existed in GlotPress for a while but it doesn't have a WP locale and didn't have a site. The ISO 639-3 code is applied to the slugs as well as the language code for consistency. The site is already created with the slug hau.

@ocean90
Copy link
Member

ocean90 commented Jul 20, 2016

@petyaraykovska Instead of updating the existing locale, let's add hau as a new one.

As per @ocean90's request, adding a new locale for Hausa in Latin script and bringing back the data for the old rtl locale. As per a GTE request & information on Ethnologue and Wikipedia, also fixing the Kirghiz locale code (former ky, which is the wrong code for the language) and exchanging the locale codes with the ISO 639-3 code.
@toolstack toolstack added the [Type] Enhancement A suggestion for improvement. label Jul 26, 2016
@toolstack toolstack added this to the 2.2 milestone Jul 26, 2016
- $ha->slug = 'ha';
- $ha->text_direction = 'rtl';
- $ha->google_code = 'ha';
- $ha->facebook_locale = 'ha_NG';
Copy link
Member

Choose a reason for hiding this comment

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

The - shouldn't be there...

@ocean90
Copy link
Member

ocean90 commented Sep 4, 2016

Please create one PR per locale.

@ocean90 ocean90 removed the [Type] Enhancement A suggestion for improvement. label Sep 19, 2016
@ocean90 ocean90 removed this from the 2.2 milestone Sep 19, 2016
@ocean90
Copy link
Member

ocean90 commented Sep 19, 2016

New PRs: #549 and #550

@ocean90 ocean90 closed this Sep 19, 2016
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.

3 participants