safe-wallet-web icon indicating copy to clipboard operation
safe-wallet-web copied to clipboard

feat: Add support for proposing transactions as a delegate

Open usame-algan opened this issue 6 months ago • 5 comments

What it solves

Resolves #1825

How this PR fixes it

  • Check if a connected wallet is a delegate and store it as a flag in useWallet
  • Enable transaction buttons for delegates
  • Display a DelegateForm when creating a transaction and only propose the transaction

ToDos

  • [ ] The transaction should be accessible via the UI even with 0 confirmations (untrusted)
  • [ ] Adjust message content on the review tx screen
  • [ ] Add v2 endpoint to the SDK instead of using deprecated v1 endpoint via getDelegates

How to test it

  1. Open a Safe that has a delegate who is not an owner
  2. Connect with that delegate
  3. Observe the New transaction button is enabled
  4. Observe being able to create a transaction

Screenshots

Screenshot 2024-07-31 at 12 39 25

Checklist

  • [ ] I've tested the branch on mobile 📱
  • [ ] I've documented how it affects the analytics (if at all) 📊
  • [ ] I've written a unit/e2e test for it (if applicable) 🧑‍💻

usame-algan avatar Jul 31 '24 10:07 usame-algan