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

Send Custom Transactions #123

Closed
lukasschor opened this issue Jun 20, 2019 · 0 comments
Closed

Send Custom Transactions #123

lukasschor opened this issue Jun 20, 2019 · 0 comments
Assignees
Labels
Major Needs to be fixed for immediate next public release.

Comments

@lukasschor
Copy link
Member

lukasschor commented Jun 20, 2019

What is this feature about? (1 sentence)

As a user I am able to send custom transactions in order to also perform transactions other than asset transfers with my Safe.

Why is it needed? What is the value? For whom do we build it?

This feature is requested by current Multisig users as this allows to interact with Smart contracts.

Screens

  • Add Receive & Send buttons at the top-right, same height as the identicon on the left
    ui v1 1 - 4f safes send

ui v1 1 - 4i safes send custom tx

  • Recipient addresses are validated to be valid smart contract addresses
  • Add Field "Value" (not in the current designs)
  • Show error when value AND data is zero
  • Rename "Data interface (ABI/JSON)" to "Data (hex encoded)"
  • DON'T show transaction fee

ui v1 1 - 4i safes send custom tx review

  • User needs to review transaction before submitting it.
  • Also show "Value" on review screen

** Transaction List **

  • Show "Data (hex encoded)" in the Transaction details
@lukasschor lukasschor self-assigned this Jun 20, 2019
@lukasschor lukasschor added this to the Safe for Teams 0.2.0 milestone Jun 24, 2019
@lukasschor lukasschor removed the Epic label Jul 2, 2019
@germartinez germartinez self-assigned this Jul 23, 2019
@lukasschor lukasschor assigned germartinez and unassigned lukasschor Aug 10, 2019
@lukasschor lukasschor modified the milestones: Safe for Teams v0.2.0 (Dappcon), Safe for Teams v1.0.0 (Devcon) Sep 2, 2019
@lukasschor lukasschor added Minor Needs to be fixed within the next 1-3 public releases. Major Needs to be fixed for immediate next public release. and removed Minor Needs to be fixed within the next 1-3 public releases. labels Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Major Needs to be fixed for immediate next public release.
Projects
None yet
Development

No branches or pull requests

2 participants