Force JSON parsing of approvals API response data.

4e03161c · Bryce Johnson · 1917ad89 · 4e03161c
Commit 4e03161c authored Jan 23, 2017 by Bryce Johnson
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

app/assets/javascripts/merge_request_widget/approvals/approvals_store.js.es6 ...pts/merge_request_widget/approvals/approvals_store.js.es6 +3 -3

No files found.
--- a/app/assets/javascripts/merge_request_widget/approvals/approvals_store.js.es6
+++ b/app/assets/javascripts/merge_request_widget/approvals/approvals_store.js.es6
@@ -36,19 +36,19 @@

    fetch() {
      return this.api.fetchApprovals()
-        .then(res => this.assignToRootStore('approvals', res.data))
+        .then(res => this.assignToRootStore('approvals', res.json()))
        .then(data => this.setMergeRequestAcceptanceStatus(data.approvals_left));
    }

    approve() {
      return this.api.approveMergeRequest()
-        .then(res => this.assignToRootStore('approvals', res.data))
+        .then(res => this.assignToRootStore('approvals', res.json()))
        .then(data => this.setMergeRequestAcceptanceStatus(data.approvals_left));
    }

    unapprove() {
      return this.api.unapproveMergeRequest()
-        .then(res => this.assignToRootStore('approvals', res.data))
+        .then(res => this.assignToRootStore('approvals', res.json()))
        .then(data => this.setMergeRequestAcceptanceStatus(data.approvals_left));
    }