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

Add preset man_made=insect_hotel #1020

Merged
merged 1 commit into from
Dec 15, 2023
Merged

Conversation

tordans
Copy link
Collaborator

@tordans tordans commented Oct 5, 2023

This adds a preset for https://wiki.openstreetmap.org/wiki/Tag:man_made%3Dinsect_hotel

The fields are all "moreFields" because I find them not too relevant for regular mapping.
Some fields are taken from the beehive preset, which is kind of similar.

The icon is https://fontawesome.com/icons/bugs?f=classic&s=solid
A bee icon would be more fitting, but with only 1.500 uses not worth the effort for now.
image


Here are all cases in Berlin https://overpass-turbo.eu/s/1BtI

This is what it looks like in the preview
https://pr-1020--ideditor-presets-preview.netlify.app/id/dist/?node=11236007840#background=Berlin-2022&disable_features=boundaries&id=n11236007840&map=20.00/52.51164/13.37981
image

@github-actions
Copy link

github-actions bot commented Oct 5, 2023

🍱 Preview the tagging presets of this pull request here: https://pr-1020--ideditor-presets-preview.netlify.app/id/dist/#locale=en.

Copy link
Contributor

@matkoniecz matkoniecz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested autodeployed test version, looks fine to me

can regular contributors do something to help with PR pile? Is review like this useful?

@matkoniecz
Copy link
Contributor

According to my knowledge this tag is OK and accepted by community.

@tyrasd tyrasd added this to the v6.6 milestone Dec 15, 2023
@tyrasd tyrasd merged commit 8864153 into openstreetmap:main Dec 15, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants