Closed henrio123 closed 4 months ago
vercel[bot]
commented
4 months ago The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| sommelier-web | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jun 4, 2024 7:05pm |
coderabbitai[bot]
commented
4 months ago The recent changes to src/utils/estimateGasLimit.ts involve increasing the default gas limit margin from 10% to 30% and adjusting the padding values for retries. These updates aim to provide a larger buffer to ensure successful transactions, enhancing reliability.
| Files | Change Summary |
|---|---|
src/utils/estimateGasLimit.ts |
Increased default gas limit margin from 10% to 30%; modified PAD array values for larger buffer on retries. |
In the world of gas, where limits rise and fall,
We’ve added a buffer, a safety net for all.
With margins now wider, transactions will soar,
Ensuring your journey, from start to end, is more secure.
So fear not the limits, for we've got your back,
In the code's embrace, you'll stay on track. 🚀🐇
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Request: we think the front end might not be estimating gas with enough of a buffer for RYBTC
had a few txs revert because of insufficient gas
Solution: Increased the default margin in the gasLimitMargin function to 30%. Updated the PAD values to start at 1.25 and go up to 1.85 to provide a larger buffer during retries. This should ensure that your transactions have a higher chance of success by accounting for the gas used in both successful and failed transactions plus an additional buffer.
Summary by CodeRabbit