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

2440/uni merge: gas amount always shows 37 #350

Closed
anxolin opened this issue Apr 2, 2022 · 2 comments
Closed

2440/uni merge: gas amount always shows 37 #350

anxolin opened this issue Apr 2, 2022 · 2 comments
Labels
app:CowSwap CowSwap app Bug Something isn't working Low Severity indicator for defects. It won't cause any major break-down of the system

Comments

@anxolin
Copy link
Contributor

anxolin commented Apr 2, 2022

Related to #2536

  1. Open the app
  2. Connect to the Mainnet
  3. Check the gas price at the bottom of the page

AR: always shows 37

ER: should show gas amount on sending TX on L1 from https://etherscan.io/gastracker

Original issue gnosis/cowswap#2640 by @elena-zh

@anxolin anxolin added Low Severity indicator for defects. It won't cause any major break-down of the system app:CowSwap CowSwap app Bug Something isn't working labels Apr 2, 2022
This was referenced Apr 4, 2022
@alfetopito
Copy link
Collaborator

Couple of points regarding the gas price:

  1. Not very precise, as it takes hours to update and might not be relevant for current gas spikes
  2. It's fetched from an onchain contract, and that's why I believe it's not updated very often
  3. Is only available on mainnet. Doesn't make much sense on other networks anyway

So, we either change how it's updated or it's a won't fix

@W3stside
Copy link
Contributor

this is also the case in uniswap, low prio

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app:CowSwap CowSwap app Bug Something isn't working Low Severity indicator for defects. It won't cause any major break-down of the system
Projects
None yet
Development

No branches or pull requests

3 participants