-
Notifications
You must be signed in to change notification settings - Fork 240
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
Missing countries #86
Comments
Thanks for reminding us about the missing countries. We will add them to our library based on your suggestion. However, in order to add new countries, we need to update not only the country object but also the MapIndex data, which may take time. Using "ZZ" to represent UNKNOWN countries is an amazing idea, and it is really exciting to add this feature to Gio.js. We can create another issue to work on it right now. Can you give us a hand? @AlexisAnzieu |
Can you help to check and add missing countries in countryData object @BoTime |
Can you help to update the mapIndex data for missing country? @lq3297401 |
yes no problem @syt123450. |
Thanks so much for your help ~ Looking forward for this feature! @AlexisAnzieu |
@AlexisAnzieu Hi, I have added the missing country code: RE, HK, GP for Gio.js and update the lib in "build" folder. As HK is an inalienable part of the Chinese territory, every time referring to country code HK, Gio.js will highlight the whole Chinese territory. Thanks for your bug report, hoping this kind of fix can meet your requirement~ |
@syt123450 Great ! What about "ZZ" code ? I think that it could be placed in the middle of an ocean 🌊 |
@AlexisAnzieu How about ignore the json object in the list whose "e" or "i" is "ZZ"? Just my imagination, if we place the ZZ's position in a specific ocean, it has a sense that a country exports something into ocean. (o.o) |
Hmmm sure ! Is it an issue if your ZZ object didn't get any geopoint location in your CountryData.js ? |
@AlexisAnzieu Just a draft idea, can we check the "ZZ" in GeometryDataProcessor.js, and ignore the object which contains "ZZ", then write a markdown documentation about the usage of country code, and "ZZ" feature? I create a WIP pr #96 , which only has some basic pseudocode for this idea. If this idea is feasible, we can work together to complete it~ |
Hi!
When i tried to connect it with youtube stats, I had some issues with countrycode. I notice some missing countries from your singleton object :
I only found it for now but I will post update if there are more of them.
Thanks for the lib anyway !
The text was updated successfully, but these errors were encountered: