pylti / lti

Learning Tools Interoperability for Python
Other
78 stars 45 forks source link

Add support for Canvas ltiLaunchUrl in outcome request #77

Open hmoffatt opened 3 years ago

hmoffatt commented 3 years ago

Canvas LMS supports an extra field in the outcome request's resultData field called ltiLaunchUrl (in addition to the standard text and url fields). If present, Canvas will launch the URL as an LTI tool (while url just shows a page).

I need this in my application so I've added it in. It's non-standard though so you may not want to accept this.

Also, the code could be cleaned up a bit more now that there's three possible fields in the resultData, and also there needs to be unit tests added for all of the resultData code.