Skip to content

fix: restaurant info update in manager panel#115

Open
kingazm wants to merge 2 commits intomainfrom
fix-restaurant-update
Open

fix: restaurant info update in manager panel#115
kingazm wants to merge 2 commits intomainfrom
fix-restaurant-update

Conversation

@kingazm
Copy link
Copy Markdown
Collaborator

@kingazm kingazm commented Apr 15, 2026

📄 Pull Request Description


Important: this PR will not work without this one: JUCanEat/java-backend#85

🧩 What was changed?

  • Fixed bug causing issues with updating description or opening hours for restaurant
  • Extended opening hours functionality in UI to allow to update opening hours separately for each day, and display this data
  • Moved some interfaces to interfaces file to improve code readability

💡 Why was it changed?

#107
#108


⚙️ How was it implemented?

  • Refactored the restaurant opening hours UI to allow both "same hours every day" and per-day editing.
  • Updated the modal to use a responsive, modern layout with compact time fields that remain on the same row on mobile.
  • Ensured all time fields can be left empty, supporting closed days without a separate checkbox.
  • Updated the facility info box to display opening hours in a compact, two-column layout, with accurate handling of days with no hours set.

Maintained current coding practices and used already introduced frameworks.


⚠️ Side Effects or Risks

None that I am aware of, except for restaurant updates working better:)


✅ Checklist

  • All 4 sections above are clearly filled out
  • Tests and documentation updated

@kingazm kingazm requested a review from cssma April 15, 2026 12:39
@kingazm kingazm force-pushed the fix-restaurant-update branch from 6462375 to 78f98d4 Compare April 15, 2026 14:36
@kingazm kingazm marked this pull request as draft April 15, 2026 14:37
@kingazm kingazm marked this pull request as ready for review April 15, 2026 17:31
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

Successfully merging this pull request may close these issues.

1 participant