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

Better display of "Transaction Builder" transactions. #3843

Open
drortirosh opened this issue May 2, 2022 · 2 comments
Open

Better display of "Transaction Builder" transactions. #3843

drortirosh opened this issue May 2, 2022 · 2 comments

Comments

@drortirosh
Copy link

Currently, transactions created by the popular "Transaction Builder" are very hard to understand:

  • addresses are shown only as hex, and don't get expanded to their addressbook values
  • "data" is show in hex, instead of getting expanded as method calls.
  • This means that transactions created by the "builder" are harder to verify by co-signers (before execution) and harder to understand from history (after execution

suggestion:

Each "action" in the transaction builder should be decoded just lkie every "top-level" transaction on the safe.

@katspaugh
Copy link
Member

Thanks @drortirosh, good points!

@dasanra is this backend or frontend?

@drortirosh
Copy link
Author

With the new UI, Such transaction can be created using the "CSV Airdrop" application.
After submitting such batched transaction, the transaction should be approved by other safe owners.
Below is sample view.
As you can see, the "Token" part is very hard to review

  1. the token is displayed with address only, instead of showing its symbol/description.
  2. the receipient address of ETH is expanded from the addressbook. but the recipient of a token transfer is left as hex string
  3. the value is left as uint256, instead of using the token's decimals values.
    These are all small UI changes, but would greatly improve the usability of the transaction builder tool (note that the "value" translation is related (to token value, but AFAIK, this is the Webpack 4 #1 use-case of batched transactions..)

image

In general, the display processing of each transaction in a batch should be exactly as for a single transaction called directly by "executeTransaction"

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

No branches or pull requests

2 participants