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 locksmith shop #1188
Add locksmith shop #1188
Conversation
Results evaluating commit 137780c (merged with base 7e9a681 as 0c66a4d). See run details. PostgreSQL DB size in MB: 2792 ⇒ 2791 (-0.0% change)
expand for details...
|
@ZeLonewolf how do you decide which shops to show? |
Hi, thanks for the PR. Locksmiths are OK to add. I would like to not adding all shops with arbitrary values. There are a lot of names that are not useful at all. There is also a higher potential of vandalism with allowing |
@TomPohys good point! I was mostly asking how you choose which ones to show |
@farfromrefug, if the shop's characteristics make sense (e.g. does not have like 10 records) to show, let's add it. |
Fixes #566
This PR adds support for
shop=locksmith
.Example OSM object: https://www.openstreetmap.org/way/874850561
Rendered tile: