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

Successfully executed transactions are not cleared #38

Open
douglasqian opened this issue Jul 26, 2022 · 1 comment
Open

Successfully executed transactions are not cleared #38

douglasqian opened this issue Jul 26, 2022 · 1 comment

Comments

@douglasqian
Copy link

douglasqian commented Jul 26, 2022

Description

After transactions are successfully executed, the UI's listing is not updated. The transaction is still displayed as an item that's pending execution, and when you try to execute it again it will fail (as expected). But only then does the UI update.

Environment

  • Browser: Chrome
  • Wallet: MetaMask
  • Safe: 0xCe3Bd9Bb075Bdb1bc999349fc045ba021B774C13 (but should be easily reproducible on any)
  • Environment: production (Celo mainnet)

Steps to reproduce

  • Set up a multisig with 2 owners
  • Create "send funds" transaction
  • Sign from both owner's wallets and execute transaction
  • Confirm on block explorer for GnosisProxy that transaction was confirmed
  • Confirm that UI still has transaction (shown as pending execution)
  • Attempt to execute transaction again (might have to manually set gas limit because execution preview fails)
  • Confirm that UI now clears the transaction
  • Confirm on block explorer that failed transaction shows up

Expected result

Transactions that are executed are cleared automatically in UI

Obtained result

Transactions that are executed stay in UI's transaction listing and are not cleared until you try to execute the transaction again (at which it will fail again!)

Screenshots

https://www.loom.com/share/8d8287dd9ec04937a11691f647a33802

@vs77bb
Copy link

vs77bb commented Jul 28, 2022

I also have this same issue. Link here to my Celo Safe, if helpful.

https://www.loom.com/share/1319ee46e5d84cd9b40c2791fc285d68

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

No branches or pull requests

2 participants