ncstate-delta / moodle-mod_zoom

Moodle plugin for Zoom meeting
https://moodle.org/plugins/mod_zoom
61 stars 108 forks source link

zoom_update_instance() - reduce update_record calls; make sure Zoom API succeeds #586

Open jrchamp opened 5 months ago

jrchamp commented 5 months ago

A future update should be able to reduce the number of update_record calls. For example, if the update_meeting() API call fails, then we don't want to save values locally that Zoom is the source of truth for. However, we would still want to save values locally that Moodle is the source of truth for.

_Originally posted by @jrchamp in https://github.com/ncstate-delta/moodle-mod_zoom/pull/585#pullrequestreview-1994529523_