Skip to content

[CORRUPTED] Synthetic Benchmark PR #26812 - fix(api): use cal.com for bookingUrl in platform organizations#550

Open
ofir-frd wants to merge 3 commits intobase_pr_26812_20260120_2755from
corrupted_pr_26812_20260120_2755
Open

[CORRUPTED] Synthetic Benchmark PR #26812 - fix(api): use cal.com for bookingUrl in platform organizations#550
ofir-frd wants to merge 3 commits intobase_pr_26812_20260120_2755from
corrupted_pr_26812_20260120_2755

Conversation

@ofir-frd
Copy link

Benchmark PR calcom#26812

Type: Corrupted (contains bugs)

Original PR Title: fix(api): use cal.com for bookingUrl in platform organizations
Original PR Description: ## What does this PR do?

Fixes the bookingUrl field in API v2 event types response for non-managed users in platform organizations.

Problem

Non-managed users in platform orgs were getting incorrect booking URLs that used the platform org's subdomain:

  • https://gmail-platform-9041df0e-a16d-46b2-a.cal.com/dhairyashil10101010-gmail/secret
  • https://cal.com/dhairyashil/secret

Platform organizations (isPlatform: true) don't have public-facing subdomains - they're internal containers for API developers. Only regular organizations should have subdomain-based booking URLs.

Solution

Updated buildBookingUrl in OutputEventTypesService_2024_06_14 to check if the organization is a platform org before using the slug for the subdomain.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Related Issues

Fixes incorrect booking URLs in companion app for platform org users (follow-up to calcom#26195)
Original PR URL: calcom#26812

Compliance Violation

  • Rule: Environment Variables Must Not Be Accessed Directly in Non-Configuration Code
  • Language: TypeScript
  • File: apps/api/v2/src/ee/event-types/event-types_2024_06_14/services/output-event-types.service.ts

dhairyashiil and others added 3 commits January 14, 2026 02:20
Platform organizations don't have public-facing subdomains, so non-managed
users in platform orgs should get cal.com URLs instead of the platform
org subdomain.

- Updated EventTypeUser type to include isPlatform field
- Modified buildBookingUrl to check isPlatform before using org slug
- Added unit test for platform org users
@github-actions
Copy link

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

No release type found in pull request title "[CORRUPTED] Synthetic Benchmark PR #26812 - fix(api): use cal.com for bookingUrl in platform organizations". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

@github-actions
Copy link

This PR has been marked as stale due to inactivity. If you're still working on it or need any help, please let us know or update the PR to keep it active.

@github-actions github-actions bot added the Stale label Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants