fix: refresh slots on timezone change for booker timezone restrictions by hackice20 · Pull Request #26026 · calcom/cal.com

hackice20 · 2025-12-18T13:17:41Z

What does this PR do?

Refreshes available slots when booker changes timezone on events with restriction schedules that use booker's timezone.
Previously slots weren't refreshing because conversion was client-side only. Now we detect timezone changes and refetch slots when needed.
Only refreshes when all conditions are met:
Timezone changed
Event has restrictionScheduleId
Event has useBookerTimezone: true
No unnecessary API calls for regular events.

Fixes Upon changing timezone, we need to update slots when restrictionSchedule is being used with useBookerTimezone enabled #22319

Visual Demo (For contributors especially)

Video Demo (if applicable):

perfect-video.mp4

Mandatory Tasks (DO NOT REMOVE)

I have self-reviewed the code (A decent size PR without self-review might be rejected).
I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

Create event type with a restriction schedule and enable "Use booker's timezone"
Open booking page in incognito
Open DevTools → Network tab, filter by slots
Change timezone in the booker
Expected: New getSchedule API call fires, slots refresh

Checklist

I have read the contributing guide
My code does follow the style guidelines of this project
I have commented my code, particularly in hard-to-understand areas
I have checked if my changes generate no new warnings

vercel · 2025-12-18T13:17:48Z

@hackice20 is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

CLAassistant · 2025-12-18T13:17:52Z

All committers have signed the CLA.

cubic-dev-ai

No issues found across 5 files

hackice20 · 2025-12-18T13:22:49Z

Hello @dhairyashiil can you please review my PR , thanking you in anticipation :)

packages/features/bookings/Booker/components/hooks/useTimezoneChangeDetection.ts

hackice20 · 2025-12-19T16:40:32Z

Hey @Ryukemeister ,Thanks for the suggestion! Refactored to use useMemo and simplified the logic as you suggested. All tests pass and the hook is now more efficient. Lmk what you think

hackice20 · 2025-12-30T13:44:02Z

@alishaz-polymath

alishaz-polymath · 2025-12-30T13:45:35Z

Hi @hackice20
I will review the PR this week. Thank you for your efforts 🙌

hackice20 · 2025-12-30T13:46:31Z

Hi @hackice20 I will review the PR this week. Thank you for your efforts 🙌

thanks

Nandinibajaj28 · 2026-01-01T11:35:31Z

I'm working on this

Ryukemeister

hi @hackice20, I saw another PR being created that was being referenced as a fix for this PR. i'm a bit confused, do you know what that is about? is it an edge case that we might be missing here or is it something different altogether? can you please work with @MaheshX18 to get this resolved, thank you!

hackice20 · 2026-01-04T11:24:26Z

hi @hackice20, I saw another PR being created that was being referenced as a fix for this PR. i'm a bit confused, do you know what that is about? is it an edge case that we might be missing here or is it something different altogether? can you please work with @MaheshX18 to get this resolved, thank you!

I'll read the code understand what this is about and get this resolved. NO Worries.

hackice20 · 2026-01-04T12:31:04Z

@Ryukemeister Mahesh's PR works on query level and has complete coverage of booker, platform, embed, etc.
To improve this PR I can either call useTimezoneChangeDetection in BookerWebWrapper.tsx , Embed.tsx , BookerPlatformWrapper.tsx , EventTypeCalendarViewComponent.tsx or I can adopt the query level calling.

alishaz-polymath · 2026-01-18T04:47:36Z

@hackice20 Type checks still failing

alishaz-polymath · 2026-01-19T12:32:51Z

Types are still failing @hackice20
Do you mind if I take it over and get it over the line?

alishaz-polymath · 2026-01-22T08:50:57Z

Hey @hackice20 I haven't been able to spare time to look at this yet, but I wanted to let you know this isn't forgotten. I just need to find time to look into this further 🙏

hackice20 · 2026-01-29T13:41:16Z

can we please check run ci one more time please @alishaz-polymath

hackice20 · 2026-01-31T08:21:20Z

Hello @alishaz-polymath sorry to bother you again and again, was looking if

 // Dynamic group events don't have restriction schedules configured
      restrictionScheduleId: null,
      useBookerTimezone: false,

in the getPublicEvent.ts file will this work ?

alishaz-polymath · 2026-02-02T07:49:45Z

Tests still failing, I'm taking it forward here: #27491

Thank you for your PR and efforts 🙏

fix: refresh slots on timezone change for booker timezone restrictions

7f4acb7

pull-request-size bot added the size/L label Dec 18, 2025

graphite-app bot added the community Created by Linear-GitHub Sync label Dec 18, 2025

github-actions bot added High priority Created by Linear-GitHub Sync ⚡ Quick Wins A collection of quick wins/quick fixes that are less than 30 minutes of work ✅ good first issue Good for newcomers 🐛 bug Something isn't working labels Dec 18, 2025

cubic-dev-ai bot reviewed Dec 18, 2025

View reviewed changes

Ryukemeister reviewed Dec 19, 2025

View reviewed changes

packages/features/bookings/Booker/components/hooks/useTimezoneChangeDetection.ts Outdated Show resolved Hide resolved

hackice20 added 2 commits December 19, 2025 17:18

refactor: use useMemo for timezone change detection

3c8fd71

revert: remove unnecessary formatting changes

a0e8196

hackice20 requested a review from Ryukemeister December 20, 2025 13:50

hackice20 added 2 commits December 22, 2025 15:50

Merge branch 'main' into fix/slots-refresh-on-timezone-change

2e097f1

Merge branch 'main' into fix/slots-refresh-on-timezone-change

862633f

dhairyashiil requested a review from alishaz-polymath December 23, 2025 13:27

Merge branch 'main' into fix/slots-refresh-on-timezone-change

3e5535d

Merge branch 'main' into fix/slots-refresh-on-timezone-change

0abc727

Merge branch 'main' into fix/slots-refresh-on-timezone-change

6fff545

MaheshX18 mentioned this pull request Jan 3, 2026

fix(booker): avoid unnecessary slot refetch on timezone change (#26026) #26423

Open

Ryukemeister reviewed Jan 4, 2026

View reviewed changes

hackice20 and others added 3 commits January 18, 2026 06:16

fix: add missing properties to BookerEvent and BookerEventProfile types

4229466

Merge branch 'main' into fix/slots-refresh-on-timezone-change

242c297

Merge branch 'main' into fix/slots-refresh-on-timezone-change

29125f7

alishaz-polymath added run-ci Approve CI to run for external contributors and removed run-ci Approve CI to run for external contributors labels Jan 19, 2026

alishaz-polymath and others added 2 commits January 19, 2026 12:15

Merge branch 'main' into fix/slots-refresh-on-timezone-change

1d9b642

Merge branch 'main' into fix/slots-refresh-on-timezone-change

ca9e21a

alishaz-polymath added run-ci Approve CI to run for external contributors and removed run-ci Approve CI to run for external contributors labels Jan 19, 2026

Merge branch 'main' into fix/slots-refresh-on-timezone-change

db2220d

Ryukemeister mentioned this pull request Jan 20, 2026

fix: refresh slots when timezone changes with restriction schedule #26977

Closed

6 tasks

hackice20 and others added 2 commits January 24, 2026 10:56

Merge branch 'main' into fix/slots-refresh-on-timezone-change

3f7d625

trying to fix type errors

1dfd71f

pull-request-size bot added size/XL and removed size/L labels Jan 24, 2026

Merge branch 'main' into fix/slots-refresh-on-timezone-change

4046f4e

alishaz-polymath added run-ci Approve CI to run for external contributors and removed run-ci Approve CI to run for external contributors labels Jan 30, 2026

hackice20 added 2 commits January 31, 2026 09:27

Add restrictionScheduleId and useBookerTimezone fields

f4b42be

Merge branch 'main' into fix/slots-refresh-on-timezone-change

95679dc

alishaz-polymath added run-ci Approve CI to run for external contributors and removed run-ci Approve CI to run for external contributors labels Feb 2, 2026

alishaz-polymath closed this Feb 2, 2026

alishaz-polymath mentioned this pull request Feb 2, 2026

fix: refresh slots on timezone change for booker timezone restrictions #27491

Merged

9 tasks

Conversation

hackice20 commented Dec 18, 2025

What does this PR do?

Visual Demo (For contributors especially)

Video Demo (if applicable):

Mandatory Tasks (DO NOT REMOVE)

How should this be tested?

Checklist

Uh oh!

vercel bot commented Dec 18, 2025

Uh oh!

CLAassistant commented Dec 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

Uh oh!

hackice20 commented Dec 18, 2025

Uh oh!

Uh oh!

hackice20 commented Dec 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hackice20 commented Dec 30, 2025

Uh oh!

alishaz-polymath commented Dec 30, 2025

Uh oh!

hackice20 commented Dec 30, 2025

Uh oh!

Nandinibajaj28 commented Jan 1, 2026

Uh oh!

Ryukemeister left a comment

Choose a reason for hiding this comment

Uh oh!

hackice20 commented Jan 4, 2026

Uh oh!

hackice20 commented Jan 4, 2026

Uh oh!

alishaz-polymath commented Jan 18, 2026

Uh oh!

alishaz-polymath commented Jan 19, 2026

Uh oh!

alishaz-polymath commented Jan 22, 2026

Uh oh!

hackice20 commented Jan 29, 2026

Uh oh!

hackice20 commented Jan 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

alishaz-polymath commented Feb 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

CLAassistant commented Dec 18, 2025 •

edited

Loading

hackice20 commented Dec 19, 2025 •

edited

Loading

hackice20 commented Jan 31, 2026 •

edited

Loading