-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Adding GB Subdivisions Per ISO 3166-2:GB #357
Comments
Hello @rubengmurray thank you for bringing up this issue again. It is time that we provide a clean solution to it.
Yes there were, but no work was made on this subject. Your contribution is more than welcome! As you can see—with my delay to answer you—I lack the time to implement all the ideas I have for this project, especially this one. There are two subjects here:
If you are willing to contribute, I would prefer to start with What do you think? |
I may be able to help with this. I have compiled a set of files containing subdivision information for each country, but I stalled when trying to come up with a way of structuring the hierarchy of data. I think it might be better to just have a flat list for each country. I agree that adding this information would greatly increase the size of the file and should probably be split into separate files. I need the information for a project due in January/February of the new year, so I should be able to help in the near future. |
Hi @mledoze @jjlharrison what did you end up doing for your project? Splitting them out into separate files is probably a good idea. I've knocked something up on a separate branch splitting the json out and I'm happy to push to this repo to share and work on if you're still interested? I don't have a ton of time either but can share the load on a branch or two. Cheers |
Hello @rubengmurray thank you for your work, yes I am still interested! I have looked at your branch on your fork. I would like to change some things:
What do you think? You said that you don't have a lot of time, I could continue your work if you want. I will also need to update the build code to accommodate these changes. |
Sounds good to me - I think this is it: Maybe we can merge this PR into a separate branch on this repo so you can do the build work? Cheers |
I have a use case for England, Scotland, Wales & Northern Ireland to be identified individually rather than blanket 'GB' attribution.
I note that this was mentioned quite a while ago in #6 (comment) but can't see there's been any discussion since...
@mledoze I'd be happy to contribute. Sounds like there were some hierarchy changes suggested for implementation, were they started? Did you have any thoughts on this specific issue? Could we put them at the top-level and retain shared information across the 4?
The text was updated successfully, but these errors were encountered: