Commit 648e4e02 authored by Justin Domingue's avatar Justin Domingue Committed by GitHub

ignore v3 state when only v2 only (#2383)

parent 9bc00cca
...@@ -181,7 +181,8 @@ export function useDerivedSwapInfo(toggledVersion: Version | undefined): { ...@@ -181,7 +181,8 @@ export function useDerivedSwapInfo(toggledVersion: Version | undefined): {
const isV2TradeBetter = useMemo(() => { const isV2TradeBetter = useMemo(() => {
try { try {
// avoids comparing trades when V3Trade is not in a ready state. // avoids comparing trades when V3Trade is not in a ready state.
return [V3TradeState.VALID, V3TradeState.SYNCING, V3TradeState.NO_ROUTE_FOUND].includes(v3Trade.state) return toggledVersion === Version.v2 ||
[V3TradeState.VALID, V3TradeState.SYNCING, V3TradeState.NO_ROUTE_FOUND].includes(v3Trade.state)
? isTradeBetter(v3Trade.trade, v2Trade, TWO_PERCENT) ? isTradeBetter(v3Trade.trade, v2Trade, TWO_PERCENT)
: undefined : undefined
} catch (e) { } catch (e) {
...@@ -189,7 +190,7 @@ export function useDerivedSwapInfo(toggledVersion: Version | undefined): { ...@@ -189,7 +190,7 @@ export function useDerivedSwapInfo(toggledVersion: Version | undefined): {
// inputs/ouputs than v2 // inputs/ouputs than v2
return undefined return undefined
} }
}, [v2Trade, v3Trade]) }, [toggledVersion, v2Trade, v3Trade.state, v3Trade.trade])
const bestTrade = isV2TradeBetter == undefined ? undefined : isV2TradeBetter ? v2Trade : v3Trade.trade const bestTrade = isV2TradeBetter == undefined ? undefined : isV2TradeBetter ? v2Trade : v3Trade.trade
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment