fix: remove extra isSuccess check

OnlyJousting · OnlyJousting · commit 4e913c9c88e3 · 2025-12-15T06:13:05.000+02:00
diff --git a/apps/main/src/dex/components/PageCreatePool/hooks/useAutoDetectErc4626.ts b/apps/main/src/dex/components/PageCreatePool/hooks/useAutoDetectErc4626.ts
@@ -26,7 +26,7 @@ export const useAutoDetectErc4626 = ({ tokenId, address }: UseAutoDetectErc4626P
     }
 
     // Only auto-set ngAssetType on first successful detection (don't override user's choice)
-    if (isErc4626 && isSuccess && !statusAlreadySet) {
+    if (isErc4626 && !statusAlreadySet) {
       updateNgAssetType(tokenId, NG_ASSET_TYPE.ERC4626)
     }
   }, [tokenId, isErc4626, isSuccess, isLoading, error, updateNgAssetType, updateTokenErc4626Status, statusAlreadySet])

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ export const useAutoDetectErc4626 = ({ tokenId, address }: UseAutoDetectErc4626P`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`// Only auto-set ngAssetType on first successful detection (don't override user's choice)`
`29`		`- if (isErc4626 && isSuccess && !statusAlreadySet) {`
	`29`	`+ if (isErc4626 && !statusAlreadySet) {`
`30`	`30`	`updateNgAssetType(tokenId, NG_ASSET_TYPE.ERC4626)`
`31`	`31`	`}`
`32`	`32`	`}, [tokenId, isErc4626, isSuccess, isLoading, error, updateNgAssetType, updateTokenErc4626Status, statusAlreadySet])`