-
Notifications
You must be signed in to change notification settings - Fork 7
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
new definitions for poi layers #246
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Just a few minor change to the Changelog for consistency reasons.
f53a7b1
to
27d667d
Compare
I updated the changelog myself. So from myside you can go ahead an merge this PR. I suggest to use squash and merge. Cheers :) |
@@ -46,11 +46,68 @@ amenities: | |||
filter: amenity=* and (type:way or type:node) | |||
|
|||
poi: | |||
name: Points-of-Interests | |||
name: Points of interest | |||
description: All features with the amenities key. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be misunderstood, because you use more than just the amanity
key. Maybe you meant Amenities of public interest
?
description: | | ||
A lot of different objects such related to natural features | ||
transportation and amenities in a city. | ||
A lot of different objects related to natural features, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sounds a bit unprofessional, please rephrase.
transportation and amenities in a city. | ||
A lot of different objects related to natural features, | ||
transportation and amenities in a city. Definition taken from | ||
Waterproofing data project. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Source already given in source:
tag
Description
Renamed old poi layer to landmarks and created new one based on openpoiservice. Removed idel_vgi_poi layer.
Corresponding issue
Closes #161
Related #259
Checklist
main
(e.g. throughgit rebase main
)