Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gasLimit in the Build Transaction response #70

Open
ramuta opened this issue Jun 27, 2021 · 0 comments
Open

gasLimit in the Build Transaction response #70

ramuta opened this issue Jun 27, 2021 · 0 comments

Comments

@ramuta
Copy link

ramuta commented Jun 27, 2021

I'm using the ParaSwap API and I noticed that the response object (in Build Transaction API call) does not include the estimated gasLimit value.

Example from your docs:

{
  "from": "0xaa013bA141Faa58F0a8FCA1FEE3bC96d44Af869e",
  "to": "0x1bD435F3C054b6e901B7b108a0ab7617C808677b",
  "chainId": 1,
  "value": "1000000000000000000",
  "data": "0xcfc0...",
  "gasPrice": "119000000000"
}

Including the gasLimit value in this response would be very beneficial, because it is hard to get it via the ethers library. I was only able to get the gas limit estimate when swapping a native coin (like ETH or MATIC on Polygon), but not when swapping ERC-20 tokens.

Is it possible to add gasLimit to the Build Transaction API response?

bogdoslavik pushed a commit to dystopia-exchange/paraswap-sdk that referenced this issue May 14, 2022
Feature/back 466 - integrate new dex `nerve` and forks with event base system
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant