commit | 56f7e79153f0ee17d7ac3192d75481015e83e773 | [log] [tgz] |
---|---|---|
author | Ben Rohlfs <brohlfs@google.com> | Mon Nov 30 10:42:23 2020 +0100 |
committer | Ben Rohlfs <brohlfs@google.com> | Mon Nov 30 10:42:23 2020 +0100 |
tree | 1d4c6a24e3d46d000c4f11408a962573f52fe466 | |
parent | d36cf01b0f03eb6a555c915cc9d570e4b07d485b [diff] |
Remove requesting DETAILED_LABELS for the dashboard This is following up on change 288347. The backend was changed such that it is not required anymore for the attention set dashboard to request DETAILED_LABELS. Just LABELS are enough. Change-Id: I9e82c94fbecd3fff821e7a8e67281c3cc395751c
diff --git a/java/com/google/gerrit/httpd/raw/IndexPreloadingUtil.java b/java/com/google/gerrit/httpd/raw/IndexPreloadingUtil.java index c17cd97..f1da6b7 100644 --- a/java/com/google/gerrit/httpd/raw/IndexPreloadingUtil.java +++ b/java/com/google/gerrit/httpd/raw/IndexPreloadingUtil.java
@@ -123,9 +123,7 @@ options.add(ListChangesOption.LABELS); options.add(ListChangesOption.DETAILED_ACCOUNTS); - if (isEnabledAttentionSet(serverApi)) { - options.add(ListChangesOption.DETAILED_LABELS); - } else { + if (!isEnabledAttentionSet(serverApi)) { options.add(ListChangesOption.REVIEWED); } return ListOption.toHex(options);
diff --git a/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface.ts b/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface.ts index 0ec9aaf..5ffdfe2 100644 --- a/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface.ts +++ b/polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-interface.ts
@@ -1424,9 +1424,7 @@ ListChangesOption.LABELS, ListChangesOption.DETAILED_ACCOUNTS, ]; - if (config?.change && config.change.enable_attention_set) { - options.push(ListChangesOption.DETAILED_LABELS); - } else { + if (!config?.change?.enable_attention_set) { options.push(ListChangesOption.REVIEWED); }