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

Greek language code gr vs. el #57

Closed
marcelkottmann opened this issue Aug 29, 2017 · 3 comments
Closed

Greek language code gr vs. el #57

marcelkottmann opened this issue Aug 29, 2017 · 3 comments

Comments

@marcelkottmann
Copy link

Hello,

since a few days greek is supported as language, using the getName(country, language) function. But I wonder why the language code was renamed from el to gr in this commit:

a331231

I thought that ISO_3166-1 define countries and e.g. iso 639 define language codes? Since you can also pass "ar" as language, which in this context is the language code from "arabic" and not the country "argentina". So it is only consequent to use "el" instead of "gr" as language code, when you mean greek (the language).

I think I am also confused, because the documentation of the getName function does not clearly state, what iso-definition the language-parameter should conform to.

@michaelwittig
Copy link
Owner

the language should follow https://en.wikipedia.org/wiki/List_of_ISO_639-1_codes

as you said: el is correct, not gr

@michaelwittig
Copy link
Owner

released as i18n-iso-countries@2.0.0

since this is a breaking change we are now on version 2 :)

@marcelkottmann
Copy link
Author

sorry about 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

No branches or pull requests

2 participants