delvtech / hyperdrive-frontend

A monorepo of apps and packages for the Hyperdrive AMM.
https://hyperdrive.box
Apache License 2.0
10 stars 1 forks source link

Remove hardcoded gas limits for transactions on Mainnet #153

Closed DannyDelott closed 9 months ago

DannyDelott commented 1 year ago

All transaction for hyperdrive flows are hardcoded at 500_000 gas limit. We need to use the gas estimator with a modifier. Goerli has trouble estimating gas to we might only need it for testing. We need to validate these estimates on mainnet before coming up with the ideal solution.

DannyDelott commented 1 year ago

For local and goerli this isn't a deal breaker to have a hard-coded gas limit. On Mainnet this will likely not be an issue at all.

DannyDelott commented 1 year ago

First thing is to remove the hardcoded limits and observe if devnet still works fine without them. If not, we can implement a solution where it's hardcoded for devnet and the default estimation for mainnet.