From 2976678a971de52d8684cca14afda0cdc60ff995 Mon Sep 17 00:00:00 2001
From: Dave Pisek <dpisek@gitlab.com>
Date: Fri, 5 Jul 2019 14:54:43 +1000
Subject: Resolve http issue with resolving notes in IE11

When calling `Vue.http.post` without passing in a data argument
IE<edge will send `undefined` as the request-body. This will cause
an issue for the API.

Setting data to `null` prevents this from happening.
---
 app/assets/javascripts/notes/services/notes_service.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/assets/javascripts/notes/services/notes_service.js b/app/assets/javascripts/notes/services/notes_service.js
index 237e70c0a4c..1434d016302 100644
--- a/app/assets/javascripts/notes/services/notes_service.js
+++ b/app/assets/javascripts/notes/services/notes_service.js
@@ -25,8 +25,10 @@ export default {
   toggleResolveNote(endpoint, isResolved) {
     const { RESOLVE_NOTE_METHOD_NAME, UNRESOLVE_NOTE_METHOD_NAME } = constants;
     const method = isResolved ? UNRESOLVE_NOTE_METHOD_NAME : RESOLVE_NOTE_METHOD_NAME;
+    // need to set this to null or ie < edge will send 'undefined' as a string
+    const data = null;
 
-    return Vue.http[method](endpoint);
+    return Vue.http[method](endpoint, data);
   },
   poll(data = {}) {
     const endpoint = data.notesData.notesPath;
-- 
cgit v1.2.1