-
Notifications
You must be signed in to change notification settings - Fork 7.8k
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
fix: Tooltip must be used within TooltipProvider #16918
Conversation
@muhammadspace is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
Hey there and thank you for opening this pull request! 👋🏼 We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted. Details:
|
Graphite Automations"Add consumer team as reviewer" took an action on this PR • (10/03/24)1 reviewer was added to this PR based on Keith Williams's automation. "Add community label" took an action on this PR • (10/03/24)1 label was added to this PR based on Keith Williams's automation. "Add ready-for-e2e label" took an action on this PR • (10/03/24)1 label was added to this PR based on Keith Williams's automation. |
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.
Great find @muhammadspace
E2E results are ready! |
Thanks for reporting; will check that. Also, these changes have not been shipped to production yet |
ok, i'll wait for the solution to be publish |
any news on this? |
Yeah, it's was shipped. Could you pls check |
Fix #16833
After installing Cal.com for the first time, users are not able to run go through the setup process on the client due to
PageWrapper
not being applied to/auth/setup
.Mandatory Tasks (DO NOT REMOVE)
How should this be tested?
yarn dev
after installing dependenciesNope
An new DB with no users
No errors when going to localhost:3000/auth/setup