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.
12 stars 7 forks source link

Instructor Dashboard Improvements #35

Closed arthurian closed 8 years ago

arthurian commented 8 years ago

This PR includes some performance improvements for the instructor dashboard.

  1. Instead of hitting the database for each annotation to lookup the assignment name, target object name, etc, the data is prepared ahead of time in the view before being passed to the template.
  2. Image thumbnails are lazy loaded by default now, so the image server won't get hammered if there's a ton of image annotations.
  3. Added basic logging configuration to the django configuration. By default, it's configured to only log ERROR-level events, but you might find it useful to set it to DEBUG for development. By logging django events, you can see stuff like the SQL queries that are being executed and other fun stuff. Just add "django_log_level": "DEBUG" to your secure.py.

@lduarte1991 Let me know if any of this raises any concerns on your end. I made sure to include the _focus_onid query parameter in the object links.

lduarte1991 commented 8 years ago

This looks good to me! :+1: