From fdfb87fe6763701901384d21c02335530f8e2518 Mon Sep 17 00:00:00 2001 From: ctrpsrf Date: Thu, 25 Apr 2024 02:57:23 +1000 Subject: [PATCH] fix problem eth_estimateGas replace the condition when preparing a transaction build_transaction prepare tx with params like gas/maxFeePerGas/maxPriorityFeePerGas --- uniswap/uniswap.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/uniswap/uniswap.py b/uniswap/uniswap.py index 5edbc85..1bc11d7 100644 --- a/uniswap/uniswap.py +++ b/uniswap/uniswap.py @@ -1436,7 +1436,7 @@ def _build_and_send_tx( tx_params = self._get_tx_params() transaction = function.build_transaction(tx_params) - if "gas" not in tx_params: + if "gas" not in transaction: # `use_estimate_gas` needs to be True for networks like Arbitrum (can't assume 250000 gas), # but it breaks tests for unknown reasons because estimate_gas takes forever on some tx's. # Maybe an issue with ganache? (got GC warnings once...) pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy