Skip to content

Add aliases and terms for shop=* presets#475

Merged
tyrasd merged 11 commits intoopenstreetmap:mainfrom
westnordost:shop-aliases
Aug 8, 2022
Merged

Add aliases and terms for shop=* presets#475
tyrasd merged 11 commits intoopenstreetmap:mainfrom
westnordost:shop-aliases

Conversation

@westnordost
Copy link
Copy Markdown
Contributor

@westnordost westnordost commented May 31, 2022

Also, corrected shop=deli preset name. In American English, "Deli" means something else as what shop=deli is originally defined with.

Sources: English Wikipedia, OpenStreetMap Wiki

Comment thread data/presets/shop/toys.json
Comment thread data/presets/shop/ticket.json Outdated
Comment thread data/presets/shop/stationery.json Outdated
Comment thread data/presets/shop/second_hand.json
Comment thread data/presets/shop/butcher.json Outdated
Comment thread data/presets/shop/confectionery.json
Comment thread data/presets/shop/country_store.json Outdated
Comment thread data/presets/shop/car_repair.json Outdated
Comment thread data/presets/shop/antiques.json Outdated
Comment thread data/presets/shop/books.json Outdated
Comment thread data/presets/shop/car_repair.json Outdated
Comment thread data/presets/shop/convenience.json Outdated
Comment thread data/presets/shop/gift.json
Comment thread data/presets/shop/health_food.json Outdated
Comment thread data/presets/shop/newsagent.json Outdated
Comment thread data/presets/shop/pawnbroker.json Outdated
Comment thread data/presets/shop/variety_store.json Outdated
Comment thread data/presets/shop/car_repair.json
Comment thread data/presets/shop/water_sports.json Outdated
Comment thread data/presets/shop/hairdresser.json Outdated
Comment thread data/presets/shop/deli.json Outdated
@westnordost
Copy link
Copy Markdown
Contributor Author

Alright, thanks for the review you all. All suggestions are incorporated now. This PR is ready for final review by the repo maintainer(s).

@tyrasd tyrasd force-pushed the main branch 2 times, most recently from 9d3204d to 49f529e Compare June 22, 2022 16:19
@tyrasd tyrasd added the new-label changes the name, aliases or terms of a preset label Aug 7, 2022
Copy link
Copy Markdown
Member

@tyrasd tyrasd left a comment

Choose a reason for hiding this comment

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

👍 Thanks for the PR and please excuse the rather long delay. Please see a few minor questions and remarks below.

Comment thread data/presets/shop/butcher.json Outdated
Comment thread data/presets/shop/butcher.json
Comment thread data/presets/shop/deli.json Outdated
Comment thread data/presets/shop/deli.json
@westnordost
Copy link
Copy Markdown
Contributor Author

Ready for re-review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-label changes the name, aliases or terms of a preset

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants