summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPhil Hughes <me@iamphill.com>2017-04-15 15:13:53 +0000
committerPhil Hughes <me@iamphill.com>2017-04-15 15:13:53 +0000
commitae52d1a4b82cd86130c92795390fc3d9664d180b (patch)
tree9c025f5ad9416e15eb44cce602f76b164c4ac3c1
parente953932d1481b013cda68c04f494e4eb897f9cc1 (diff)
parentef443a99eea066c644922ea66e5d01ab1ca66743 (diff)
downloadgitlab-ce-ae52d1a4b82cd86130c92795390fc3d9664d180b.tar.gz
Merge branch '30492-using-icon-to-mark-discussion-as-resolved-does-not-resolve-discussion' into 'master'
Resolve "Using icon to mark discussion as resolved does not resolve discussion" Closes #30492 See merge request !10652
-rw-r--r--app/assets/javascripts/diff_notes/components/resolve_btn.js8
1 files changed, 4 insertions, 4 deletions
diff --git a/app/assets/javascripts/diff_notes/components/resolve_btn.js b/app/assets/javascripts/diff_notes/components/resolve_btn.js
index 312f38ce241..dcfc40c1013 100644
--- a/app/assets/javascripts/diff_notes/components/resolve_btn.js
+++ b/app/assets/javascripts/diff_notes/components/resolve_btn.js
@@ -20,8 +20,7 @@ import Vue from 'vue';
data: function () {
return {
discussions: CommentsStore.state,
- loading: false,
- note: {},
+ loading: false
};
},
watch: {
@@ -34,6 +33,9 @@ import Vue from 'vue';
discussion: function () {
return this.discussions[this.discussionId];
},
+ note: function () {
+ return this.discussion ? this.discussion.getNote(this.noteId) : {};
+ },
buttonText: function () {
if (this.isResolved) {
return `Resolved by ${this.resolvedByName}`;
@@ -112,8 +114,6 @@ import Vue from 'vue';
authorAvatar: this.authorAvatar,
noteTruncated: this.noteTruncated,
});
-
- this.note = this.discussion.getNote(this.noteId);
}
});