Conversation
Graphite Automations"Add consumer team as reviewer" took an action on this PR • (10/15/24)1 reviewer was added to this PR based on Keith Williams's automation. "Add ready-for-e2e label" took an action on this PR • (10/15/24)1 label was added to this PR based on Keith Williams's automation. |
keithwillcode
added
core
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Skipped Deployments
|
| if (shouldListUserEvents || !teamId) { | ||
| filteredEventTypes = filteredEventTypes.filter( | ||
| (evType) => evType.schedulingType !== SchedulingType.MANAGED | ||
| ); | ||
| } |
There was a problem hiding this comment.
moved this to prisma query
E2E results are ready! |
Pull request was converted to draft
| { | ||
| where: { | ||
| teamId: null, | ||
| schedulingType: null, |
There was a problem hiding this comment.
Where will the Managed Event eventtypes show up?
There was a problem hiding this comment.
Managed event type would still show up in the user personal event tab. When you create a managed event type it is created on team level with scheduling type "MANAGED" and on user level with scheduling type null
There was a problem hiding this comment.
We were earlier filtering this after prisma query to prevent displaying two event types
4 participants
What does this PR do?
Mandatory Tasks (DO NOT REMOVE)
How should this be tested?