MRWidget: Only show remove approval button for merged MR.

5cc1abf9 · Fatih Acet · ffaccec7 · 5cc1abf9
Commit 5cc1abf9 authored May 19, 2017 by Fatih Acet
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

app/assets/javascripts/vue_merge_request_widget/ee/components/approvals/approvals_footer.js ...equest_widget/ee/components/approvals/approvals_footer.js +2 -1

No files found.
--- a/app/assets/javascripts/vue_merge_request_widget/ee/components/approvals/approvals_footer.js
+++ b/app/assets/javascripts/vue_merge_request_widget/ee/components/approvals/approvals_footer.js
@@ -46,7 +46,8 @@ export default {
  },
  computed: {
    showUnapproveButton() {
-      return this.userHasApproved && !this.userCanApprove;
+      const isMerged = this.mr.state === 'merged';
+      return this.userHasApproved && !this.userCanApprove && !isMerged;
    },
  },
  methods: {