Skip to content

Conversation

@gcooper407
Copy link
Contributor

@gcooper407 gcooper407 commented Oct 6, 2025

Changes

When users RSVP to a meeting (i.e. provide their availability for a design review meeting), as soon as the meeting time is selected by the host, all attendee's availability calendars get updated to reflect the meeting, i.e. the time block of the meeting shows as no longer available.

Notes

It seems that availability dates stored in the db are typically off by ~1 day, so as a temporary measure until this behavior is fixed, I offset the date/time by 1 day to account for the error, but once resolved, that logic can simply be removed (I left in the code that should work as a comment).

I'm putting up this PR now just to have progress and so I can move onto other ticket, but might want to wait on merging this until that date bug gets resolved.

Screenshots

Here's a walkthrough of the full process, just so it's clear when the availability gets adjusted and how it appears to the user:

demo.mp4

Checklist

It can be helpful to check the Checks and Files changed tabs.
Please review the contributor guide and reach out to your Tech Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.

  • All commits are tagged with the ticket number
  • No linting errors / newline at end of file warnings
  • All code follows repository-configured prettier formatting
  • No merge conflicts
  • All checks passing
  • Screenshots of UI changes (see Screenshots section)
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • No yarn.lock changes (unless dependencies have changed)
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (fill in the closes line below)

Closes #2743

@gcooper407 gcooper407 force-pushed the #2743-design-review-calendar-RSVP-availability branch from c1d4441 to d3d049a Compare October 6, 2025 04:49
Copy link
Contributor

@superhvarn superhvarn left a comment

Choose a reason for hiding this comment

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

LGTM

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.

[Design Reviews] - User Calendar Availability Should Be Unavailable When RSVP'd to Meeting at that Time

3 participants