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: before/after event buffers not rendering correctly #13451

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

keithwillcode
Copy link
Contributor

What does this PR do?

Fixes #13422

Type of change

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

How should this be tested?

  • Save before/after event buffer fields on event types and ensure they save properly

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

@keithwillcode keithwillcode added High priority Created by Linear-GitHub Sync event types Created by Linear-GitHub Sync labels Jan 29, 2024
@keithwillcode keithwillcode added this to the v3.8 milestone Jan 29, 2024
@keithwillcode keithwillcode requested a review from a team January 29, 2024 14:07
@keithwillcode keithwillcode linked an issue Jan 29, 2024 that may be closed by this pull request
Copy link

linear bot commented Jan 29, 2024

Copy link

vercel bot commented Jan 29, 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 Jan 29, 2024 3:20pm
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 29, 2024 3:20pm
dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 29, 2024 3:20pm
5 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jan 29, 2024 3:20pm
cal-demo ⬜️ Ignored (Inspect) Jan 29, 2024 3:20pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Jan 29, 2024 3:20pm
qa ⬜️ Ignored (Inspect) Visit Preview Jan 29, 2024 3:20pm
ui ⬜️ Ignored (Inspect) Visit Preview Jan 29, 2024 3:20pm

@github-actions github-actions bot added 🐛 bug Something isn't working 📉 regressing This used to work. Now it doesn't anymore. labels Jan 29, 2024
Copy link
Contributor

github-actions bot commented Jan 29, 2024

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

Copy link
Contributor

github-actions bot commented Jan 29, 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 Jan 29, 2024

Current Playwright Test Results Summary

✅ 444 Passing - ⚠️ 19 Flaky

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

(Last updated on 01/29/2024 03:27:30pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 75da257

Started: 01/29/2024 03:17:17pm UTC

⚠️ Flakes

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

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
pro user -- future Time slots should be reserved when selected
Retry 1Initial Attempt
0% (0) 0 / 65 runs
failed over last 7 days
4.62% (3) 3 / 65 runs
flaked over last 7 days

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

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Checkbox Group Question and Each Other Question Booking With Checkbox Group Question and Address Question Booking With Checkbox Group Question and select Question Checkbox Group required and select required
Retry 1Initial Attempt
0% (0) 0 / 258 runs
failed over last 7 days
3.49% (9) 9 / 258 runs
flaked over last 7 days

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

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Check availability tab in a event-type Check availability in event type
Retry 1Initial Attempt
18.49% (49) 49 / 265 runs
failed over last 7 days
44.91% (119) 119 / 265 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 matching orgAutoAcceptEmail and a Verified Organization Team Invitation
Retry 2Retry 1Initial Attempt
5.79% (15) 15 / 259 runs
failed over last 7 days
8.49% (22) 22 / 259 runs
flaked over last 7 days

📄   apps/web/playwright/integrations-stripe.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Stripe integration When event is paid and confirmed Payment should confirm pending payment booking
Retry 1Initial Attempt
0.77% (2) 2 / 261 runs
failed over last 7 days
2.30% (6) 6 / 261 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 / 278 runs
failed over last 7 days
26.26% (73) 73 / 278 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
0.72% (2) 2 / 279 runs
failed over last 7 days
51.97% (145) 145 / 279 runs
flaked over last 7 days
Popup Tests should be able to reschedule
Retry 1Initial Attempt
-101.46% (-139) -139 / 137 runs
failed over last 7 days
109.49% (150) 150 / 137 runs
flaked over last 7 days
Popup Tests should open Routing Forms embed on click
Retry 1Initial Attempt
-100.74% (-137) -137 / 136 runs
failed over last 7 days
102.21% (139) 139 / 136 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
-101.48% (-137) -137 / 135 runs
failed over last 7 days
106.67% (144) 144 / 135 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
-102.22% (-138) -138 / 135 runs
failed over last 7 days
102.22% (138) 138 / 135 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
-102.22% (-138) -138 / 135 runs
failed over last 7 days
102.22% (138) 138 / 135 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
-102.22% (-138) -138 / 135 runs
failed over last 7 days
102.22% (138) 138 / 135 runs
flaked over last 7 days
Popup Tests prendered embed should be loaded and apply the config given to it
Retry 1Initial Attempt
-101.48% (-137) -137 / 135 runs
failed over last 7 days
102.22% (138) 138 / 135 runs
flaked over last 7 days

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

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Webhook tests FORM_SUBMITTED on submitting team form, triggers team webhook
Retry 2Retry 1Initial Attempt
0% (0) 0 / 92 runs
failed over last 7 days
19.57% (18) 18 / 92 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 Test preview should return correct route
Retry 1Initial Attempt
0% (0) 0 / 281 runs
failed over last 7 days
9.61% (27) 27 / 281 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
1.44% (4) 4 / 278 runs
failed over last 7 days
39.57% (110) 110 / 278 runs
flaked over last 7 days

📄   apps/web/playwright/embed-code-generator.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
Embed Code Generator Tests Non-Organization Event Types Page open Embed Dialog and choose Inline for First Event Type
Retry 1Initial Attempt
0% (0) 0 / 288 runs
failed over last 7 days
8.33% (24) 24 / 288 runs
flaked over last 7 days
Embed Code Generator Tests Non-Organization Event Types Page open Embed Dialog and choose element-click for First Event Type
Retry 1Initial Attempt
0% (0) 0 / 288 runs
failed over last 7 days
7.64% (22) 22 / 288 runs
flaked over last 7 days

View Detailed Build Results


Copy link
Contributor

@emrysal emrysal left a comment

Choose a reason for hiding this comment

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

👍

@keithwillcode keithwillcode enabled auto-merge (squash) January 29, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working core area: core, team members only event types Created by Linear-GitHub Sync foundation High priority Created by Linear-GitHub Sync 📉 regressing This used to work. Now it doesn't anymore.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CAL-3035] Buffer not saving
3 participants