Skip to content

perf(api-v2): optimize routing form event lookup by ID#27947

Open
deepshekhardas wants to merge 1 commit intocalcom:mainfrom
deepshekhardas:perf/routing-form-event-lookup
Open

perf(api-v2): optimize routing form event lookup by ID#27947
deepshekhardas wants to merge 1 commit intocalcom:mainfrom
deepshekhardas:perf/routing-form-event-lookup

Conversation

@deepshekhardas
Copy link

Optimizes the routing form response handling by prioritizing direct eventTypeId lookup from the URL parameters (cal.eventTypeId) instead of relying solely on saving/parsing slugs.

Changes

  • Modified extractEventTypeAndCrmParams in SharedRoutingFormResponseService:
    • Added logic to checks for cal.eventTypeId in the URL.
    • If present, fetches the event type directly by ID.
    • Falls back to existing slug-based lookup if ID is missing.
  • Refactored CRM parameter extraction into a helper method extractCrmParamsFromUrl to reduce code duplication.

Fixes TODO in shared-routing-form-response.service.ts.

@deepshekhardas deepshekhardas requested a review from a team as a code owner February 14, 2026 10:37
@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Feb 14, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Created by Linear-GitHub Sync size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant