chore: although unfortunate, it is possible for a user to be in no orgs

Emyrk · Emyrk · commit b8899d59d10d · 2024-07-19T16:24:55.000-05:00
diff --git a/coderd/database/dbmem/dbmem.go b/coderd/database/dbmem/dbmem.go
@@ -3006,9 +3006,6 @@ func (q *FakeQuerier) GetOrganizationIDsByMemberIDs(_ context.Context, ids []uui
 			OrganizationIDs: userOrganizationIDs,
 		})
 	}
-	if len(getOrganizationIDsByMemberIDRows) == 0 {
-		return nil, sql.ErrNoRows
-	}
 	return getOrganizationIDsByMemberIDRows, nil
 }
 
diff --git a/coderd/users.go b/coderd/users.go
@@ -1293,9 +1293,12 @@ func userOrganizationIDs(ctx context.Context, api *API, user database.User) ([]u
 	if err != nil {
 		return []uuid.UUID{}, err
 	}
+
+	// If you are in no orgs, then return an empty list.
 	if len(organizationIDsByMemberIDsRows) == 0 {
-		return []uuid.UUID{}, xerrors.Errorf("user %q must be a member of at least one organization", user.Email)
+		return []uuid.UUID{}, nil
 	}
+
 	member := organizationIDsByMemberIDsRows[0]
 	return member.OrganizationIDs, nil
 }

Original file line number	Diff line number	Diff line change
`@@ -3006,9 +3006,6 @@ func (q *FakeQuerier) GetOrganizationIDsByMemberIDs(_ context.Context, ids []uui`
`3006`	`3006`	`OrganizationIDs: userOrganizationIDs,`
`3007`	`3007`	`})`
`3008`	`3008`	`}`
`3009`		`- if len(getOrganizationIDsByMemberIDRows) == 0 {`
`3010`		`- return nil, sql.ErrNoRows`
`3011`		`- }`
`3012`	`3009`	`return getOrganizationIDsByMemberIDRows, nil`
`3013`	`3010`	`}`
`3014`	`3011`
Original file line number	Diff line number	Diff line change
`@@ -1293,9 +1293,12 @@ func userOrganizationIDs(ctx context.Context, api *API, user database.User) ([]u`
`1293`	`1293`	`if err != nil {`
`1294`	`1294`	`return []uuid.UUID{}, err`
`1295`	`1295`	`}`
	`1296`	`+`
	`1297`	`+ // If you are in no orgs, then return an empty list.`
`1296`	`1298`	`if len(organizationIDsByMemberIDsRows) == 0 {`
`1297`		`- return []uuid.UUID{}, xerrors.Errorf("user %q must be a member of at least one organization", user.Email)`
	`1299`	`+ return []uuid.UUID{}, nil`
`1298`	`1300`	`}`
	`1301`	`+`
`1299`	`1302`	`member := organizationIDsByMemberIDsRows[0]`
`1300`	`1303`	`return member.OrganizationIDs, nil`
`1301`	`1304`	`}`