Skip to content

Bugfix/UI fixes#90

Merged
EnricoBausenhart merged 4 commits intomainfrom
bugfix/ui-fixes
Jul 19, 2025
Merged

Bugfix/UI fixes#90
EnricoBausenhart merged 4 commits intomainfrom
bugfix/ui-fixes

Conversation

@EnricoBausenhart
Copy link
Copy Markdown
Collaborator

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • ♻️ Code refactor
  • 📝 Documentation update
  • 🚀 Performance improvement
  • ✅ Test addition/update
  • 🧹 Chore/maintenance
  • 📐 API Spec update
  • Other (please describe):

Changes

  • Page reloads on subpages like /matchrequests were failing -> Solution: add nginx config and copy it in Dockerfile to have nginx SPA fallbacks
  • Correct formatting of timeslots on the "Last 5 Pending Match Requests" tab
  • API returns 409 if a match request is sent but there already is a request for that day -> Error message is now displayed to inform the user about that constraint
  • When selecting the 20.7 for example, the request was sent with the 19.7 -> Cause: The time picker took 20.7 00:00 and performed a UTC shift when formatting the date for the request

Copy link
Copy Markdown
Collaborator

@stark-cybertech stark-cybertech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good fix!

@EnricoBausenhart EnricoBausenhart merged commit 6933149 into main Jul 19, 2025
34 checks passed
@a-kori a-kori deleted the bugfix/ui-fixes branch July 20, 2025 21:56
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.

3 participants