Skip to content

Commit 0c47e3b

Browse files
committed
fix: Delete was wiping all groups, not just the single user's groups
1 parent c207351 commit 0c47e3b

File tree

2 files changed

+8
-20
lines changed

2 files changed

+8
-20
lines changed

coderd/database/queries.sql.go

Lines changed: 5 additions & 11 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

coderd/database/queries/groupmembers.sql

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -35,16 +35,10 @@ FROM
3535

3636
-- name: DeleteGroupMembersByOrgAndUser :exec
3737
DELETE FROM
38-
group_members
39-
USING
40-
group_members AS gm
41-
LEFT JOIN
42-
groups
43-
ON
44-
groups.id = gm.group_id
38+
group_members
4539
WHERE
46-
groups.organization_id = @organization_id AND
47-
gm.user_id = @user_id;
40+
group_members.user_id = @user_id
41+
AND group_id = ANY(SELECT id FROM groups WHERE organization_id = @organization_id);
4842

4943
-- name: InsertGroupMember :exec
5044
INSERT INTO

0 commit comments

Comments
 (0)