Add preset for leisure=summer_camp#2000
Conversation
|
🍱 Your pull request preview is ready Please use this preview to check your changes. Ideally use the test documentation template and document your test results by commenting on the PR. This will speed up the review process for everyone. FYI, once this PR is merged, you can use the iD Editor Preview to test your changes in interaction with all other changes. |
There was a problem hiding this comment.
disclaimer: I am not really familiar with this POI, but based on #1495 (review) and OSM Wiki and cursory check of info on Internet it seems a real and reasonable POI type
I expect it will stay open for a while for review from wider OSM community
(but even if tagging scheme is bad, that does not change that PR itself is fine and was based on open issue)
|
I approve this change. But I suggest we add It shouldn't hold up this PR, but it looks like the seasonal field is only a yes/no toggle, ideally it would support all the other values Other fields could include:
|
|
discussion is active at https://community.openstreetmap.org/t/if-leisure-summer-camp-is-a-terrible-tagging-schema-it-would-be-a-good-moment-to-protest/142098/31 so for now I would wait with merge I posted there that I would recommend waiting with merge here, as long as discussion is active or if there would be attempt to introduce a less confusing tagging schema. |
|
EDIT: looks like it will be merged relatively soon |
|
Thanks for the PR! It is now merged but note that before you will see it in iD few things need to happen. iD tagging schema needs to get release (see https://github.com/openstreetmap/id-tagging-schema/releases - latest one mentioned there is just a draft). And later needs to be pulled in by iD. Thanks again for your contribution! If you are interested in making other one it would be really welcome! |
leisure=summer_camp

Description, Motivation & Context
Added a preset for
leisure=summer_campto resolve #1481.Related issues
Closes #1481
Created because the previous pr #1495 attempting to solve this seemed to fail builds and abandoned.
Links and data
Relevant OSM Wiki links:
Relevant tag usage stats:
Checklist and Test-Documentation Template
Read on to get your PR merged faster…
Follow these steps to test your PR yourself and make it a lot easier and faster for maintainers to check and approve it.
This is how it works:
After you submit your PR, the system will create a preview and comment on your PR:
Once the preview is ready, use it to test your changes.
Now copy the snippet below into a new comment and fill out the blanks.
Now your PR is ready to be reviewed.