search

lduarte1991 / hxat

Contains the currently-in-development project by HarvardX to bring the annotation tool currently living in the edX platform to a more accessible LTI implementation.
13 stars 7 forks source link

Minor tweaks to views.py to get it working in Canvas #25

Closed arthurian closed 8 years ago

arthurian commented 8 years ago

Hey @lduarte1991 I was able to get the _mergecandidate working in Canvas with a few small changes (see diff). All of the functionality appears to be there, but we'll have to do some more testing. Take a look at the changes and let me know what you think.

One quick question: I was getting a weird error related to the JWT library. It was complaining that the jwt module didn't have an "encode" method. I tried installing PyJWT instead, and that seemed to fix the issue. Is that what you're using? I didn't update the requirements.txt with that, because I wasn't sure if that was the case or not.

lduarte1991 commented 8 years ago

@arthurian yup, I'm using PyJWT v 1.4.0, sorry about that, I'll email you my current pip freeze output in case I added something else without letting you know.