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: org user event types with same event slug #13571

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

CarinaWolli
Copy link
Member

What does this PR do?

When org members had the same event slugs wrong event types were shown on org event type links (acme.cal.local:3000/user/slug)

Copy link

vercel bot commented Feb 6, 2024

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 Feb 6, 2024 10:33pm
api 🔄 Building (Inspect) Visit Preview Feb 6, 2024 10:33pm
4 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Feb 6, 2024 10:33pm
cal-demo ⬜️ Ignored (Inspect) Feb 6, 2024 10:33pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Feb 6, 2024 10:33pm
ui ⬜️ Ignored (Inspect) Visit Preview Feb 6, 2024 10:33pm

Copy link
Contributor

github-actions bot commented Feb 6, 2024

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

@keithwillcode keithwillcode added consumer core area: core, team members only labels Feb 6, 2024
Copy link
Contributor

@joeauyeung joeauyeung left a comment

Choose a reason for hiding this comment

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

Found the same fix.

This was used in a findFirst query so it was grabbing the first member of the org with the same slug.

Copy link
Contributor

github-actions bot commented Feb 6, 2024

📦 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

deploysentinel bot commented Feb 6, 2024

Current Playwright Test Results Summary

✅ 440 Passing - ⚠️ 23 Flaky

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

(Last updated on 02/06/2024 10:48:18pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 10a4b4f

Started: 02/06/2024 10:38:36pm UTC

⚠️ Flakes

📄   apps/web/playwright/booking/multipleEmailQuestion.e2e.ts • 3 Flakes

Top 1 Common Error Messages

null

3 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Multiple Email Question and Each Other Question Booking With Multiple Email Question and Address Question Multiple Email and Address not required
Retry 1Initial Attempt
2.80% (9) 9 / 321 runs
failed over last 7 days
3.74% (12) 12 / 321 runs
flaked over last 7 days
Booking With Multiple Email Question and Each Other Question Booking With Multiple Email Question and checkbox group Question Multiple Email and checkbox group not required
Retry 1Initial Attempt
2.80% (9) 9 / 322 runs
failed over last 7 days
5.90% (19) 19 / 322 runs
flaked over last 7 days
Booking With Multiple Email Question and Each Other Question Booking With Multiple Email Question and Long text Question Multiple Email required and Long text required
Retry 1Initial Attempt
0.32% (1) 1 / 310 run
failed over last 7 days
7.10% (22) 22 / 310 runs
flaked over last 7 days

📄   apps/web/playwright/booking/longTextQuestion.e2e.ts • 4 Flakes

Top 1 Common Error Messages

null

4 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Long Text Question and Each Other Question Long Text and Address required
Retry 1Initial Attempt
0.33% (1) 1 / 299 run
failed over last 7 days
5.02% (15) 15 / 299 runs
flaked over last 7 days
Booking With Long Text Question and Each Other Question Booking With Long Text Question and multiselect Question Long Text required and multiselect text not required
Retry 1Initial Attempt
0% (0) 0 / 297 runs
failed over last 7 days
2.69% (8) 8 / 297 runs
flaked over last 7 days
Booking With Long Text Question and Each Other Question Booking With Long Text Question and Number Question Long Text and Number required
Retry 1Initial Attempt
0.34% (1) 1 / 297 run
failed over last 7 days
4.04% (12) 12 / 297 runs
flaked over last 7 days
Booking With Long Text Question and Each Other Question Booking With Long Text Question and Short text question Long Text and Short text required
Retry 1Initial Attempt
0% (0) 0 / 292 runs
failed over last 7 days
4.11% (12) 12 / 292 runs
flaked over last 7 days

📄   apps/web/playwright/booking/collectiveEventType/collectiveEventType.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Collective event type Book a Collective event type (with added guest)
Retry 1Initial Attempt
0% (0) 0 / 294 runs
failed over last 7 days
2.72% (8) 8 / 294 runs
flaked over last 7 days

📄   apps/web/playwright/login.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
user can login & logout succesfully -- future login flow user & logout using dashboard
Retry 1Initial Attempt
5.67% (17) 17 / 300 runs
failed over last 7 days
37% (111) 111 / 300 runs
flaked over last 7 days

📄   apps/web/playwright/booking/radioGroupQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Radio Question and Each Other Question Booking With Radio Question and Address Question Booking With Radio Question and Phone Question Radio and Phone not required
Retry 1Initial Attempt
0.34% (1) 1 / 293 run
failed over last 7 days
4.44% (13) 13 / 293 runs
flaked over last 7 days

📄   apps/web/playwright/organization/organization-invitation.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Organization Email not matching orgAutoAcceptEmail Org Invitation
Retry 2Retry 1Initial Attempt
13.42% (42) 42 / 313 runs
failed over last 7 days
6.07% (19) 19 / 313 runs
flaked over last 7 days

📄   apps/web/playwright/event-types.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Event Types tests -- future user can add multiple organizer address
Retry 1Initial Attempt
0.60% (2) 2 / 332 runs
failed over last 7 days
17.77% (59) 59 / 332 runs
flaked over last 7 days

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

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Preview Preview - embed-core should load
Retry 1Initial Attempt
0% (0) 0 / 315 runs
failed over last 7 days
27.30% (86) 86 / 315 runs
flaked over last 7 days

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

Top 1 Common Error Messages

null

8 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should open embed iframe on click - Configured with light theme
Retry 1Initial Attempt
-3.16% (-10) -10 / 316 runs
failed over last 7 days
50.32% (159) 159 / 316 runs
flaked over last 7 days
Popup Tests should be able to reschedule
Retry 1Initial Attempt
-95.78% (-159) -159 / 166 runs
failed over last 7 days
95.78% (159) 159 / 166 runs
flaked over last 7 days
Popup Tests should open Routing Forms embed on click
Retry 1Initial Attempt
-84.94% (-141) -141 / 166 runs
failed over last 7 days
87.35% (145) 145 / 166 runs
flaked over last 7 days
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
-89.51% (-145) -145 / 162 runs
failed over last 7 days
89.51% (145) 145 / 162 runs
flaked over last 7 days
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe(Booker Profile Page) with dark theme when configured with dark theme using Embed API
Retry 1Initial Attempt
-89.51% (-145) -145 / 162 runs
failed over last 7 days
89.51% (145) 145 / 162 runs
flaked over last 7 days
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 configured with 'auto' theme using Embed API
Retry 1Initial Attempt
-89.51% (-145) -145 / 162 runs
failed over last 7 days
89.51% (145) 145 / 162 runs
flaked over last 7 days
Popup Tests prendered embed should be loaded and apply the config given to it
Retry 1Initial Attempt
-89.51% (-145) -145 / 162 runs
failed over last 7 days
89.51% (145) 145 / 162 runs
flaked over last 7 days
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe(Event Booking Page) with dark theme when configured with dark theme using Embed API
Retry 1Initial Attempt
-89.51% (-145) -145 / 162 runs
failed over last 7 days
89.51% (145) 145 / 162 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
0.32% (1) 1 / 314 run
failed over last 7 days
40.76% (128) 128 / 314 runs
flaked over last 7 days

📄   apps/web/playwright/booking/phoneQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Phone Question and Each Other Question Booking With Phone Question and Address Question Booking With Phone Question and checkbox group Question Phone required and checkbox group not required
Retry 1Initial Attempt
0.65% (2) 2 / 309 runs
failed over last 7 days
3.88% (12) 12 / 309 runs
flaked over last 7 days

View Detailed Build Results


@CarinaWolli CarinaWolli merged commit 1a3d54a into main Feb 6, 2024
38 checks passed
@CarinaWolli CarinaWolli deleted the fix/org-user-event-types branch February 6, 2024 22:50
Shyam-Raghuwanshi pushed a commit to Shyam-Raghuwanshi/cal.com that referenced this pull request Feb 7, 2024
Co-authored-by: CarinaWolli <wollencarina@gmail.com>
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 Urgent Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants