Skip to content

chore: add default form field support for booker embed routing url#19023

Merged
ThyMinimalDev merged 2 commits intomainfrom
add-default-field-for-routingurl
Jan 31, 2025
Merged

chore: add default form field support for booker embed routing url#19023
ThyMinimalDev merged 2 commits intomainfrom
add-default-field-for-routingurl

Conversation

@ThyMinimalDev
Copy link
Copy Markdown
Contributor

@ThyMinimalDev ThyMinimalDev commented Jan 31, 2025

What does this PR do?

Add default field support to booker embed using routing form url

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - 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.

@keithwillcode keithwillcode added core area: core, team members only platform Anything related to our platform plan labels Jan 31, 2025
@graphite-app graphite-app Bot requested a review from a team January 31, 2025 09:03
@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 31, 2025

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

Name Status Preview Comments Updated (UTC)
cal-com-ui-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 31, 2025 10:17am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jan 31, 2025 10:17am
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Jan 31, 2025 10:17am

@dosubot dosubot Bot added the routing-forms area: routing forms, routing, forms label Jan 31, 2025
@ThyMinimalDev ThyMinimalDev changed the title chore: add default forn field support for booker embed routing url chore: add default form field support for booker embed routing url Jan 31, 2025
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Jan 31, 2025

Graphite Automations

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

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

"Add ready-for-e2e label" took an action on this PR • (01/31/25)

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

Comment thread packages/platform/atoms/booker-embed/useGetRoutingFormUrlProps.tsx
@supalarry supalarry self-requested a review January 31, 2025 09:26
supalarry
supalarry previously approved these changes Jan 31, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 31, 2025

E2E results are ready!

@ThyMinimalDev ThyMinimalDev merged commit 024d47c into main Jan 31, 2025
@ThyMinimalDev ThyMinimalDev deleted the add-default-field-for-routingurl branch January 31, 2025 10:30
MuhammadAimanSulaiman pushed a commit to hit-pay/cal.com that referenced this pull request Feb 25, 2025
…alcom#19023)

* chore: add default forn field support for booker embed routing url

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

Labels

core area: core, team members only platform Anything related to our platform plan ready-for-e2e routing-forms area: routing forms, routing, forms

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants