-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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: cal video recording email #12079
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
5 Ignored Deployments
|
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link to collect XP and win prizes! |
📦 Next.js Bundle Analysis for @calcom/webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
Current Playwright Test Results Summary✅ 231 Passing - Run may still be in progress, this comment will be updated as current testing workflow or job completes... (Last updated on 10/31/2023 04:29:51pm UTC) Run DetailsRunning Workflow PR Update on Github Actions Commit: d4b95f0 Started: 10/31/2023 04:24:46pm UTC
|
|
2 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 2 • Retry 1 • Initial Attempt |
2.73% (9)9 / 330 runsfailed over last 7 days |
48.79% (161)161 / 330 runsflaked over last 7 days |
Popup Tests should be able to reschedule
Retry 1 • Initial Attempt |
16.97% (56)56 / 330 runsfailed over last 7 days |
80.91% (267)267 / 330 runsflaked 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 |
---|---|---|
Login and logout tests Login flow validations Should warn when password is incorrect
Retry 1 • Initial Attempt |
0% (0)0 / 313 runsfailed over last 7 days |
0.32% (1)1 / 313 runflaked over last 7 days |
📄 apps/web/playwright/teams.e2e.ts • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Teams - NonOrg Can create a booking for Collective EventType
Retry 1 • Initial Attempt |
0% (0)0 / 129 runsfailed over last 7 days |
0.78% (1)1 / 129 runflaked over last 7 days |
📄 packages/embeds/embed-core/playwright/tests/embed-pages.e2e.ts • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Embed Pages isEmbed, getEmbedNamespace, getEmbedTheme testing when `window.name` is set to 'cal-embed=' and `theme` is supplied as a query param
Retry 1 • Initial Attempt |
0% (0)0 / 321 runsfailed over last 7 days |
0.62% (2)2 / 321 runsflaked 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 user Different Locations Tests can add Attendee Phone Number location and book with it
Retry 1 • Initial Attempt |
0% (0)0 / 321 runsfailed over last 7 days |
7.48% (24)24 / 321 runsflaked over last 7 days |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me :)
* fix: cal video recording email * fix: add check for recording * chore: remove logs * chore: change message --------- Co-authored-by: Peer Richelsen <peeroke@gmail.com>
* fix: cal video recording email * fix: add check for recording * chore: remove logs * chore: change message --------- Co-authored-by: Peer Richelsen <peeroke@gmail.com>
* fix: cal video recording email * fix: add check for recording * chore: remove logs * chore: change message --------- Co-authored-by: Peer Richelsen <peeroke@gmail.com>
Fixes #12076
Added another button "Check for Recordings" just in case isRecorded field in DB is not updated to true due to error or failure. User would still be check if there are any recordings
Email:-
Screen.Recording.2023-10-25.at.10.43.27.PM.mov
Screen.Recording.2023-10-25.at.10.38.08.PM.mov