Do not send abort event in usePromise when promise has already finished

project-gauntlet · Sep 10, 2024 · 0eb1ba4 · 0eb1ba4
1 parent bff3bd8
commit 0eb1ba4
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/js/api/src/hooks.ts b/js/api/src/hooks.ts
@@ -95,6 +95,10 @@ export function usePromise<T extends (...args: any[]) => Promise<any>, R>(
             if (promise === promiseRef.current) {
                 setState({ error, isLoading: false })
 
+                if (options && options.abortable) {
+                    options.abortable.current = undefined;
+                }
+
                 options?.onError?.(error);
             }
             return
@@ -105,6 +109,10 @@ export function usePromise<T extends (...args: any[]) => Promise<any>, R>(
         if (promise === promiseRef.current) {
             setState({ data: promiseResult, isLoading: false });
 
+            if (options && options.abortable) {
+                options.abortable.current = undefined;
+            }
+
             options?.onData?.(promiseResult)
         }
     }, args || []);