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

added missing country code #421

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

Conversation

VirtualRoad
Copy link
Contributor

added missing country code of the Democratic Republic of the Congo, as the list contains only the code of the neighboring country with a similar name - the Republic of the Congo. They are two different countries.

added missing country code of Democratic Republic of the Congo, as the list contains only the code of the neighboring country with a similar name - Republic of the Congo. They are two different countries.
@hellais
Copy link
Collaborator

hellais commented Jan 1, 2019

Good call, though I actually wonder if we want to be keeping a list of iso alpha2 country codes to country names mappings inside of this repo.

@sneft @jakubd do you recall what the original reason to have these country codes in the repo was? Do you think it would be reasonable to drop this file from the repo and just document that the country codes are ISO 3166 alpha2.

@sneft
Copy link
Collaborator

sneft commented Jan 11, 2019

I think if we point to a definitive external source for the country codes I have no problem removing this file. (I usually just google it myself, anyway)

@hellais
Copy link
Collaborator

hellais commented Jan 13, 2019

I think the wikipedia page on ISO 3166 alpha2 and the official ISO standards code page are the most authoritative sources on this.

In relation to mapping the country codes to names automatically, this is something that is a bit more tricky to do (for example Taiwan is labeled by the ISO body as "Taiwan, Province of China"). I started this project: https://github.com/hellais/country-util to do that. Instead of the "official" ISO name of the countries, it uses the names from the Unicode Common Locale Data Repository (CLDR) Project, which is a bit more responsive to community input and has more sensible names for countries.

We can maybe link all three of these resources and then it's up to consumers of the data to make up their mind on how to resolve a country code to a name (which I think is beyond the scope of this repo).

@agrabeli agrabeli requested a review from hellais January 26, 2022 18:34
@agrabeli
Copy link
Collaborator

@hellais May we merge this PR?

@RandomGuy48
Copy link
Contributor

If this PR has been recognized to be merged, I would like to point out that aside from adding CD,"Congo,Democratic Republic of" which corresponds to https://github.com/citizenlab/test-lists/blob/master/lists/cd.csv, please consider adding the following country codes as well:

Additionally, please consider changing the following:

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.

6 participants