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

Mensaplan broken #2

Closed
christophfriedrich opened this issue Oct 27, 2016 · 4 comments
Closed

Mensaplan broken #2

christophfriedrich opened this issue Oct 27, 2016 · 4 comments

Comments

@christophfriedrich
Copy link
Member

Since the Studentenwerk changed their Mensaplan publishing method, the mensa parser is broken. There seems to be an API for the new system which already works in the WiWi app. We should wait for the outcome of this conversation before attempting to fix anything ourselves.

@christophfriedrich
Copy link
Member Author

I wrote to the Studentenwerk from our fsgi mail, they forwarded that to infomax.de who manages the plans for them and they gave us the information that the new publishing method is via XMLs that are located at http://speiseplan.stw-muenster.de/mensa_am_ring.xml I implemented the new method into mensaplaene.php with bc37750

@noerw
Copy link
Member

noerw commented Nov 14, 2016

The filter for Aktionsstand (WOK) does not work.
photo465669404184651972

Can we just limit the output to 4 items? It takes to much space.

@noerw noerw reopened this Nov 14, 2016
@christophfriedrich
Copy link
Member Author

Yes, the filter does work, it's supposed to only remove the "Heute am " to make it somewhat shorter.

I thought about limiting the items too, but couldn't decide which ones to keep and which not. If we reduce it, I'd say keep only Menu 1-3 and nothing else. To start with, I'd certainly remove the Dessertbuffet.

Alternatively, use a smaller font size? (Or redesign the whole thing?)

christophfriedrich added a commit that referenced this issue Nov 14, 2016
Exclude Dessertbuffet, glue with implode to remove trailing `<br>`, stylable with id `#mensaplan`
ref issue #2
christophfriedrich added a commit that referenced this issue Nov 14, 2016
@noerw
Copy link
Member

noerw commented Feb 6, 2017

should be working now

@noerw noerw closed this as completed Feb 6, 2017
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

2 participants