Make execution functions payable in the standard #2
filmakarov
started this conversation in
Ideas
Replies: 1 comment
-
I think this is a pretty good idea. we should do this if there is no vetos :D |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Idea
There are use cases, that require passing value from the Executor to the SA along with
executeFromExecutor
andexecuteBatchFromExecutor
calls.For example, a module that enables perp trading features with the MSA.
In such a use case, to keep the interface consistent between perp DEX and the module and avoid extra gas overhead, the above mentioned methods should be payable.
They are currently payable in the reference implementation.
I propose marking them (along with other execution methods) in the ERC description as well.
Beta Was this translation helpful? Give feedback.
All reactions