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.

Sign up for GitHub

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

Jump to bottom

Update regions.yaml #193

Merged
merged 5 commits into from
Dec 6, 2021
Merged

Update regions.yaml #193

merged 5 commits into from
Dec 6, 2021

Conversation

GITMichaelK
Copy link
Contributor

added: germany

@GITMichaelK
Update regions.yaml
fdf2b47 
added: germany
@jbelien
Copy link
Member

jbelien commented Nov 29, 2021

Hello @GITMichaelK,

Thank you for your contribution.
Could you follow the contributing guidelines and add what's missing (GeoJSON file of the region)?

@jbelien jbelien added the region Add a new region label Nov 29, 2021
@GITMichaelK
Copy link
Contributor Author

GITMichaelK commented Dec 1, 2021 •
edited
Loading

Hello @jbelien,
Thank you and sorry that I missed the GeoJSON file, I did my best to add it now; however I'm not sure if it's correct: I used http://polygons.openstreetmap.fr/?id=51477
How to get the checks from a maintainer?
Thanks!!

@jbelien jbelien added this to the Version 2.2 milestone Dec 6, 2021
@jbelien jbelien mentioned this pull request Dec 6, 2021
Closed
@jbelien jbelien merged commit 69b6091 into osmbe:2.x Dec 6, 2021
@jbelien
Copy link
Member

jbelien commented Dec 6, 2021

🚀 Deployed: https://welcome.osm.be/germany/list

@jbelien
Copy link
Member

jbelien commented Dec 6, 2021

There seems to be an issue in the process for Germany currently, I'll have a look.

@jbelien
Copy link
Member

jbelien commented Dec 10, 2021

Follow up on the issue will be tackled in #262

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
region Add a new region
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@GITMichaelK @jbelien