* Return a busy block placeholder if calendar throws an error
* Refactor `getCalendarsEvents` to return an object with a success prop
* Throw error in `getBusyTimes` if failed to fetch calendar availability
* Return empty available days if error getting busy times
* yeet.
* Type fix
* Fix type error in getLuckyUsers
* Type fixes
* Type fix
* Type fix
* Fix test
* Fix test mocks
* Refactor calendars.service to use new calendarBusyTimesQuery
---------
Co-authored-by: Alex van Andel <me@alexvanandel.com>
* refactor: convert checkBookingLimits to class service with dependency injection
- Create CheckBookingLimitsService class following AvailableSlotsService pattern
- Add countBookingsByEventTypeAndDateRange method to BookingRepository
- Move direct prisma calls from service to repository layer
- Implement dependency injection with proper DI tokens and modules
- Update all usage points to use the new service through DI
- Maintain backward compatibility with error-throwing wrapper functions
- Update tests to use the new service pattern
- Resolve TODO comment in AvailableSlotsService for DI integration
Co-Authored-By: morgan@cal.com <morgan@cal.com>
* chore: DI CheckBookingLimitsService in v2 slots service
* chore: bump libraries
* chore: create getCheckBookingLimitsService
* refactor: convert checkBookingAndDurationLimits to service class with DI
- Create CheckBookingAndDurationLimitsService class following DI pattern
- Add DI tokens and module for the new service
- Update booking-limits container to provide the new service
- Refactor handleNewBooking.ts to use service through DI
- Maintain backward compatibility with deprecated function export
- Preserve all existing functionality while improving code organization
Co-Authored-By: morgan@cal.com <morgan@cal.com>
* chore: CheckBookingAndDurationLimitsService
* chore: bump platform libs
---------
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: morgan@cal.com <morgan@cal.com>
Co-authored-by: Morgan <33722304+ThyMinimalDev@users.noreply.github.com>
* Automatically enable migration of a team that conflicts with Orgs slug
* Allow changing the orgs slug and name if user goes back and changes it
* avoid crashing on some scenarios
* Revert "Allow changing the orgs slug and name if user goes back and changes it"
This reverts commit f8872b0116868d59757ef911c4940df79f93c6e0.
* fix: handle unpublished teams gracefully in org migration
- Change 'No oldSlug for team' from error to warning for unpublished teams
- Keep 'No slug for team' as error since org onboarding ensures teams have names
- Add test for migrating unpublished teams without oldSlug
- Add clarifying comments about when each condition can occur
* remove first weeks and add last
* fix added last week
* prefetch availability of next month
* don't switch month
* On hover show month
* only show new UI in monthly view
* show month tooldtip only when needed
* show month on first day of month
* remove isFirstDayOfNextMonth
* fix prefetching next month
* fix datePicker tests
* preventMonthSwitching in monthly view
* add tests
* code clean up
* code clean up
* code clena up for ooo days
* push first day of month
* remove bg color for the month badge
* fix text colour
* remove not needed
* use object param
* revert: use object param
* use object param
* fix DatePicker tests
---------
Co-authored-by: CarinaWolli <wollencarina@gmail.com>
Co-authored-by: Sean Brydon <sean@cal.com>
Co-authored-by: Eunjae Lee <hey@eunjae.dev>
* fix: Adjacency issue when working hours connect over multiple days
* Add tests to validate the new merging of day end logic
* Update to correct annotation.
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
* Implement subsequent date ranges for date overrides also
* The map needs to be updated on successful resolve.
* test: add failing test for overlapping ranges with same end time
Demonstrates bug where overlapping working hour ranges (6:00-10:00 and 8:00-10:00)
lose the earlier portion (6:00-8:00), showing only 8:00 and 9:00 slots
instead of all 4 slots (6:00, 7:00, 8:00, 9:00).
Related to Carina's comment on PR #21912.
Co-Authored-By: alex@cal.com <me@alexvanandel.com>
* fix: properly merge overlapping ranges with same end time
Fixes bug where overlapping working hour ranges with the same end time
(e.g., 6:00-10:00 and 8:00-10:00) would lose the earlier portion of the
first range. The merging logic now correctly preserves the earliest
start time when ranges overlap and share the same end time.
This ensures all expected time slots are available (6:00, 7:00, 8:00, 9:00)
instead of losing the earlier slots (6:00, 7:00).
Resolves the issue identified in Carina's comment on PR #21912.
Co-Authored-By: alex@cal.com <me@alexvanandel.com>
* perf: optimize overlapping range detection from O(n²) to O(n)
Replaces Object.keys().find() with Map-based lookup for ranges with same end time.
This optimization handles 2000+ date ranges efficiently, reducing complexity from
4M operations to linear time while maintaining the same merging behavior.
Performance improvement for high-volume event types with many availability ranges.
Co-Authored-By: alex@cal.com <me@alexvanandel.com>
---------
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
* only make api call once access token is verified
* add date fns tz
* use date-fns to format ranges for platform
* update yarn.lock
* fix merge conflicts
* fix updating overrides for atoms
* add changesets
* feat: rename repository files to include Repository suffix
- Rename apiKey.ts to apiKeyRepository.ts
- Rename assignmentReason.ts to assignmentReasonRepository.ts
- Update all import statements to use new file names
Co-Authored-By: eunjae@cal.com <hey@eunjae.dev>
* feat: rename repository files to PrismaRepository with Prisma prefix
- Rename apiKeyRepository.ts to PrismaApiKeyRepository.ts
- Rename assignmentReasonRepository.ts to PrismaAssignmentReasonRepository.ts
- Update class names to PrismaApiKeyRepository and PrismaAssignmentReasonRepository
- Update all import statements and usages across 3 files
Co-Authored-By: eunjae@cal.com <hey@eunjae.dev>
---------
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
* feat: add race condition test with dynamic date selection
- Recreate race condition test from PR #22401 for calendar-cache-serve booking flow
- Replace hardcoded TEST_DATE with dynamic first available slot selection
- Use getDynamicBookingDate function to select first available day dynamically
- Prevent test flakiness by avoiding hardcoded tomorrow dates
- Test validates concurrent booking prevention and cache functionality
Co-Authored-By: anik@cal.com <adhabal2002@gmail.com>
* fix: replace dynamic date selection with fixed future date for efficiency
Co-Authored-By: anik@cal.com <adhabal2002@gmail.com>
* Revert "fix: replace dynamic date selection with fixed future date for efficiency"
This reverts commit b4b7d358de719ec8fb495352180eea774a5d76c1.
---------
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
* Add userAvailability type
* Add user availability to return value of getUsersAvailability
* Log availabilitySnapshot when booking is created
* Type fix
* app initialization
* btcpay-calcom payment
* include logo and images
* resolve comments
* include USD and webhook cleaning
* currency display
* fix type error
* payment service create error
* type error fix
* icon update
* bot feedback update
* Remove console
* Remove currency suffix in price
* fix coderRabbit comment
* resolve extra comments
* Use repositories and declarative installation ocode for app
* use PrismaBookingPaymentRepository as well as fix UI view
* Avoid fetching booking just for title which is already passed to create fn in handlePayment
* fix type issues
* return 200 if payment is already processed
---------
Co-authored-by: Anik Dhabal Babu <81948346+anikdhabal@users.noreply.github.com>
Co-authored-by: Hariom Balhara <hariombalhara@gmail.com>
Co-authored-by: Omar López <zomars@me.com>
* feat: add cancelledBy and rescheduledBy parameters to workflow links
- Extended workflow reminder selection to include user and team data
- Added getWorkflowAssigneeEmail helper function to resolve assignee email
- Updated email and SMS workflow templates to include assignee in cancel/reschedule links
- For user workflows, uses user email; for team workflows, uses team owner email
- Maintains backward compatibility when assignee data is unavailable
Co-Authored-By: joe@cal.com <j.auyeung419@gmail.com>
* fix: use workflow recipient email instead of assignee email for link parameters
- Updated getWorkflowRecipientEmail to determine recipient based on workflow action type
- EMAIL_ADDRESS uses sendTo email, EMAIL_HOST uses organizer, EMAIL_ATTENDEE uses attendee
- SMS_ATTENDEE/WHATSAPP_ATTENDEE use attendee email for cancelledBy/rescheduledBy params
- Maintains backward compatibility when recipient data unavailable
Co-Authored-By: joe@cal.com <j.auyeung419@gmail.com>
* fix: use emailReminderManager for email workflows with recipient parameters
- Updated emailReminderManager to include recipient email in cancel/reschedule links
- Added getEmailWorkflowRecipientEmail function for proper recipient determination
- Removed changes from deprecated scheduleEmailReminders.ts
- SMS_ATTENDEE correctly uses attendee email like EMAIL_ATTENDEE
Co-Authored-By: joe@cal.com <j.auyeung419@gmail.com>
* Get attendee's email when sending SMS
* Remove unused query
* feat: add cancelledBy and rescheduledBy parameters to getSMSMessageWithVariables
Co-Authored-By: joe@cal.com <j.auyeung419@gmail.com>
* Clean up Devin
---------
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
* Middleware only logic to add CSP header as Next.js App Router doesnt allow setting header in server component
* add tests
* Remove nonce from Page Router and fix ts issues
* Add checkly test
* add fallback to service
* fix: org memebership fallback
* remove log
* remove log
* add merge test case
* add test for no parentId on team for getResources
* refactor: rename SelectedSlotsRepository to PrismaSelectedSlotRepository
- Rename class from SelectedSlotsRepository to PrismaSelectedSlotRepository for consistency
- Update all imports and type references throughout the codebase
- Update DI module bindings and variable names
- Maintain type safety without breaking changes
Co-Authored-By: alex@cal.com <me@alexvanandel.com>
* refactor: complete PrismaSelectedSlotRepository rename
- Update DI module binding property name from selectedSlotsRepo to selectedSlotRepo
- Update test mock to use PrismaSelectedSlotRepository class name
- Ensure all references are consistently updated
Co-Authored-By: alex@cal.com <me@alexvanandel.com>
* fix: rename file to prismaSelectedSlotRepository.ts and fix plural variable references
- Rename apps/api/v2/src/lib/repositories/prisma-selected-slots.repository.ts to prismaSelectedSlotRepository.ts
- Update class name from PrismaSelectedSlotsRepository to PrismaSelectedSlotRepository
- Fix plural variable reference selectedSlotsRepository to selectedSlotRepository in service
- Update all import paths to reference the new file name
- Maintain consistency with camelCase naming convention
Co-Authored-By: alex@cal.com <me@alexvanandel.com>
* fix: rename SelectedSlotsRepositoryFixture to SelectedSlotRepositoryFixture for consistency
- Rename test fixture class from SelectedSlotsRepositoryFixture to SelectedSlotRepositoryFixture (singular)
- Update all import statements in test files to use the renamed class
- Resolves naming mismatch between fixture class and variable declarations
- Fixes TypeScript compilation errors in CI tests
Co-Authored-By: alex@cal.com <me@alexvanandel.com>
* feat: add SelectedSlotRepositoryInterface and update DI to use interface
Co-Authored-By: alex@cal.com <me@alexvanandel.com>
* refactor: rename selectedSlots.ts to selectedSlot.ts and update all imports
Co-Authored-By: alex@cal.com <me@alexvanandel.com>
* fix: update test mock import path after file rename
Co-Authored-By: alex@cal.com <me@alexvanandel.com>
* Update apps/api/v2/src/lib/modules/available-slots.module.ts
* Update apps/api/v2/src/lib/modules/available-slots.module.ts
* Implement DTO
* dont declare dependencies locally, duplicating
* Small DTO/token fix
* chore: bump @calcom/platform-libraries from 0.0.266 to 0.0.267
* oops.
* Update fixture names also
* Omg these vscode actions preventing saves
* Final fix, hopefully
---------
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: Somay Chauhan <somaychauhan98@gmail.com>
* refactor: event trends chart on /insights to use InsightsBookingService
* extract service instantiation
* remove unused code
* rename from eventsTimeline to eventTrends
* clean up
---------
Co-authored-by: Anik Dhabal Babu <81948346+anikdhabal@users.noreply.github.com>
emrysal