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 for man_made=flare #1237

Merged
merged 1 commit into from
Jun 14, 2024
Merged

add preset for man_made=flare #1237

merged 1 commit into from
Jun 14, 2024

Conversation

k-yle
Copy link
Collaborator

@k-yle k-yle commented May 25, 2024

add preset for man_made=flare

Copy link

🍱 You can preview the tagging presets of this pull request here.

data/presets/man_made/flare.json Show resolved Hide resolved
data/presets/man_made/flare.json Show resolved Hide resolved
@tordans
Copy link
Collaborator

tordans commented Jun 5, 2024

I was wondering which other fields would fit. https://taginfo.openstreetmap.org/tags/man_made%3Dflare#combinations is not very helpful. Looking at a few examples https://overpass-turbo.eu/s/1Mvs showed things like name, material, description.

@1ec5 if you have the time could you create a qlever query showing the number of other keys that are used together with the tag man_made=flare?
This would give a quick list to fields that we can add as fields/moreFields.


Testing

@tordans
Copy link
Collaborator

tordans commented Jun 14, 2024

I think we can merge this now and add other fields later – see #1237 (comment) for thoughts on that.

@tordans tordans merged commit 2bd6a13 into openstreetmap:main Jun 14, 2024
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.

2 participants