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.

Sign up for GitHub

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

Jump to bottom

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

Open
jrchamp opened this issue Apr 11, 2024 · 0 comments
Open

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

jrchamp opened this issue Apr 11, 2024 · 0 comments
Labels
bug Fixes problems or reduces technical debt help wanted We need your help to make this possible

Comments

@jrchamp
Copy link
Collaborator

jrchamp commented Apr 11, 2024

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 #585 (review)
@jrchamp jrchamp added the bug Fixes problems or reduces technical debt label Apr 11, 2024
@jrchamp jrchamp added the help wanted We need your help to make this possible label Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Fixes problems or reduces technical debt help wanted We need your help to make this possible
Projects
mod_zoom Workflow
Status: To Do
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jrchamp