tatumio / tatum-js

🚀 Tatum SDK: A 💪 powerful, 🌟 feature-rich TypeScript/JavaScript 📚 library that streamlines the 🛠️ development of 🌐 blockchain applications.
https://docs.tatum.io
MIT License
360 stars 110 forks source link

Skip gas estimation if already present (ethereum) #1081

Closed korkoshko closed 5 months ago

korkoshko commented 5 months ago

Description

We don't need call estimateGas if the gas property already present in the serialized transaction.

Type of change

Checklist:

korkoshko commented 5 months ago

@Hathoriel could you please review this?

Hathoriel commented 5 months ago

Hello @korkoshko Thank you for your PR. It looks perfect, but can you please upgrade the package.json version? After you do it, I will approve and merge this pull request. Thank you

korkoshko commented 5 months ago

@Hathoriel Done. Thanks!