-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
Calendly & SavvyCal import #1512
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/cal/calendso/8uovf1nMMobzCJwSopweAxuDDYS7 |
very nice! can you add some description how to test this properly? maybe set up some test accounts too as I believe there is no way to retrigger the onboarding. |
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.
Code looks good, some screenshots would be nice tho :)
It looks good to me 🎉 |
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.
We have to change to strict comparison but the rest is perfect
Co-authored-by: Omar López <zomars@me.com>
Co-authored-by: Omar López <zomars@me.com>
Co-authored-by: Omar López <zomars@me.com>
This adds two buttons to the onboarding flow where you can choose which platform you'd like to import from, and runs two API queries to either API to get the user data and update the user with it, and the other API query gets your event types and creates them on our end.
Our goal is to make scheduling as open and accessible as possible and this helps customers make the right choice by not feeling locked in by other products.