Skip to content
This repository was archived by the owner on Jun 24, 2022. It is now read-only.

Use our own Gas Estimation #708

Open
anxolin opened this issue May 28, 2021 · 4 comments
Open

Use our own Gas Estimation #708

anxolin opened this issue May 28, 2021 · 4 comments
Assignees
Labels
Enhancement New feature or request wontfix Stale issue

Comments

@anxolin
Copy link
Contributor

anxolin commented May 28, 2021

From last test session

If possible to suggest gas price we should suggest a higher one as default is too low (e.g. on Opeara Mobile)

Suggestion
Add the average gas price in all Ethereum transactions instead of relying in the Wallet.

Nice to have
We could suggest a gas price. Some UIs also allow you to choose slow/medium/fast which is great.
This could even be done together with this proposal #571 so we, for Ethereum TX we present a nice UX where user selects his gas price. For meta-transactions we could use the gas price to show how much you are saving in gas.

@anxolin anxolin added the Enhancement New feature or request label May 28, 2021
@anxolin
Copy link
Contributor Author

anxolin commented Aug 11, 2021

I think Metamask not always estimate right. We could improve the UX by selecting a better gas.

Also, maybe some UI helps, showing current gasprices? it was suggested.
For wrapping/unwrapping/approving ideally we would let the user choose between FAST/AVERAGE/SLOW

gasnow is quite reliable afait

@anxolin
Copy link
Contributor Author

anxolin commented Aug 16, 2021

@nenadV91 I just assigned you a task that I think can be nice to start with:

David could help u if u need to find yourself around. It would be to make sure we add a gasPrice to all Ethereum transactions (wrap/unwrap/approve).

Basically:
Get an estimation from some external source: i.e. https://www.gasnow.org . In case of error, we can have some fallback logic
use that to fill the gasPrice field in the transactions

Note that we already have one way to get the gasprice right now, using another API (Safe relayer). So u can try to do something similar for this other service.

Also, note orders are not Ethereum transactions, so no need to add gasPRice there.

@anxolin
Copy link
Contributor Author

anxolin commented Oct 7, 2021

This issue is outdated! specially because gasnow is deprecated now, but also because they don't implement EIP1559 gas estimations

If we use an estimation service, we should try to use one supporting the EIP.

One that looks great we could use: https://www.blocknative.com/gas-estimator

@stale
Copy link

stale bot commented Mar 30, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you think it shouldn't be closed, speak now or forever hold your peace.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Enhancement New feature or request wontfix Stale issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants