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

Warn users if submitting transaction when several tx already pending #5126

Closed
bdresser opened this issue Aug 22, 2018 · 3 comments
Closed

Warn users if submitting transaction when several tx already pending #5126

bdresser opened this issue Aug 22, 2018 · 3 comments
Labels

Comments

@bdresser
Copy link
Contributor

What problem are you trying to solve?
Users often submit multiple transactions in a row and are confused / frustrated when they don't confirm, especially if they submit a transaction with high gas while something else is already pending. Many folks don't realize tx need to be processed in nonce order. See #5113 and MetaMask/Design#25 (comment)

Describe the solution you'd like
When a user has several pending tx and is about to confirm another tx, we could show a dismiss-able warning that says something like.

You have X pending transactions that must be confirmed before this transaction will confirm. If you'd like to speed up those transactions, click here.
Open to discussion whether we should show this warning if the user has as few as 1 pending tx, or if we should only show it if the user has several (>3) backed up.

@danjm
Copy link
Contributor

danjm commented Jul 15, 2019

Some further suggestions/details:

The warning can look like the cancel modal (but with only title, text and buttons; not the highlighted eth amount):

Screenshot from 2019-03-13 14-03-25

Clicking the yes option should take the user to the main screen with the speed up sidebar opened as is done here:

Screenshot from 2019-03-13 14-02-25

This should be opened for the blocking transaction. That transaction should be highlighted in some way so that the user can know which transaction they are speeding up.

@graemeblackwood
Copy link

I'd like to upvote this – it's even more critical now with gas prices the way they are.

@bschorchit
Copy link

Closing this as we now have:
Screen Shot 2022-06-13 at 18 32 22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants