docs: update comments in Multisend example

[solarthesis]

What I did

Updated the comments on multisend showing users how to stage transaction without getting caught with the error in case not enough signers are available on local.

fixes: #

As per request #32 (comment)

How I did it

NA

How to verify it

NA

Checklist

• Passes all linting checks (pre-commit and CI jobs)
• New test cases have been added and are passing
• Documentation has been updated
• PR title follows Conventional Commit standard (will be automatically included in the changelog)

[fubuloubu]

Would like to find a way to refactor the plugin so that this is no longer what you need to do to get a partially signed transaction into the Safe API

cc @antazoey what do you think?

[antazoey]

cc @antazoey what do you think?

I thought the issue was on the core side. In any case, documenting the way it works today is not a bad thing.

[fubuloubu]

cc @antazoey what do you think?

I thought the issue was on the core side. In any case, documenting the way it works today is not a bad thing.

Issue can be improved with some core updates, but also the assumption if you are executing a transaction directly is that you want to execute it right now, so there should be an easier way to do the second workflow where you don't want to execute it right now that doesn't involve raising an error even though it is actually successfully performing that task

maybe something like SafeAccount.propose(txn: TransactionAPI, **safetx_kwargs) and you can imitate this example with something like:

from ape_safe import multisend
from ape import accounts
# load the safe
safe = accounts.load("my-safe")

ms = multisend.MultiSend()
ms.add(contractA.myMethod1, *call_args)
ms.add(contractB.myMethod2, *call_args)
...  # Add as many calls as desired to execute
ms.add(contractC.myMethod3, *call_args)

safetx = safe.propose(ms.as_transaction())

[antazoey]

there is a propose cli method you could try

[fubuloubu]

there is a propose cli method you could try

can that be "upstreamed" into the SafeAccount?

[fubuloubu]

maybe something like SafeAccount.propose(txn: TransactionAPI, **safetx_kwargs) and you can imitate this example with something like:

from ape_safe import multisend
from ape import accounts
# load the safe
safe = accounts.load("my-safe")

ms = multisend.MultiSend()
ms.add(contractA.myMethod1, *call_args)
ms.add(contractB.myMethod2, *call_args)
...  # Add as many calls as desired to execute
ms.add(contractC.myMethod3, *call_args)

safetx = safe.propose(ms.as_transaction())

p.s. added this here: 9aa25ab

[antazoey]

can that be "upstreamed" into the SafeAccount?

Probably!