Merge branch 'feature/arjun/new-discussions' of github.com:MITx/mitx into feature/arjun/new-discussions

Conflicts:
	lms/djangoapps/django_comment_client/forum/views.py
	lms/static/sass/_discussion.scss

loaded into the page in a data attribute instead of loaded on request via ajax.
The user profile page also does not provide facilities for adding
comments/responses, which I think would clutter the page too much.

Also removed some unused stuff from views.py.

Kill requests for loading responses when selecting another thread. This ensures that the thread you're viewing has the right responses loaded.

course id for seed_permissions_roles, and also the script itself to tell you that.

Backbone.Model attributes because Model attributes get serialized into
JSON which is wasteful.

This just puts them in three traversal order rather than chronological order.
Not sure which makes the most sense, but I'll implement both ways.

down on the amount of data shuffled around and saves some rendering time
on the forum index as well.

used anywhere anyway.

I don't know why this was ever defined when python has a ternary operator
which doesn't require its arguments to be evaluated if the condition
evaluates to false, whereas this helper method does.

This reverts commit b820c345.