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

Add day of week to room booking details / timeline view #5718

Closed
olifre opened this issue Mar 30, 2023 · 1 comment · Fixed by #5743
Closed

Add day of week to room booking details / timeline view #5718

olifre opened this issue Mar 30, 2023 · 1 comment · Fixed by #5743
Assignees

Comments

@olifre
Copy link
Contributor

olifre commented Mar 30, 2023

Is your feature request related to a problem? Please describe.
When booking a room (and checking the details at the end) or looking at booking details afterwards, an additional calender is needed to find out the day of week. Example:
weekly-bookings
This is a weekly booking, all 15 occurrences are on Fridays.

The day of week would be nice to have both for the booking details view and the timeline view.

Describe the solution you'd like
It would be nice if the day of week would be shown in addition. This would ease canceling single dates from the timeline view (and finding the correct date) and also help to find out whether the correct day of week was chosen for a weekly booking.

Describe alternatives you've considered
Using a separate calender in addition, but this is of course extra hassle for the users.

Additional context
If I understand the code correctly, currently L is used as format string for the moment data type. I'm not deep into javascript, but it might be sufficient to add the localized, short day of week (i.e. dddd) to the format string?

@olifre
Copy link
Contributor Author

olifre commented Apr 26, 2023

Thanks a lot to you, @ThiefMaster and also @GovernmentPlates ! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants