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: lock event types for org users #14000
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Ignored Deployments
|
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link. |
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.
@@ -810,6 +813,7 @@ const CTA = ({ data }: { data: GetByViewerResponse }) => { | |||
|
|||
const profileOptions = data.profiles |
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 need to disable the CreateButton
if we don't have any profileOptions. Now I am getting an error when clicking on the 'New' button Uncaught TypeError: Cannot read properties of undefined (reading 'slug')
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.
Never even thought of this - nice spot
…es-orgs' into feat/lock-event-types-orgs
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.
Looking good now, nice work 👏🏻
@sean-brydon can you fix the merge conflicts? |
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.
re-approve after fixing merge conflicts
fixes: CAL-3152 #13716
Adds the ability for org customers to stop users creating events on their own
acme.cal.com/[user]
profile and only rely on managed event typesLoom: https://www.loom.com/share/9dd0f8704d7e42a0891f50deffb9bf17