cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

feat: add RECORDING_READY webhook

Open Udit-takkar opened this issue 2 years ago • 4 comments

What does this PR do?

Fixes https://github.com/calcom/cal.com/issues/6368

TODO

  • [x] remove "any" type and console log
  • [x] test once again

Log of payload

Screenshot 2023-04-14 at 11 19 04 PM

How to test ?

  1. Create a team event type and select cal video as location
  2. Create a webhook with trigger "Recording Download Link Ready"
  3. Book a meeting and click on Record button in the meeting
  4. Stop recording

Environment: Staging(main branch) / Production

Type of change

  • New feature (non-breaking change which adds functionality)

Udit-takkar avatar Apr 14 '23 17:04 Udit-takkar

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

Name Status Preview Comments Updated (UTC)
cal ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2023 4:51pm
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2023 4:51pm

vercel[bot] avatar Apr 14 '23 17:04 vercel[bot]

📦 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! 🙌

github-actions[bot] avatar Apr 28 '23 17:04 github-actions[bot]

Current Playwright Test Results Summary

✅ 112 Passing - ⚠️ 5 Flaky

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

(Last updated on 05/09/2023 05:00:47pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: f0b72a717237e313f3e4168c6a857770bf943e18

Started: 05/09/2023 04:51:04pm UTC

⚠️ Flakes

📄   packages/embeds/embed-core/playwright/tests/action-based.test.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should be able to reschedule
Retry 2Retry 1Initial Attempt
17.16% (29) 29 / 169 runs
failed over last 7 days
62.72% (106) 106 / 169 runs
flaked over last 7 days

📄   packages/app-store/routing-forms/playwright/tests/basic.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
Routing Forms Seeded Routing Form Routing Link - Reporting and CSV Download
Retry 1Initial Attempt
4.42% (8) 8 / 181 runs
failed over last 7 days
8.29% (15) 15 / 181 runs
flaked over last 7 days
Routing Forms Seeded Routing Form Router URL should work
Retry 2Retry 1Initial Attempt
0% (0) 0 / 181 runs
failed over last 7 days
8.84% (16) 16 / 181 runs
flaked over last 7 days

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

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking with Seats -- new-booker Reschedule for booking with seats -- old-booker Should reschedule booking with seats and if everyone rescheduled it should be deleted
Retry 1Initial Attempt
0% (0) 0 / 284 runs
failed over last 7 days
7.04% (20) 20 / 284 runs
flaked over last 7 days

📄   apps/web/playwright/app-store.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
App Store - Authed Browse apple-calendar and try to install
Retry 1Initial Attempt
0.34% (1) 1 / 294 run
failed over last 7 days
64.63% (190) 190 / 294 runs
flaked over last 7 days

View Detailed Build Results


deploysentinel[bot] avatar Apr 28 '23 17:04 deploysentinel[bot]

Screenshot 2023-05-09 at 10 59 13 PM Unrelated to this PR, but saw this broken UI while testing

hariombalhara avatar May 09 '23 17:05 hariombalhara