[ARVADOS] updated: 6058d8a2af0a7461f9a412b1b69948e4d0e9224f

git at public.curoverse.com git at public.curoverse.com
Fri Mar 6 17:22:17 EST 2015


Summary of changes:
 apps/workbench/app/assets/javascripts/report_issue.js       | 13 +++++++------
 .../app/views/application/_report_issue_popup.html.erb      |  2 +-
 2 files changed, 8 insertions(+), 7 deletions(-)

       via  6058d8a2af0a7461f9a412b1b69948e4d0e9224f (commit)
      from  db1542b94b228e92e9b5bd82ad93ef83239a2768 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 6058d8a2af0a7461f9a412b1b69948e4d0e9224f
Author: Tom Clegg <tom at curoverse.com>
Date:   Fri Mar 6 17:23:47 2015 -0500

    5105: Include form data with report_issue request.

diff --git a/apps/workbench/app/assets/javascripts/report_issue.js b/apps/workbench/app/assets/javascripts/report_issue.js
index 371dacc..539489d 100644
--- a/apps/workbench/app/assets/javascripts/report_issue.js
+++ b/apps/workbench/app/assets/javascripts/report_issue.js
@@ -8,14 +8,15 @@ $(document).
     }
     $('div').remove('.modal-footer-status');
 
-    $.ajax('/report_issue', {type: 'POST'}).
-      success(function(data, status, jqxhr) {
+    $.ajax('/report_issue', {
+        type: 'POST',
+        data: $(this).parents('form').serialize()
+    }).success(function(data, status, jqxhr) {
         var $sendButton = $('#report-issue-submit');
         $sendButton.html('Report sent');
         $('div').remove('.modal-footer-status');
         $('.modal-footer').append('<div><br/></div><div class="modal-footer-status alert alert-success"><p class="contain-align-left">Thanks for reporting this issue!</p></div>');
-      }).
-      fail(function(jqxhr, status, error) {
+    }).fail(function(jqxhr, status, error) {
         var $sendButton = $('#report-issue-submit');
         if ($sendButton && $sendButton.prop('disabled')) {
           $('div').remove('.modal-footer-status');
@@ -25,6 +26,6 @@ $(document).
         }
         var $cancelButton = $('#report-issue-cancel');
         $cancelButton.html('Cancel');
-      });
-
+    });
+    return false;
   });
diff --git a/apps/workbench/app/views/application/_report_issue_popup.html.erb b/apps/workbench/app/views/application/_report_issue_popup.html.erb
index 315116b..1c964ab 100644
--- a/apps/workbench/app/views/application/_report_issue_popup.html.erb
+++ b/apps/workbench/app/views/application/_report_issue_popup.html.erb
@@ -27,7 +27,7 @@
   <div class="modal-content">
 
     <%= form_tag report_issue_path, {name: 'report-issue-form', method: 'post',
-        class: 'form-horizontal', remote: true} do %>
+        class: 'form-horizontal'} do %>
 
       <%
         title = 'Version / debugging info'

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list