Skip to content

Fix all day events in week calendar view#1720

Merged
devmount merged 4 commits into
mainfrom
features/1703-all-day-events
Jun 23, 2026
Merged

Fix all day events in week calendar view#1720
devmount merged 4 commits into
mainfrom
features/1703-all-day-events

Conversation

@devmount

Copy link
Copy Markdown
Collaborator

What changed?

This change moves all day events from the calendar grid into a dedicated row on top of the grid.

Why?

The all day events currently span over all rows of a column hiding other events and breaking the UI.

Limitations and Notes

This is meant as a temporary fix until design for this all-day row is ready.

Applicable Issues

Fixes #1702
Fixes #1703

Screenshots

image

@devmount devmount marked this pull request as draft June 18, 2026 14:49
@devmount devmount self-assigned this Jun 18, 2026

@rwood-moz rwood-moz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This doesn't seem to be working for me, here's how I am trying it out:

  • I created a bunch of all-day appointments in my google calendar that is already connected to my local appointment stack (one single all-day appointment, and some repeating all-day appointments too); I see the all-day appointments in my email's calendar outside of Appointment
  • I pulled this branch locally and restarted my local stack with docker compose up --build
  • I sign into my local Appointment, and in the calendar view the all-day appointments are not appearing at the top of the day columns or in the main view either; I do see my other appointments visible
  • Even if I refresh the Appointment calendar view page using the refresh arrow, my all-day appointments still do not appear

Also quick question: If you have an all-day appointment will that remove that day from your availability / will that day appear as unavailable on the bookee / book appt share link page? I'm assuming it should? Thanks.

@devmount

devmount commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator Author

@rwood-moz Thanks for testing!

  • I created a bunch of all-day appointments in my google calendar

Just to be sure: Did you make sure that those events actually have the busy/blocking status?

Also quick question: If you have an all-day appointment will that remove that day from your availability / will that day appear as unavailable on the bookee / book appt share link page? I'm assuming it should? Thanks.

They are supposed to be, yes. But there might be issues with that, I'll take a look at this.

@rwood-moz

rwood-moz commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Thank you @devmount

Just to be sure: Did you make sure that those events actually have the busy/blocking status?

Ahh that was it, no I had the appointments marked as all day but by default they were marked as free time; I changed them to busy and they now appear in my Appointment calendar, nice!

Screenshot 2026-06-23 at 10 48 55 AM

Also quick question: If you have an all-day appointment will that remove that day from your availability / will that day appear as unavailable on the bookee / book appt share link page? I'm assuming it should? Thanks.

They are supposed to be, yes. But there might be issues with that, I'll take a look at this.

On my booking page the all-day appointments are making the time unavailable at least from the testing I did. I tested a single all-day appointment, an all-day appointment that repeats on one day each week, and an all-day appointment that repeats daily.

@devmount

Copy link
Copy Markdown
Collaborator Author

@rwood-moz Nice, thanks so much for re-testing this! So is it working as intended now or are there more issues on your end?

@rwood-moz rwood-moz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@rwood-moz Nice, thanks so much for re-testing this! So is it working as intended now or are there more issues on your end?

Nope LGTM!

@devmount devmount marked this pull request as ready for review June 23, 2026 16:06
@devmount devmount merged commit 717821a into main Jun 23, 2026
8 checks passed
@devmount devmount deleted the features/1703-all-day-events branch June 23, 2026 18:18
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.

All day events should be aggregated at the top row in the calendar UI All day events overflow the Dashboard calendar UI

2 participants