-
Notifications
You must be signed in to change notification settings - Fork 0
Merge into master #24
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
No password change (yet).
"Settings" is very generic. This will give us problems later, with importing components in the router file.
I think tailwind was the issue. Also, the color theme is now always light.
At this point in time, we don't have an API gateway. So I created another axios instance, just for the room microservice, for now, since it's quicker. I'll fix it later.
It's unfinished since you can't set photos or commodities yet.
That way, the user can check in real time how much this reservation can cost (provided he can book the reservation in the first place).
This is ugly, but it works.
Since this needs to be runtime, I had to do the `env.js` trick.
Ignore the styling. It should be improved.
Maybe I should just rename `RoomFind` to `Home`.
Fixed an issue where the `Paginator` emitted two events during page number change, despite the page size remaining unchanged. This caused duplicate requests for room data. Now, the `Paginator` only triggers a single event when the page number is updated.
Previously, nights were calculated based on `.getDay()`, which returns only the day of the week.
Vasilijez
approved these changes
Sep 23, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
It's about time we pushed our changes into
master.