Skip to content

fix: use opencraft zoomus fork with single recording

Navin Karkera requested to merge navin/fix-zoom-id into master

Using opencraft fork of zoomus, we can now delete specific zoom recording file based on recording_id. Also, the numeric meeting id is not supported in zoom delete API.

Replaced it with meeting_id from recordings API which may contain / or //. Double encoding it to escape.

Testing instructions

  1. Setup GOOGLE_*, ZOOM_* env variables
  2. Enable FEATURE_ZOOM_AUTOMATION setting and set DEBUG = False
  3. Run upload_recordings_from_zoom from sprintcraft/sprintcraft/dashboard/tasks.py in django shell
  4. Verify that the videos are uploaded to drive
  5. Verify in zoom cloud that recordings are moved to trash.
Edited by Navin Karkera

Merge request reports