Remove state allReviewers from reply dialog allReviewers is simply a copy of the existing state reviewers. Release-Notes: skip Google-bug-id: b/236921879 Change-Id: Ia6a9fdf7119263850ed5140076a657c162fccf3c
diff --git a/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.ts b/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.ts index 144d81d..bfd92d6 100644 --- a/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.ts +++ b/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.ts
@@ -321,9 +321,6 @@ @state() isResolvedPatchsetLevelComment = true; - @state() - allReviewers: (AccountInfo | GroupInfo)[] = []; - private readonly restApiService: RestApiService = getAppContext().restApiService; @@ -663,9 +660,6 @@ if (changedProperties.has('draftCommentThreads')) { this.handleHeightChanged(); } - if (changedProperties.has('reviewers')) { - this.computeAllReviewers(); - } if (changedProperties.has('sendDisabled')) { this.sendDisabledChanged(); } @@ -704,7 +698,7 @@ name="change" .value=${this.change} ></gr-endpoint-param> - <gr-endpoint-param name="reviewers" .value=${this.allReviewers}> + <gr-endpoint-param name="reviewers" .value=${[...this.reviewers]}> </gr-endpoint-param> ${this.renderReviewerList()} <gr-endpoint-slot name="below"></gr-endpoint-slot> @@ -2108,10 +2102,6 @@ } this.reporting.reportInteraction('attention-set-actions', {actions}); } - - computeAllReviewers() { - this.allReviewers = [...this.reviewers]; - } } declare global {