Skip to content

Conversation

@OmkarKovvali
Copy link

What does this PR do?

This PR updates the booking success redirect logic to include custom question responses in the URL parameters. It adds support for multi-select answers to be forwarded as multiple URL parameters. Example is ?option=A&option=B instead of a single comma-separated string, while maintaining backward compatibility for existing array fields like hostname.

Before this PR, when "forward parameters" was enabled, only standard fields such as name email and date were included. Custom questions were ignored. Additionally, multi-select answers were flattened into a single string, making them hard to parse by external tooling.

Visual Demo (For contributors especially)

Screenshot 2025-12-23 at 2 57 09 PM Screenshot 2025-12-23 at 2 59 38 PM

Mandatory Tasks (DO NOT REMOVE)

  • [X ] I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • [X ] 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.
  • [ X] I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  1. Standard Booking Redirect: Book a regular meeting with no custom fields and verify the redirect url contains standard params like hostName and date like before.

  2. Custom text question: create an event type with a custom question like "Company Name" and book it and answer "Acme". Verify the redirect url contains &company_name=Acme.

  3. Multi-select custom question: Create an event type with Checkboxes question e.g. "Dietary Requirements" and book it, selecting "Vegan" AND "Gluten-free". Verify the redirect URL contains &dietary=Vegan&dietary=Gluten%20Free. Verify that its not just &dietary=Vegan,Gluten Free.

@OmkarKovvali OmkarKovvali requested a review from a team as a code owner December 23, 2025 20:03
@vercel
Copy link

vercel bot commented Dec 23, 2025

@OmkarKovvali is attempting to deploy a commit to the cal-staging Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Dec 23, 2025

CLA assistant check
All committers have signed the CLA.

@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Dec 23, 2025
@github-actions github-actions bot added the ✨ feature New feature or request label Dec 23, 2025
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 1 file

@OmkarKovvali
Copy link
Author

OmkarKovvali commented Dec 27, 2025

@dhairyashiil @keithwillcode @CarinaWolli Please lmk if there's anything else to add :)

@OmkarKovvali
Copy link
Author

Hi Team, looking for feedback to merge this PR. Please lmk what the next steps are.

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 ✨ feature New feature or request size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ability for custom question field parameter forwarding for booking success redirect URL

2 participants