Skip to content
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

List of recordings requires a 'from' parameter incompatible with Python #135

Open
gregorydlogan opened this issue Aug 31, 2020 · 1 comment

Comments

@gregorydlogan
Copy link
Contributor

Per https://marketplace.zoom.us/docs/api-reference/zoom-api/cloud-recording/recordingslist, if you don't include the from parameter in the REST call you get a default of yesterday. I wanted more like a week, so I tried this:

recordings_response = self.zoom_client.recording.list(user_id=user_id, mc="false", page_size=30, trash_type="meeting_recordings", from="2020-01-01")

which obviously doesn't work. Attempting to use from_ instead also did not work. This does, at the cost of legibility

recordings_response = self.zoom_client.recording.list(user_id=user_id, mc="false", page_size=30, trash_type="meeting_recordings", **{"from": "2020-01-01"})

Resolution: Bake in from_ support, or some other kind of parameter renaming.

@patcon
Copy link

patcon commented May 10, 2021

for future readers, this also works and could appear pretty conventional (though yes, this seems like an odd thing to have no guidance on:

options = {
  'user_id': user_id,
  'mc': 'false',
  'page_size': 30,
  'trash_type': 'meeting_recordings',
  'from': '2020-01-01',
}
recordings_response = self.zoom_client.recording.list(**options)

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

No branches or pull requests

2 participants