Skip to content
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: custom templates on team plan #18792

Merged
merged 2 commits into from
Jan 21, 2025
Merged

fix: custom templates on team plan #18792

merged 2 commits into from
Jan 21, 2025

Conversation

CarinaWolli
Copy link
Member

What does this PR do?

Custom templates should be available also for user workflows if the user is a member of a active team plan.

@keithwillcode keithwillcode added consumer core area: core, team members only labels Jan 21, 2025
@CarinaWolli CarinaWolli marked this pull request as ready for review January 21, 2025 19:21
@graphite-app graphite-app bot requested a review from a team January 21, 2025 19:21
@dosubot dosubot bot added teams area: teams, round robin, collective, managed event-types workflows area: workflows, automations labels Jan 21, 2025
Copy link

vercel bot commented Jan 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jan 21, 2025 7:22pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Jan 21, 2025 7:22pm

@CarinaWolli CarinaWolli added High priority Created by Linear-GitHub Sync and removed teams area: teams, round robin, collective, managed event-types workflows area: workflows, automations labels Jan 21, 2025
@CarinaWolli CarinaWolli added this to the v5.0 milestone Jan 21, 2025
Copy link

graphite-app bot commented Jan 21, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (01/21/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add ready-for-e2e label" took an action on this PR • (01/21/25)

1 label was added to this PR based on Keith Williams's automation.

if (!teams.length) return false;

// check if user has at least on membership with an active plan
for (const team of teams) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thoughts about wrapping this in a Promise.all?

@zomars zomars enabled auto-merge (squash) January 21, 2025 19:49
@zomars zomars merged commit f523ce7 into main Jan 21, 2025
60 of 61 checks passed
@zomars zomars deleted the fix/custom-templates branch January 21, 2025 20:16
Copy link
Contributor

E2E results are ready!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consumer core area: core, team members only High priority Created by Linear-GitHub Sync ready-for-e2e
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants