Skip to content

Commit 12e2558

Browse files
committed
fix: update invalidate revoke query
1 parent 5335f70 commit 12e2558

3 files changed

Lines changed: 6 additions & 12 deletions

File tree

src/authz-module/data/hooks.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ export const useRevokeUserRoles = () => {
102102
data: RevokeUserRolesRequest
103103
}) => revokeUserRoles(data),
104104
onSettled: (_data, _error, { data: { scope } }) => {
105-
queryClient.invalidateQueries({ queryKey: authzQueryKeys.teamMembers(scope) });
105+
queryClient.invalidateQueries({ queryKey: authzQueryKeys.teamMembersAll(scope) });
106106
},
107107
});
108108
};

src/authz-module/index.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ const AuthZModule = () => (
2424
</Routes>
2525
</Suspense>
2626
</ToastManagerProvider>
27-
2827
</ErrorBoundary>
2928
)}
3029
</QueryErrorResetBoundary>

src/authz-module/libraries-manager/LibrariesUserManager.tsx

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ const LibrariesUserManager = () => {
2929
if (!canManageTeam) {
3030
navigate(teamMembersPath);
3131
}
32-
// eslint-disable-next-line react-hooks/exhaustive-deps
32+
// eslint-disable-next-line react-hooks/exhaustive-deps
3333
}, [canManageTeam]);
3434

3535
const { data: library } = useLibrary(libraryId);
@@ -49,7 +49,9 @@ const LibrariesUserManager = () => {
4949
const [showConfirmDeletionModal, setShowConfirmDeletionModal] = useState(false);
5050
const { handleShowToast, handleDiscardToast } = useToastManager();
5151

52-
const { data: teamMember, isLoading: isLoadingTeamMember, isFetching: isFetchingMember } = useTeamMembers(libraryId, querySettings);
52+
const {
53+
data: teamMember, isLoading: isLoadingTeamMember, isFetching: isFetchingMember,
54+
} = useTeamMembers(libraryId, querySettings);
5355
const user = teamMember?.results?.find(member => member.username === username);
5456

5557
const userRoles = useMemo(() => {
@@ -69,7 +71,7 @@ const LibrariesUserManager = () => {
6971
navigate(teamMembersPath);
7072
}
7173
}
72-
// eslint-disable-next-line react-hooks/exhaustive-deps
74+
// eslint-disable-next-line react-hooks/exhaustive-deps
7375
}, [isFetchingMember, isLoadingTeamMember, user?.username]);
7476

7577
const handleCloseConfirmDeletionModal = () => {
@@ -115,13 +117,6 @@ const LibrariesUserManager = () => {
115117
});
116118
};
117119

118-
useEffect(() => {
119-
if (!isLoadingTeamMember && !userRoles.length) {
120-
navigate(teamMembersPath);
121-
}
122-
// eslint-disable-next-line react-hooks/exhaustive-deps
123-
}, [userRoles.length]);
124-
125120
return (
126121
<div className="authz-libraries">
127122
<ConfirmDeletionModal

0 commit comments

Comments
 (0)