-
Notifications
You must be signed in to change notification settings - Fork 2.1k
feat: New Question Type Meet Scheduling with Cal.com #1722
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
Conversation
|
@KapadiaNaitik is attempting to deploy a commit to the formbricks Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for following the naming conventions for pull request titles! 🙏 |
|
@pandeymangg Thanks a lot for updating the PR 😊🚀
|
|
hey @KapadiaNaitik Thanks for the PR :) Here is my feedback:
Its a bit confusing with the Cal UI inside of the question card. To make sure that the people actually complete the booking, we change the default button label to "Skip" so it becomes clearer what the button does.
|
|
@jobenjada Anshuman updated the PR; can you please check again 😊 |
|
Hey! Looks great 😍 Last thing: Pls change default from After this, LGTM! Good job @KapadiaNaitik and @pandeymangg |









What does this PR do?
Fixes # (issue)
Type of change
How should this be tested?
Checklist
Required
pnpm buildconsole.logsgit pull origin mainAppreciated