refactor(hooks): don't set empty result before throwing Next navigati…

…on errors
TheEdoRan · Jun 12, 2024 · c3a4b40 · c3a4b40
1 parent ec95a10
commit c3a4b40
Showing 1 changed file with 0 additions and 4 deletions.
diff --git a/packages/next-safe-action/src/hooks.ts b/packages/next-safe-action/src/hooks.ts
@@ -156,7 +156,6 @@ export const useAction = <
 					.then((res) => setResult(res ?? EMPTY_HOOK_RESULT))
 					.catch((e) => {
 						if (isRedirectError(e) || isNotFoundError(e)) {
-							setResult(EMPTY_HOOK_RESULT);
 							throw e;
 						}
 
@@ -180,7 +179,6 @@ export const useAction = <
 				startTransition(() => {
 					safeActionFn(input as S extends Schema ? InferIn<S> : undefined)
 						.then((res) => {
-							setResult(res ?? EMPTY_HOOK_RESULT);
 							resolve(res);
 						})
 						.catch((e) => {
@@ -271,7 +269,6 @@ export const useOptimisticAction = <
 					.then((res) => setResult(res ?? EMPTY_HOOK_RESULT))
 					.catch((e) => {
 						if (isRedirectError(e) || isNotFoundError(e)) {
-							setResult(EMPTY_HOOK_RESULT);
 							throw e;
 						}
 
@@ -296,7 +293,6 @@ export const useOptimisticAction = <
 					setOptimisticValue(input as S extends Schema ? InferIn<S> : undefined);
 					safeActionFn(input as S extends Schema ? InferIn<S> : undefined)
 						.then((res) => {
-							setResult(res ?? EMPTY_HOOK_RESULT);
 							resolve(res);
 						})
 						.catch((e) => {