Skip to content

Conversation

@michelle0927
Copy link
Collaborator

@michelle0927 michelle0927 commented Jul 18, 2022

Resolves #3342

Historical events will be emitted in the deploy hooks of the following sources/triggers:
Meeting Created
Meeting Ended
Meeting Started
Meeting Updated
Recording Completed
Webinar Created
Webinar Ended
Webinar Started
Webinar Updated

The following events require a paid subscription, and have not been tested.
Recording Completed
Webinar Created
Webinar Ended
Webinar Started
Webinar Updated

The following sources don't yet emit historical events because the Zoom API doesn't yet offer a way to retrieve generic events or deleted resources.
Custom Event
Phone Event
Meeting Deleted
Webinar Deleted

@michelle0927 michelle0927 self-assigned this Jul 18, 2022
@vercel
Copy link

vercel bot commented Jul 18, 2022

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

Name Status Preview Updated
pipedream-docs ✅ Ready (Inspect) Visit Preview Jul 20, 2022 at 4:27PM (UTC)
pipedream-docs-redirect-do-not-edit ✅ Ready (Inspect) Visit Preview Jul 20, 2022 at 4:27PM (UTC)

@dylburger
Copy link
Contributor

Thanks for submitting this PR! When we review PRs, we follow the Pipedream component guidelines. If you're not familiar, here's a quick checklist:

lcaresia
lcaresia previously approved these changes Jul 19, 2022
Copy link
Collaborator

@lcaresia lcaresia left a comment

Choose a reason for hiding this comment

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

LGTM!

@DilanAthukorala
Copy link

@michelle0927 Recording Completed source throws error upon source creation
image

@DilanAthukorala
Copy link

@michelle0927 For "Recording Completed" source, there are no events emitted after the recordings are completed.
image
image
image

@michelle0927
Copy link
Collaborator Author

@DilanAthukorala The "Recording Competed" source emits events for new cloud recordings (https://marketplace.zoom.us/docs/api-reference/zoom-api/events#/paths/recording.completed/post). The screenshot above shows a recording under Local Recordings. Do you have recordings under Cloud Recording that are not showing up? I don't have a Pro account, so I haven't been able to test that component.

@DilanAthukorala
Copy link

@michelle0927 Thank you, I was only able to test the components from my free account.

I was not able to test the following sources that requires paid subscription.

  • Recording Completed
  • Webinar Created
  • Webinar Ended
  • Webinar Started
  • Webinar Updated
  • Phone Events

Verification completed for the following sources and no issues observed.

  • Meeting Created
  • Meeting Ended
  • Meeting Started
  • Meeting Updated
  • Meeting Deleted
  • Custom Events

@dannyroosevelt There were only 50% of sources that could be tested with free account, can we sign off with the above test results considering these were updates to already existing sources?

@michelle0927
Copy link
Collaborator Author

/approve

@jcortes jcortes self-requested a review July 25, 2022 18:12
Copy link
Collaborator

@jcortes jcortes left a comment

Choose a reason for hiding this comment

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

Approved!

@michelle0927 michelle0927 merged commit bd22040 into master Jul 26, 2022
@michelle0927 michelle0927 deleted the zoom-3342 branch July 26, 2022 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TRIGGER] zoom

6 participants