fix(trends): placeholder data only if same source

dataesr · Jan 31, 2025 · e965646 · e965646
1 parent 67a21f8
commit e965646
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/client/src/pages/trends/components/ranking/index.tsx b/client/src/pages/trends/components/ranking/index.tsx
@@ -20,8 +20,8 @@ function TrendsRankingItems() {
   return (
     <Container fluid>
       <div className="fr-accordions-group">
-        {items.map((item, index) => (
-          <TrendsRankingItem key={`${index}-${includes}`} item={item} />
+        {items.map((item) => (
+          <TrendsRankingItem key={item.id} item={item} />
         ))}
       </div>
       {isFetching && <BaseSkeleton height="500px" />}

diff --git a/client/src/pages/trends/hooks/useTrends.ts b/client/src/pages/trends/hooks/useTrends.ts
@@ -35,7 +35,8 @@ export default function useTrends() {
         normalized: normalized,
         includes: includes,
       }),
-    placeholderData: (prev) => prev,
+    placeholderData: (previousData, previousQuery) =>
+      previousQuery?.queryKey && previousQuery.queryKey[1] === currentSource ? previousData : undefined,
   })
 
   const values = useMemo(() => {