neuland-ingolstadt / neuland.app

A free & open source, web-based replacement for the official app of the Technische Hochschule Ingolstadt built with React and Next.js.
https://neuland.app
GNU Affero General Public License v3.0
36 stars 33 forks source link

✨ add static Reimanns meals #309

Closed BuildmodeOne closed 1 year ago

BuildmodeOne commented 1 year ago

Closes #293

localhost_3000_food_(iPhone 12 Pro) (1)

πŸ€– Generated by Copilot at 50bdaed

Summary

πŸ½οΈπŸ“…πŸŒŸ

This pull request enhances the app's functionality and user interface for showing the meals of Reimanns restaurant. It adds static fallback meals, a weekly view, and more details about the variations of each meal. It modifies the getReimannsMeals function, the food.js file, the Mensa.module.css file, and the date-utils.js file. It also adds a new JSON file with the static meals.

We scrape the web for food of doom We fill the gaps with static meals We render variations of Reimanns' gloom We fix the styles and margins of our deals

Walkthrough

alexhorn commented 1 year ago

Are the prices really identical for students / employees / guests?

BuildmodeOne commented 1 year ago

Are the prices really identical for students / employees / guests?

I spoke to Mr. Reimann personally about the prices and the menus etc. are not discounted.

BuildmodeOne commented 1 year ago

image

I made some changes to the food page to clean it up a bit. Please let me know what you think :)

Robert27 commented 1 year ago

I really like the new design of the category headings. However, I am not sure about the green / red label.

Why do we show the abbreviations on the home page at all? Unless the user has selected allergies or flags, this info is relatively irrelevant. Besides, no user knows to know what, for example, the code "3" even means.

My suggestion would be uniformly to hide all abbreviations and show them only if there are matches with the filter criteria. Maybe we could make an exception for vegan or vegetarian.

BuildmodeOne commented 1 year ago

@Robert27

I tried to integrate the suggested changes:

image