-
Notifications
You must be signed in to change notification settings - Fork 3
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
use canvasapi
#43
Comments
lsloan
added a commit
to lsloan/tool-migration
that referenced
this issue
Dec 20, 2023
But the API call that provides the data needs to be corrected first. See tl-its-umich-edu#43.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Update the app to use UCF's
canvasapi
module instead of the buggy, bespoke one.For example,
get_term_names()
didn't load all pages of term data from the API, so it didn't always have data for the requested term(s). Using the other module would've avoided this problem.The text was updated successfully, but these errors were encountered: