Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

Don't allow submit a tx if the safe nonce was already used #4012

Open
liliya-soroka opened this issue Jul 5, 2022 · 0 comments
Open

Don't allow submit a tx if the safe nonce was already used #4012

liliya-soroka opened this issue Jul 5, 2022 · 0 comments
Labels
Bug 🐛 Something isn't working Minor Needs to be fixed within the next 1-3 public releases.

Comments

@liliya-soroka
Copy link
Member

liliya-soroka commented Jul 5, 2022 •
edited
Loading

Description

Don't allow submit a tx if the safe nonce was already used

Environment

  • Browser: Chrome
  • Wallet: MetaMask
  • Chain: Rinkeby

Steps to reproduce

  1. Go to the send Fund or create tx in the safe app
  2. open advanced parameters and set up safe nonce to already existed nonce in the history
  3. click submit button
    Current result:
  4. For safe apps: the tx stuckes in the submitting state
  5. for send funds : create tx model is closed without any error and without the tx creation. The error appears only in the console ( see screenshot)

Expected result

The submit button should not be available if the safe nonce was already used

Screenshots

image

image
@liliya-soroka liliya-soroka added Bug 🐛 Something isn't working Minor Needs to be fixed within the next 1-3 public releases. labels Jul 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Bug 🐛 Something isn't working Minor Needs to be fixed within the next 1-3 public releases.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@liliya-soroka