Add shops from osm.org and iD#1447
Conversation
| - filter: | ||
| shop: [bakery, books, butcher, car, car_repair, clothes, computer, convenience, | ||
| fashion, florist, gift, greengrocer, hairdresser, jewelry, mobile_phone, optician, pet] | ||
| shop: [art, bakery, beauty, books, butcher, car, car_repair, clothes, |
There was a problem hiding this comment.
shop=coffee and shop=deli seem more like amenity=cafe and should get that min_zoom range.
shop=variety_store and shop=furniture and shop=shoes seems more like craft: [brewery, carpenter...] and should get that min_zoom range. Even though this has been around awhile, shop=pet should also be in this set.
I'm fine with the rest being min_zoom: 17.
There was a problem hiding this comment.
shop=variety_store and shop=furniture and shop=shoes seems more like craft: [brewery, carpenter...] and should get that min_zoom range. Even though this has been around awhile, shop=pet should also be in this set.
I'm making the changes, but craft is a bit of a weird thing in OSM, and a better word would probably have been trade. I don't see the craft values and those shop values forming much of a coherent set - some are places a customer goes to buy stuff, others don't interact directly with customers at their locations.
|
Pushed in a missing file, haven't handled the review comments yet |
fac4149 to
1f05da5
Compare
|
Everything should be good, and PEP style issues are fixed. |
This is a big part of #1425