Reviewer: ignoring spaces in the diff will save significant time.
Previously, if the API call failed due to network or rate-limiting issues, the delete recordings task would think that the recording had been deleted from Zoom (and the local entries in Moodle would also be deleted).
One potential concern: will the API return a "that Meeting UUID doesn't exist" error? If it did, then we wouldn't delete the recordings. Maybe we should check that they exist directly? At least now, we're failing into a safer stance.
Reviewer: ignoring spaces in the diff will save significant time.
Previously, if the API call failed due to network or rate-limiting issues, the delete recordings task would think that the recording had been deleted from Zoom (and the local entries in Moodle would also be deleted).
One potential concern: will the API return a "that Meeting UUID doesn't exist" error? If it did, then we wouldn't delete the recordings. Maybe we should check that they exist directly? At least now, we're failing into a safer stance.
Fixes #539 (hopefully!)