cdodge/fire-progress-on-html-module-render: modify the xmodule HTML to f...

...ire a progress event to record the fact that the user visited that content
Alrighty, then -- +1 from me


On Wed, Aug 6, 2014 at 3:05 PM, chrisndodge <notifications@github.com>
wrote:

> In lms/djangoapps/courseware/module_render.py:
>
> > @@ -360,6 +360,8 @@ def handle_progress_event(block, event_type, event):
> >          tie into the CourseCompletions datamodels that are exposed in the api_manager djangoapp
> >          """
> >          user_id = event.get('user_id', user.id)
> > +        if not user_id:
>
> Yes, there are valid cases where user is none. For example, browsing
> courseware content when not yet logged in, such as the course about page.
>
> I put it here, because I stumbled on the crash when I was running it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/edx-solutions/edx-platform/pull/159/files#r15895838>.
>



-- 
Matthew Drayer
Solutions Architect
edX | MIT
mattdrayer@edx.org