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

Single tx safe apps interactions should not use multisend #1791

Closed
rmeissner opened this issue Jan 18, 2021 · 0 comments · Fixed by #1792
Closed

Single tx safe apps interactions should not use multisend #1791

rmeissner opened this issue Jan 18, 2021 · 0 comments · Fixed by #1792
Assignees

Comments

@rmeissner
Copy link
Member

Title/Description

If a safe app triggers only a single transaction, we should not use multisend. This would safe gas and make the decoding easier.

Steps to reproduce

  • Use transaction builder
  • Add single contract interaction
  • Submit

Expected result

  • Data of triggered tx to be the contract interaction and to should be contract specified in tx builder

Obtained result

  • Data is a invocation of the multisend method with the encoded contract interaction and the to is the multisend
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants