-
Notifications
You must be signed in to change notification settings - Fork 8.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: show pay & book
label to the meeting confirmation button when payment is enabled.
#12960
Conversation
@SomayChauhan is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link. |
📦 Next.js Bundle Analysis for @calcom/webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution @SomayChauhan!
Tested both with a new booking and "Reschedule" still renders when rescheduling a paid booking.
As a follow up, I noticed that when the payment type is charge no-show fee we don't communicate that. It would be cool if we displayed that on the availability page and changed the confirm button text to reflect that. |
What does this PR do?
Fixes #12271
Screencast.from.29-12-23.12.30.27.PM.IST.webm