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

Add Autogrill and Chef Express to cafe.json #8918

Closed
wants to merge 7 commits into from
Closed

Conversation

Mannivu
Copy link
Contributor

@Mannivu Mannivu commented Nov 8, 2023

Both Autogrill and Chef Express have cafes and restaurants POIs, so I added them to both cafe and restaurant entries

Add brands. Both Autogrill and Chef Express have cafe and restaurants POIs, so I added them to both `cafe` and `restaurant` entries
@AntiCompositeNumber AntiCompositeNumber changed the title Add brands Add Autogrill and Chef Express to cafe.json and restaurant.json Nov 8, 2023
@AntiCompositeNumber
Copy link
Collaborator

Both Autogrill and Chef Express are catering companies operating a variety of external and internal brands in airports/train stations/motorway services. As far as I can tell, the only thing using the Chef Express brand is https://www.chefexpress.it/insegna/chef-express/, which I'd call a cafe. Autogrill operates a cafe using their branding, but calls it Acafè. They also have a cafeteria called Autogrill Più. I don't see anything on their website that uses just the Autogrill branding, other than the service areas themselves.

@Mannivu
Copy link
Contributor Author

Mannivu commented Nov 8, 2023

While I can agree on Chef Express, there are a lot of Autogrill POOs that are not restaurants and have the "Autogrill" insigna (I have to use GMaps since Autogrill hasn't got a dedicated page for their restaurants and cafes)

So having "Autogrill" for restaurants is not wrong IMHO.

Removed Chef Express, only a cafe
@Mannivu Mannivu changed the title Add Autogrill and Chef Express to cafe.json and restaurant.json Add Autogrill and Chef Express to cafe.json Nov 8, 2023
@AntiCompositeNumber AntiCompositeNumber added the needs discussion Waiting for other contributors to voice their opinion label Nov 17, 2023
@AntiCompositeNumber
Copy link
Collaborator

@DnaX @ThorOnTheRocks (Listed for Italy on #2588), do you have thoughts on whether Autogrill belongs in amenity/restaurant.json?

I'm not particularly opposed, but I'm also not confident enough to merge it myself.

Fix Brico area, add Brico OK
@Mannivu Mannivu closed this Mar 2, 2024
@DnaX
Copy link
Contributor

DnaX commented Mar 3, 2024

@AntiCompositeNumber sorry for the late answer.

For me Autogrill always have a cafe zone as well restaurant area. For suce you can eat decently there!

Altrough this PR cointains too much things in order to be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs discussion Waiting for other contributors to voice their opinion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants