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

Copy Despar to shop=convenience with an extended locationSet; unify Spar brands #10178

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

Conversation

gy-mate
Copy link
Contributor

@gy-mate gy-mate commented Nov 26, 2024

Despar currently only has a shop=supermarket entry for Italy. But the chain also has shop=convenience stores both in Italy and Hungary.

See https://en.wikipedia.org/wiki/Spar_(retailer)#:~:text=There%20are%20some%20international%20naming%20variants for more info.

@Snowysauce Snowysauce added the add brand Actionable - add a brand to the index label Nov 26, 2024
@UKChris-osm
Copy link
Collaborator

Would creating a DESPAR brand be more suitable? Despar Facebook

@gy-mate
Copy link
Contributor Author

gy-mate commented Nov 28, 2024

@UKChris-osm

Good idea—done! But I would keep the capital case (Despar) in accordance with other Spar entries and https://en.wikipedia.org/wiki/Talk:Spar_(retailer)#Requested_move_13_September_2022.

@gy-mate gy-mate marked this pull request as draft November 28, 2024 18:03
@gy-mate gy-mate marked this pull request as ready for review November 29, 2024 08:33
@gy-mate gy-mate changed the title Copy Despar to shop=convenience with a locationSet extended to hu Copy Despar to shop=convenience with an extended locationSet; unify Spar brands Nov 29, 2024
@gy-mate
Copy link
Contributor Author

gy-mate commented Nov 29, 2024

@UKChris-osm Based on @urbalazs's request I've unified all *spar entries to use brand=Spar.

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants