Skip to content

Commit 16bbb14

Browse files
fix: make memberType required in RemoveProjectMemberDialog
Prevents silent misuse where group removals would default to 'app/user' principal type and fail. Also fix reset state to include memberType. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 89bb65e commit 16bbb14

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

web/sdk/react/views/projects/details/project-detail-page.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ export const ProjectDetailPage = ({
184184

185185
const handleRemoveMemberOpenChange = (value: boolean) => {
186186
if (!value) {
187-
setRemoveMemberState({ open: false, memberId: '' });
187+
setRemoveMemberState({ open: false, memberId: '', memberType: 'user' });
188188
refetchTeamAndMembers();
189189
} else {
190190
setRemoveMemberState(prev => ({ ...prev, open: value }));

web/sdk/react/views/projects/details/remove-project-member-dialog.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,15 @@ export interface RemoveProjectMemberDialogProps {
2222
onOpenChange?: (value: boolean) => void;
2323
projectId: string;
2424
memberId: string;
25-
memberType?: 'user' | 'group';
25+
memberType: 'user' | 'group';
2626
}
2727

2828
export const RemoveProjectMemberDialog = ({
2929
open,
3030
onOpenChange,
3131
projectId,
3232
memberId,
33-
memberType = 'user'
33+
memberType
3434
}: RemoveProjectMemberDialogProps) => {
3535
const queryClient = useQueryClient();
3636
const transport = useTransport();

0 commit comments

Comments
 (0)