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

Put only city names into the app description #19

Open
mswart opened this issue Jul 19, 2013 · 13 comments
Open

Put only city names into the app description #19

mswart opened this issue Jul 19, 2013 · 13 comments

Comments

@mswart
Copy link

mswart commented Jul 19, 2013

The application description should be improved by add only a list of supported cities (much smaller) and maybe later a full list or a link to a full list of canteens.

@jgraichen
Copy link
Collaborator

I think that would require to have a distinct city attribute on canteen resources available on OpenMensa API.

@mswart
Copy link
Author

mswart commented Jul 19, 2013

It could be done manually or via using the city value from the canteen address or do we not provide the address via the API?

@jgraichen
Copy link
Collaborator

Address is provided but you still need to parse that address and there is no standardized format (enforced). We should really add an additional city attribute.

@domoritz
Copy link
Owner

Since the API does not provide this information, I'm going to close this issue for now.

@jgraichen
Copy link
Collaborator

Canteen resources do have a city field.

@domoritz domoritz reopened this May 24, 2019
@domoritz
Copy link
Owner

@l-jonas do you want to group canteens by city then?

@l-jonas
Copy link
Collaborator

l-jonas commented May 25, 2019

@domoritz In the app description? Or are you talking about the canteen list? Which one (favorite selection or dropdown). The search in the favorite selection already uses the city field.

@domoritz
Copy link
Owner

I thought he meant the canteen list. It could be grouped by city or we could at least show the city in light gray below the name of the mensa. I don't really have strong opinions here.

@l-jonas
Copy link
Collaborator

l-jonas commented May 27, 2019

The city field is currently not usable, some cities are a bit wrong:

  • Würzburg, W
  • Zürich, Zürich-Mülligen
  • L instead of Lüneburg
  • Halle, Halle (Saale)
  • Garching, Garching bei München
  • Frankfurt, Frankfurt am Main

@klemens
Copy link

klemens commented Jun 1, 2019

@l-jonas

I corrected the ones you mentioned except Zürich, as I could not open the parser (server is rather slow currently). We could easily fix the rest. I also noticed that we currently do not display the city anywhere on the canteen site, but this should also be an easy fix.

When we have support for the cities in the app and on openmensa.org, we could then also remove the redundant city prefixes from the names (which have always annoyed me 😅).

Btw. I really like your update of the app!

@l-jonas
Copy link
Collaborator

l-jonas commented Jun 2, 2019

@klemens There are still problems:

  • the database contains two different writings for Würzburg
    • 259 is correct
    • the others have got a invisible special char between "Würz" and "burg"
    • this different cases are interpreted as different cities => not good
  • now there is "Garching b.München" and "Garching bei München"
  • and as you said, Zürich

I've built a two level canteen selection (city, then the canteen within the city; both lists suggest nearby items if a location is available). This needs an accurate city list without duplicates.

@klemens
Copy link

klemens commented Jun 2, 2019

@l-jonas

Würzburg

There was a SOFT HYPHEN in there, which should normally be ignored by a unicode string comparison (e.g. in the firefox site search, it finds both the ones with and without hyphens when searching for Würzburg). But I have removed it anyway.

Garching

Fixed, there were actually two parsers für München. 🤔

Zürich

Still cannot open the parser (openmensa/openmensa#97), but fixed https://openmensa.org/c/646 manually. Not sure if there are more.

@l-jonas
Copy link
Collaborator

l-jonas commented Jun 2, 2019

@klemens Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants