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: unlock limit tab for managed event types #11936

Merged
merged 2 commits into from
Oct 17, 2023

Conversation

CarinaWolli
Copy link
Member

What does this PR do?

We want to have all fields in the limit tab unlocked for managed event types. In v2 admins will be able to define themselves which fields should be locked and unlocked.

Fixes #11882

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • Create managed event type and assign to users
  • Test if users can change the settings in the limit tab of their managed event type

@vercel
Copy link

vercel bot commented Oct 17, 2023

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

Name Status Preview Comments Updated (UTC)
ai ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2023 9:32am
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2023 9:32am
cal-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2023 9:32am
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2023 9:32am
qa 🔄 Building (Inspect) Visit Preview 💬 Add feedback Oct 17, 2023 9:32am
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2023 9:32am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Oct 17, 2023 9:32am

@github-actions github-actions bot added event types Created by Linear-GitHub Sync High priority Created by Linear-GitHub Sync teams area: teams, round robin, collective, managed event-types ✨ feature New feature or request labels Oct 17, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 17, 2023

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link to collect XP and win prizes!

@deploysentinel
Copy link

deploysentinel bot commented Oct 17, 2023

Current Playwright Test Results Summary

✅ 153 Passing - ⚠️ 17 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 10/17/2023 09:36:41am UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: f5832ee

Started: 10/17/2023 09:33:33am UTC

⚠️ Flakes

📄   apps/web/playwright/locale.e2e.ts • 13 Flakes

Top 1 Common Error Messages

null

13 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
unauthorized user sees correct translations (de) should use correct translations and html attributes
Retry 1Initial Attempt
-125.61% (-103) -103 / 82 runs
failed over last 7 days
125.61% (103) 103 / 82 runs
flaked over last 7 days
unauthorized user sees correct translations (ar) should use correct translations and html attributes
Retry 1Initial Attempt
-125.61% (-103) -103 / 82 runs
failed over last 7 days
125.61% (103) 103 / 82 runs
flaked over last 7 days
unauthorized user sees correct translations (zh) should use correct translations and html attributes
Retry 1Initial Attempt
-26.32% (-5) -5 / 19 runs
failed over last 7 days
31.58% (6) 6 / 19 runs
flaked over last 7 days
unauthorized user sees correct translations (zh-CN) should use correct translations and html attributes
Retry 1Initial Attempt
-33.33% (-6) -6 / 18 runs
failed over last 7 days
33.33% (6) 6 / 18 runs
flaked over last 7 days
unauthorized user sees correct translations (zh-TW) should use correct translations and html attributes
Retry 1Initial Attempt
-33.33% (-6) -6 / 18 runs
failed over last 7 days
33.33% (6) 6 / 18 runs
flaked over last 7 days
unauthorized user sees correct translations (pt) should use correct translations and html attributes
Retry 1Initial Attempt
-33.33% (-6) -6 / 18 runs
failed over last 7 days
33.33% (6) 6 / 18 runs
flaked over last 7 days
unauthorized user sees correct translations (pt-br) should use correct translations and html attributes
Retry 1Initial Attempt
-33.33% (-6) -6 / 18 runs
failed over last 7 days
33.33% (6) 6 / 18 runs
flaked over last 7 days
unauthorized user sees correct translations (es-419) should use correct translations and html attributes
Retry 1Initial Attempt
-33.33% (-6) -6 / 18 runs
failed over last 7 days
33.33% (6) 6 / 18 runs
flaked over last 7 days
authorized user sees correct translations (de) should return correct translations and html attributes
Retry 1Initial Attempt
-22.22% (-4) -4 / 18 runs
failed over last 7 days
22.22% (4) 4 / 18 runs
flaked over last 7 days
authorized user sees correct translations (pt-br) should return correct translations and html attributes
Retry 1Initial Attempt
-11.11% (-2) -2 / 18 runs
failed over last 7 days
22.22% (4) 4 / 18 runs
flaked over last 7 days
authorized user sees correct translations (ar) should return correct translations and html attributes
Retry 1Initial Attempt
-36.36% (-12) -12 / 33 runs
failed over last 7 days
72.73% (24) 24 / 33 runs
flaked over last 7 days
authorized user sees changed translations (de->ar) should return correct translations and html attributes
Retry 1Initial Attempt
-109.52% (-23) -23 / 21 runs
failed over last 7 days
109.52% (23) 23 / 21 runs
flaked over last 7 days
authorized user sees changed translations (de->pt-BR) [locale1] should return correct translations and html attributes
Retry 1Initial Attempt
-20% (-3) -3 / 15 runs
failed over last 7 days
26.67% (4) 4 / 15 runs
flaked over last 7 days

📄   packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 2 Flakes

Top 1 Common Error Messages

null

2 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe according to system theme when no theme is configured through Embed API
Retry 1Initial Attempt
3.70% (11) 11 / 297 runs
failed over last 7 days
42.76% (127) 127 / 297 runs
flaked over last 7 days
Popup Tests should be able to reschedule
Retry 1Initial Attempt
9.12% (27) 27 / 296 runs
failed over last 7 days
89.53% (265) 265 / 296 runs
flaked over last 7 days

📄   packages/app-store/routing-forms/playwright/tests/basic.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Routing Forms Seeded Routing Form Routing Link should validate fields
Retry 1Initial Attempt
0% (0) 0 / 291 runs
failed over last 7 days
10.65% (31) 31 / 291 runs
flaked over last 7 days

📄   packages/embeds/embed-core/playwright/tests/inline.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Inline Iframe Inline Iframe - Configured with Dark Theme
Retry 1Initial Attempt
3.74% (11) 11 / 294 runs
failed over last 7 days
11.90% (35) 35 / 294 runs
flaked over last 7 days

View Detailed Build Results


@github-actions
Copy link
Contributor

github-actions bot commented Oct 17, 2023

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

Copy link
Member

@PeerRich PeerRich left a comment

Choose a reason for hiding this comment

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

looks good! only code removed afaik

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core area: core, team members only event types Created by Linear-GitHub Sync ✨ feature New feature or request High priority Created by Linear-GitHub Sync teams area: teams, round robin, collective, managed event-types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

managed event types: be able to have your own limits
3 participants