singer-io / tap-zoom

GNU Affero General Public License v3.0
4 stars 16 forks source link

Change reports to pull webinar_uuid #32

Open alisa-aylward-toast opened 2 months ago

alisa-aylward-toast commented 2 months ago

Description of change

Changing report webinars and participants to pull each UUID rather than ID; this means they're a child of webinars

Per the documentation, if you pass webinar_id you get only the last result per webinar, but if you pass UUID, you get the results for the UUID: https://developers.zoom.us/docs/api/rest/reference/zoom-api/methods/#operation/reportWebinarDetails

The webinar's ID or universally unique ID (UUID).If you provide a webinar ID, the API will return a response for the latest webinar instance.If you provide a webinar UUID that begins with a / character or contains the // characters, you must double encode the webinar UUID before making an API request.

webinarId string The webinar's ID or universally unique ID (UUID).

If you provide a webinar ID, the API will return a response for the latest webinar instance. If you provide a webinar UUID that begins with a / character or contains the // characters, you must double encode the webinar UUID before making an API request.

Add new fields to participants: https://developers.zoom.us/docs/api/rest/reference/zoom-api/methods/#operation/reportMeetingParticipants

Manual QA steps

I built locally and ran. before, I got a total of 6 results for report_webinars, all grouped together, all with different ids.

After, I got a record for report_webinars after every webinar, closer to 50 results

Risks

Rollback steps