Fix: swap manual refetch to disable refetchOnWindowFocus instead

isomerpages · Oct 18, 2022 · 01641dd · 01641dd
1 parent e28c2c4
commit 01641dd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/src/hooks/commentsHooks/useGetComments.ts b/src/hooks/commentsHooks/useGetComments.ts
@@ -16,7 +16,7 @@ export const useGetComments = ({
     () => CommentsService.getComments({ siteName, requestId }),
     {
       retry: false,
-      enabled: false, // Manually triggered
+      refetchOnWindowFocus: false,
     }
   )
 }
diff --git a/src/layouts/ReviewRequest/components/Comments/CommentsDrawer.tsx b/src/layouts/ReviewRequest/components/Comments/CommentsDrawer.tsx
@@ -76,11 +76,7 @@ export const CommentsDrawer = ({ siteName, requestId }: CommentProps) => {
     onClose: onCommentsClose,
   } = useDisclosure()
 
-  const {
-    data: commentsData,
-    isLoading: isCommentsLoading,
-    refetch: refetchRecentNotificationData,
-  } = useGetComments({
+  const { data: commentsData, isLoading: isCommentsLoading } = useGetComments({
     siteName,
     requestId,
   })
@@ -95,11 +91,6 @@ export const CommentsDrawer = ({ siteName, requestId }: CommentProps) => {
     await updateNotifications({ siteName, requestId, message: comment })
   }
 
-  useEffect(() => {
-    refetchRecentNotificationData()
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [])
-
   return (
     <>
       <IconButton