Skip to content

Commit 44fff54

Browse files
authored
fix: avoid displaying 'everyone' group for idp group sync (#18261)
fixes #16987 Fix implemented through the coder tasks UI using Coder with Claude Code. Prompt: fix this issue, #16987
1 parent cb9c483 commit 44fff54

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

site/src/pages/OrganizationSettingsPage/IdpSyncPage/IdpGroupSyncForm.tsx

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ import { useFormik } from "formik";
3232
import { Plus, Trash, TriangleAlert } from "lucide-react";
3333
import { type FC, type KeyboardEventHandler, useId, useState } from "react";
3434
import { docs } from "utils/docs";
35+
import { isEveryoneGroup } from "utils/groups";
3536
import { isUUID } from "utils/uuid";
3637
import * as Yup from "yup";
3738
import { ExportPolicyButton } from "./ExportPolicyButton";
@@ -259,15 +260,17 @@ export const IdpGroupSyncForm: FC<IdpGroupSyncFormProps> = ({
259260
className="min-w-60 max-w-3xl"
260261
value={coderGroups}
261262
onChange={setCoderGroups}
262-
options={groups.map((group) => ({
263-
label: group.display_name || group.name,
264-
value: group.id,
265-
}))}
263+
options={groups
264+
.filter((group) => !isEveryoneGroup(group))
265+
.map((group) => ({
266+
label: group.display_name || group.name,
267+
value: group.id,
268+
}))}
266269
hidePlaceholderWhenSelected
267270
placeholder="Select group"
268271
emptyIndicator={
269272
<p className="text-center text-md text-content-primary">
270-
All groups selected
273+
No more groups to select
271274
</p>
272275
}
273276
/>

0 commit comments

Comments
 (0)