commit d1b70c970a315a36e04772db2144c5a9a81703ee
author Sven Selberg <sven.selberg@axis.com> Wed Apr 06 14:15:15 2022 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Wed Apr 06 14:15:15 2022 +0000
tree 4e11c5ce2885f5db68d70cdb5b302c0ec8a3be58
parent 37f39f4c02ece259c6cd1e3885027db0248248cb
parent 07848690ac95cd508964f3e54aa24abdd7ae99d6

Merge "Document that configured groups must be visible to uploader" into stable-3.2

rv-reviewers/rv-reviewer.js

diff --git a/rv-reviewers/rv-reviewer.js b/rv-reviewers/rv-reviewer.js
index dd516c4..77500b2 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'; }
@@ -92,7 +94,8 @@
       if (!groups) { return []; }
       const groupSuggestions = [];
       for (const key in groups) {
-        if (!groups.hasOwnProperty(key)) { continue; }
+        if (!groups.hasOwnProperty(key) ||
+            groups[key].id.startsWith(SYSTEM_GROUP_PREFIX)) { continue; }
         groupSuggestions.push({
           name: key,
           value: key,