Merge pull request #1752 from edx/gprice/inline-discussion-error-recovery

Add error recovery to inline discussion loading

CHANGELOG.rst

@@ -12,6 +12,9 @@ Blades: Update the calculator hints tooltip with full information. BLD-400.
 
 Blades: Fix transcripts 500 error in studio (BLD-530)
 
+LMS: Add error recovery when a user loads or switches pages in an
+inline discussion.
+
 Blades: Allow multiple strings as the correct answer to a string response question. BLD-474.
 
 Blades: a11y - Videos will alert screenreaders when the video is over.

common/static/coffee/src/discussion/discussion_module_view.coffee

@@ -36,12 +36,15 @@ if Backbone?
       event.preventDefault()
       @newPostForm.slideUp(300)
 
+    hideDiscussion: ->
+      @$("section.discussion").slideUp()
+      @toggleDiscussionBtn.removeClass('shown')
+      @toggleDiscussionBtn.find('.button-text').html("Show Discussion")
+      @showed = false
+
     toggleDiscussion: (event) ->
       if @showed
-        @$("section.discussion").slideUp()
-        @toggleDiscussionBtn.removeClass('shown')
-        @toggleDiscussionBtn.find('.button-text').html("Show Discussion")
-        @showed = false
+        @hideDiscussion()
       else
         @toggleDiscussionBtn.addClass('shown')
         @toggleDiscussionBtn.find('.button-text').html("Hide Discussion")
@@ -51,9 +54,17 @@ if Backbone?
           @showed = true
         else
           $elem = @toggleDiscussionBtn
-          @loadPage $elem
+          @loadPage(
+            $elem,
+            =>
+              @hideDiscussion()
+              DiscussionUtil.discussionAlert(
+                "Sorry",
+                "We had some trouble loading the discussion. Please try again."
+              )
+          )
 
-    loadPage: ($elem)=>
+    loadPage: ($elem, error) =>
       discussionId = @$el.data("discussion-id")
       url = DiscussionUtil.urlFor('retrieve_discussion', discussionId) + "?page=#{@page}"
       DiscussionUtil.safeAjax
@@ -63,6 +74,7 @@ if Backbone?
         type: "GET"
         dataType: 'json'
         success: (response, textStatus, jqXHR) => @renderDiscussion($elem, response, textStatus, discussionId)
+        error: error
 
     renderDiscussion: ($elem, response, textStatus, discussionId) =>
       window.user = new DiscussionUser(response.user_info)
@@ -131,5 +143,14 @@ if Backbone?
     navigateToPage: (event) =>
       event.preventDefault()
       window.history.pushState({}, window.document.title, event.target.href)
+      currPage = @page
       @page = $(event.target).data('page-number')
-      @loadPage($(event.target))
+      @loadPage(
+        $(event.target),
+        =>
+          @page = currPage
+          DiscussionUtil.discussionAlert(
+            "Sorry",
+            "We had some trouble loading the threads you requested. Please try again."
+          )
+      )