safe-react-apps icon indicating copy to clipboard operation
safe-react-apps copied to clipboard

[Drain Account] Enhance transaction summary decoding

Open JagoFigueroa opened this issue 3 years ago • 2 comments

Description

On Drain review screen:

  1. specify that the address we are sending the funds to is a smart contract address when more than one asset is drained. A drain is a critical operation so seeing an address different to the one specified recipient itself can be scary for users.

  2. Change the Send 0 eth to into something more generic, like send all/selected assets to:

Screenshot 2021-11-02 at 10 23 02

JagoFigueroa avatar Nov 02 '21 09:11 JagoFigueroa

this should be a part of https://github.com/gnosis/safe-react-apps

mmv08 avatar Nov 03 '21 14:11 mmv08

Removing a couple of enhancements from the list as yago already did some enhancements in pr https://github.com/gnosis/safe-react-apps/pull/173 + adding a new possible one :)

JagoFigueroa avatar Nov 04 '21 11:11 JagoFigueroa