ButterflyEffect/uniswap-interface-uncensored
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: disable showing approve cost for arbitrum (#7594)

Browse Source 
disable showing approve cost for arbitrum
This commit is contained in:
Tina 2023-11-15 13:24:13 -05:00 committed by GitHub
parent 52dc441e31
commit 79e74e1d13
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/state/routing/gas.ts
View File

@ -1,5 +1,5 @@
import { MaxUint256, PERMIT2_ADDRESS } from '@uniswap/permit2-sdk' import { MaxUint256, PERMIT2_ADDRESS } from '@uniswap/permit2-sdk'
import { Currency } from '@uniswap/sdk-core' import { ChainId, Currency } from '@uniswap/sdk-core'
import ERC20_ABI from 'abis/erc20.json' import ERC20_ABI from 'abis/erc20.json'
import { Erc20, Weth } from 'abis/types' import { Erc20, Weth } from 'abis/types'
import WETH_ABI from 'abis/weth.json' import WETH_ABI from 'abis/weth.json'
@ -26,6 +26,12 @@ export async function getApproveInfo(
// If any of these arguments aren't provided, then we cannot generate approval cost info // If any of these arguments aren't provided, then we cannot generate approval cost info
if (!account || !usdCostPerGas) return { needsApprove: false } if (!account || !usdCostPerGas) return { needsApprove: false }
// routing-api under estimates gas for Arbitrum swaps so it inflates cost per gas by a lot
// so disable showing approves for Arbitrum until routing-api gives more accurate gas estimates
if (currency.chainId === ChainId.ARBITRUM_ONE || currency.chainId === ChainId.ARBITRUM_GOERLI) {
return { needsApprove: false }
}
const provider = DEPRECATED_RPC_PROVIDERS[currency.chainId as SupportedInterfaceChain] const provider = DEPRECATED_RPC_PROVIDERS[currency.chainId as SupportedInterfaceChain]
const tokenContract = getContract(currency.address, ERC20_ABI, provider) as Erc20 const tokenContract = getContract(currency.address, ERC20_ABI, provider) as Erc20