Merge branch 'stable-3.2' into stable-3.3
* stable-3.2:
UI: Don't suggest unsupported system groups
Document that configured groups must be visible to uploader
Bazel: Remove unnecessary plugin_name parameter
Change-Id: I0781085e75a0145b49d4d86a4ea061518d8da410
diff --git a/BUILD b/BUILD
index b0ef90f..6f5f9dc 100644
--- a/BUILD
+++ b/BUILD
@@ -37,7 +37,6 @@
polygerrit_plugin(
name = "rv-reviewers",
app = "reviewers-bundle.js",
- plugin_name = "rv-reviewers",
)
rollup_bundle(
diff --git a/rv-reviewers/rv-reviewer.js b/rv-reviewers/rv-reviewer.js
index 6145013..338b428 100644
--- a/rv-reviewers/rv-reviewer.js
+++ b/rv-reviewers/rv-reviewer.js
@@ -16,6 +16,8 @@
*/
import {htmlTemplate} from './rv-reviewer_html.js';
+const SYSTEM_GROUP_PREFIX = 'global%3A';
+
class RvReviewer extends Polymer.Element {
/** @returns {string} name of the component */
static get is() { return 'rv-reviewer'; }
@@ -104,7 +106,9 @@
if (!groups) { return []; }
const groupSuggestions = [];
for (const key in groups) {
- if (!groups.hasOwnProperty(key) || key.startsWith('user/')) {
+ if (!groups.hasOwnProperty(key) ||
+ groups[key].id.startsWith(SYSTEM_GROUP_PREFIX) ||
+ key.startsWith('user/')) {
continue;
}
groupSuggestions.push({
diff --git a/src/main/resources/Documentation/config.md b/src/main/resources/Documentation/config.md
index 1b51181..173f51a 100644
--- a/src/main/resources/Documentation/config.md
+++ b/src/main/resources/Documentation/config.md
@@ -51,7 +51,10 @@
filter.\<filter\>.reviewer
: An account or a group name. Must be an exact match (case sensitive) with the
account's email address or username, or the group name. Multiple `reviewer`
- occurrences are allowed.
+ occurrences are allowed.\
+ **NOTE**: *Reviewers are added in the context of the uploader which means
+ that if a group is configured it needs to be visible to the uploader for
+ reviewers to be added.*
filter.\<filter\>.cc
: An account or a group name. Must be an exact match (case sensitive) with the
