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

Release 2017-03-15: Consolidates PRs 78,79,80 #82

Closed arthurian closed 7 years ago

arthurian commented 7 years ago

@lduarte1991 Turning the release branch you created into a PR so we can discuss the changes contained in PRs #78 #79 and #80 as a unit. I imagine you can just close the other PRs once this one is merged to master.

I'll make any comments below.

arthurian commented 7 years ago

:+1: to merge!

lduarte1991 commented 7 years ago

@arthurian if you're cool with the logger_url being a secure.py option then I'm ready to merge as well

arthurian commented 7 years ago

@lduarte1991 I'm good with the logger_url being a setting option.