diff options
author | Phil Hughes <me@iamphill.com> | 2017-04-15 15:13:53 +0000 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2017-04-15 15:13:53 +0000 |
commit | ae52d1a4b82cd86130c92795390fc3d9664d180b (patch) | |
tree | 9c025f5ad9416e15eb44cce602f76b164c4ac3c1 | |
parent | e953932d1481b013cda68c04f494e4eb897f9cc1 (diff) | |
parent | ef443a99eea066c644922ea66e5d01ab1ca66743 (diff) | |
download | gitlab-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.js | 8 |
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); } }); |