-
Notifications
You must be signed in to change notification settings - Fork 362
Display tx with 1 owner in queue if direct execution failed. #1956
Comments
Just to be specific: you mean when the tx reverts, right? If the safe tx fails (so the ethereum tx is still a success) then you would see it in the history. |
What do we even expect in such a case? Should there be a tx in the queue with 0/1 signatures? |
I think expected behavior there should be: Show 0/1 or 1/1 signatures and the user can just re-execute. |
With the introduction of the deep link view this particular issue is now different: Now the tx is created and is shown in its deep link view. We should discuss what to do after the tx fails, so it doesn't sit there in the deep link view until the user decides to exit that page. |
Env
https://safe-team.dev.gnosisdev.com/app/
Steps
Create a safe with 1 owner
Create a tx and give it low gas on purpose (to force the tx's failure)
Sign and execute the tx
The system will tell you that the tx failed
Result
Nothing is shown in the Queue or History tab
Expected
The tx to be shown as a tx on "awaiting for execution" tx on the queue tab
The text was updated successfully, but these errors were encountered: