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

TLT Suggested Structure Changes #3

Closed lduarte1991 closed 8 years ago

lduarte1991 commented 8 years ago

The following PR contains the following:

  1. Changes from TLT about structuring the django project. Folders and files were moved/renamed
  2. Added a way to stand up the project using a secure.py file created during deployment
  3. Added the course_users attribute (though optional) to the LTI_Course object
  4. Moved the x-frame files into a middleware that handles the cases where both http_referer exists or not
  5. Fixed the unicode issue within both the Django Admin panel and the views (most of the problem is caused by the "debug" attribute that will be removed during production, but I did add unicode toStrings)
arthurian commented 8 years ago

:+1: Looks good to me, but I'll defer to @xueharry and @condaatje since they're closer to the code than I am.

condaatje commented 8 years ago

Looks good, will definitely make it easier for us to stay synced up in the future. Thanks for being so responsive.

lduarte1991 commented 8 years ago

Sounds good! Will merge now.