feat: skip confirm step in booker

[SomayChauhan]

What does this PR do?

• Fixes [CAL-4943] If all information for "confirm" step is provided skip step #18307 (GitHub issue number)
• Fixes CAL-4943 (Linear issue number - should be visible at the bottom of the GitHub issue description)

with Turnstile disabled

https://www.loom.com/share/afbfc32af32f4fad94c1e9f80c832827?sid=4b5aee5a-5019-4eaf-bda8-5e9ca17be7e3

with Turnstile enabled

https://www.loom.com/share/ad9fb69d1d544bdaa1923e44f86a733f?sid=b18bd4a0-e53a-43e3-a82a-cb3175d67c20

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?

• Are there environment variables that should be set?
• What are the minimal test data to have?
• What is expected (happy path) to have (input and output)?
• Any other important info that could help to test that PR

Checklist

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

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
cal	⬜️ Ignored (Inspect)	Visit Preview		Jan 28, 2025 4:42am
calcom-web-canary	⬜️ Ignored (Inspect)	Visit Preview		Jan 28, 2025 4:42am

[graphite-app]

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (01/23/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add platform team as reviewer" took an action on this PR • (01/23/25)

1 reviewer was added to this PR based on Keith Williams's automation.

[Udit-takkar]

we should simplify this

[Udit-takkar]

type checks are failing

[linear]

CAL-4943 If all information for "confirm" step is provided skip step

[ThyMinimalDev]

looking good, we might want to add en e2e test with this flow using the iframe embed in the future

[sean-brydon]

Looks good to me from my testing. @eunjae-lee is taking a look and if he doesnt see any issues happy to change this to an approval

[eunjae-lee]

I've tested it and it looks good to me too.
@sean-brydon i didn't approve it yet, in case we're still waiting for foundation team's review.

(Nevermind. I misread your message about another PR.)

[sentry-io]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ HttpError: Cannot read properties of null (reading 'trim') POST /api/bookings View Issue

_{Did you find this useful? React with a 👍 or 👎}