feat: added ability to assign event to all members#13219
feat: added ability to assign event to all members#13219PeerRich merged 24 commits intocalcom:mainfrom
Conversation
|
@Pradumn27 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! 🙌 |
|
@Pradumn27 does this also work with round robin and collective? if not, can you make it work for all team events? round robin, collective, managed events |
@PeerRich yes have added an update to incorporate also for round robin and collective events can be tested in the similar manner as above
|
Udit-takkar
left a comment
There was a problem hiding this comment.
@Pradumn27 I created a collective team event type and invited one more member to the team
I don't see the new user (teampro2@example.com) on the booking page
Udit-takkar
left a comment
There was a problem hiding this comment.
Another thing i don't think we should use this toggle for round robin event types because round robin assignment has two types 'Fixed hosts' and 'Round Robin Hosts'
cc: @PeerRich
|
@Udit-takkar have resolved the issue, it was due to an error in the update team events query i wrote, but everything should be working fine now. |
|
@CarinaWolli have resolved the above issues |
review again
CarinaWolli
left a comment
There was a problem hiding this comment.
Found another issue:
- Enable switch for collective event types, save
- Change scheduling type to round-robin, switch is and stays enabled, save
- Result: Round-robin has no hosts
That's a nice catch, have resolved it now. @CarinaWolli |
Head branch was pushed to by a user without write access
PeerRich
left a comment
There was a problem hiding this comment.
nice job! I wonder if we wanna do something in a v2 of this where you can
- ✅ Everyone
- Exclude: Anna, Brian, etc.
Sounds like a good idea, can be usefull for users, a few questions though:
cc: @PeerRich |



What does this PR do?
Fixes #10991
/claim #10991
Have added the ability to assign all team members a managed event type.
Requirement/Documentation
Design Provided :
Type of change
How should this be tested?
Mandatory Tasks
Checklist