feat: remove unnecessary keepPreviousData

graasp · Sep 17, 2024 · 2f4b188 · 2f4b188
1 parent f385e8d
commit 2f4b188
Showing 1 changed file with 2 additions and 16 deletions.
diff --git a/src/item/hooks.ts b/src/item/hooks.ts
@@ -5,11 +5,7 @@ import {
   WebsocketClient,
 } from '@graasp/sdk';
 
-import {
-  keepPreviousData,
-  useInfiniteQuery,
-  useQuery,
-} from '@tanstack/react-query';
+import { useInfiniteQuery, useQuery } from '@tanstack/react-query';
 
 import { splitRequestByIdsAndReturn } from '../api/axios.js';
 import {
@@ -163,12 +159,7 @@ const config = (
 
     useDescendants: useDescendants(queryConfig),
 
-    useItem: (
-      id?: UUID,
-      options?: {
-        keepPreviousData?: boolean;
-      },
-    ) =>
+    useItem: (id?: UUID) =>
       useQuery({
         queryKey: itemKeys.single(id).content,
         queryFn: () => {
@@ -179,11 +170,6 @@ const config = (
         },
         enabled: Boolean(id),
         ...defaultQueryOptions,
-        placeholderData: options?.keepPreviousData
-          ? // KeepPreviousData prevents unnecessary re-renders of the Player's three components
-            // when data is being fetched. This avoids flickering or visual glitches.
-            keepPreviousData
-          : undefined,
       }),
 
     useItems: (ids: UUID[]) => {