Closed IanRFerguson closed 11 months ago
we're rockin 😎
@Jason94 + @willyraedy I'm OOO tomorrow and Friday, will prioritize merging this on Monday with additional unit tests to stay ahead of the 9/1 deadline
Need to finish unit tests for new endpoints
@Jason94 I need some more time to work through the unit testing, but the authentication works as is so I think this is ready to merge with the caveat that the poll endpoints might not be 100% functional just yet (happy to dep those temporarily so they're not in the live version)
Something went wrong with this PR, a lot of unrelated code got included, e.g. on the ActionNetwork connector 🤔
Something went wrong with this PR, a lot of unrelated code got included, e.g. on the ActionNetwork connector 🤔
Likely my fault, sorry for making more work
This PR adds two significant changes to the Zoom connector
Server to Server OAuth
Zoom is deprecating JWT authentication on September 1st, which necessitates an update to the authentication strategy used by this connector. The
Zoom()
constructor now requires three values to connect to the API:account_id
client_id
client_secret
These values are passed to the Zoom OAuth callback URL and an access token is assigned to a class attribute.
Please note that these values replace the
api_key
andapi_secret
previously passed into the Zoom connectorSetting Up a Server-to-Server OAuth App
Follow the steps below to generate the required env variables and scopes to authenticate your Zoom connector:
Zoom Poll API
Per request from @Jason94 and co, we've added functions to query Zoom polls from a given meeting. There are two relevant functions here, one to get a
Table
of all polls for a given meeting (just requiresmeeting_id
) and one to get data about one specific poll (requiresmeeting_id
ANDpoll_id
)