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

[shop/furniture] add The Bed Shop #10678

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

RicsPOITT
Copy link
Contributor

Adding new brand for Furniture store in south Africa called The Bed Shop

https://www.wikidata.org/wiki/Q129571574
https://thebedshop.co.za/

Adding new brand for Furniture store in south Africa called The Bed Shop

https://www.wikidata.org/wiki/Q129571574
https://thebedshop.co.za/
@Snowysauce Snowysauce changed the title Update furniture.json for south africa new brand [shop/furniture] add The Bed Shop Mar 13, 2025
@Snowysauce
Copy link
Collaborator

Locations currently mapped in OSM are split between shop=bed (3 POIs) and shop=furniture (2).

duplicates of the main tags, and no other language tags are present
@Snowysauce Snowysauce added add brand Actionable - add a brand to the index tagging labels Mar 13, 2025
@RicsPOITT
Copy link
Contributor Author

It would actually make more sense to have it as shop=bed reason i added it as shop=furniture was because some if these brands locations also specializes in furniture. Now thinking of it definately under shop=bed

do you need me to close the pr and reopen new one under correct json?

@Snowysauce
Copy link
Collaborator

do you need me to close the pr and reopen new one under correct json?

Not at all; if you move the preset to the shop=bed json file in your local copy of the NSI, GitHub will detect that change and automatically update this PR with the new commits.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add brand Actionable - add a brand to the index tagging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants