abort current request if we're changing what we're going to display

Athou · Jul 4, 2024 · ec7e97e · ec7e97e
1 parent d4c9bd1
commit ec7e97e
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/commafeed-client/src/pages/app/FeedEntriesPage.tsx b/commafeed-client/src/pages/app/FeedEntriesPage.tsx
@@ -64,7 +64,7 @@ export function FeedEntriesPage(props: FeedEntriesPageProps) {
 
     // biome-ignore lint/correctness/useExhaustiveDependencies: we subscribe to state.timestamp because we want to reload entries even if the props are the same
     useEffect(() => {
-        dispatch(
+        const promise = dispatch(
             loadEntries({
                 source: {
                     type: props.sourceType,
@@ -73,6 +73,7 @@ export function FeedEntriesPage(props: FeedEntriesPageProps) {
                 clearSearch: true,
             })
         )
+        return () => promise.abort()
     }, [dispatch, props.sourceType, id, location.state?.timestamp])
 
     const noSubscriptions = rootCategory && flattenCategoryTree(rootCategory).every(c => c.feeds.length === 0)