Skip to content
Snippets Groups Projects
Unverified Commit fe9a7c43 authored by Tasso Evangelista's avatar Tasso Evangelista Committed by GitHub
Browse files

fix: Avoid `useEffect` loop (#28699)

parent fcb25518
No related branches found
No related tags found
No related merge requests found
......@@ -8,7 +8,7 @@ import { useDismissUserBannerMutation } from './useDismissUserBannerMutation';
export const useUserBanners = () => {
const user = useUser();
const dismissBannerMutation = useDismissUserBannerMutation();
const { mutate: dismissUserBanner } = useDismissUserBannerMutation();
useEffect(() => {
if (!user?.banners || Object.keys(user.banners).length === 0) {
......@@ -37,8 +37,8 @@ export const useUserBanners = () => {
}
},
onClose() {
dismissBannerMutation.mutate({ id: firstBanner.id });
dismissUserBanner({ id: firstBanner.id });
},
});
}, [dismissBannerMutation, user]);
}, [dismissUserBanner, user?.banners]);
};
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment