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

tbd #124

Draft
wants to merge 36 commits into
base: main
Choose a base branch
from
Draft

tbd #124

wants to merge 36 commits into from

Conversation

SjoenH
Copy link
Collaborator

@SjoenH SjoenH commented May 9, 2024

No description provided.

SjoenH added 30 commits April 25, 2024 11:14
New booking page
- Not tightly linked to a draw. Show the whole year and let the user indicate when they want it.

Styling
- Remove base styling that broke normal input style etc
- Add daisyui so that it is easier to do simple styling + themes
- Removed the current theme and dark-mode implementation...
This commit partially simplifies the FormCalendar component in several ways:
- Removed unnecessary comments.
- Used destructuring to import useState from react.
- Simplified handleWeekSelect function by using a ternary operator.
- Simplified the creation of currentWeek by using Array.fill().
- Kept the weekNumber variable for clarity and readability.
- Simplified the TableHeader and WeekRow components by directly returning the JSX.
- Simplified the onChange handler in the WeekRow component by using a short-circuit evaluation."
Updated the URL rewrite rule in `web.config` to ensure that certain static files,
including `manifest.json`, `.ico`, and image files, are not rewritten to `/`. This
change prevents the unintentional rewrite of these files, which can cause them to
not be served correctly.

Additionally, verified and ensured correct MIME types for static content, including
`application/json` for JSON files.

This should resolve issues with static resources not loading properly on IIS.
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