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 {
