search

UBC-DSCI / rudaux

LMS + External Tool Workflow Orchestration
https://ubc-dsci.github.io/rudaux
MIT License
7 stars 5 forks source link

consider using canvasapi #98

Closed trevorcampbell closed 3 years ago

trevorcampbell commented 3 years ago

https://github.com/ucfopen/canvasapi

This is not a super high priority item, but it seems like this package has been under consistent active dev for a while and looks stable. Maybe better error checking / more efficient requests? / etc than what we do. Also if Canvas updates their API, those folks might fix bugs faster?

We originally opted not to use it because we use only a very small subset of the API, and rather than depend on a large package from somewhere else we figured it would be easy enough to just make our own small interface. That logic still holds but generally if we can simplify rudaux I'm all for it.

trevorcampbell commented 3 years ago

Update: not going to do this as part of the big version upgrade in #95. But I'll leave it open since it is totally possible to swap out the new Canvas API code in course_api.py for this package -- just no reason to do it right now. Maybe later.

trevorcampbell commented 3 years ago

Closing, since in the new modularized version of Rudaux the code will be agnostic to LMS api. We can revisit this later if necessary.