You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In our repo we are using Safe as the entity holding funds for some of our agents. For some contract interactions, it failed since the parameter accessList was missing (see this for rationale, transaction fails due out-of-gas error).
We realized it was missing from the references below:
I'm not saying you should change anything in the code, I would add a hint to the reader (on any README file) saying that access_list might be required when executing transactions with Safe's having multiple signers.
I would add a hint to the reader (on any README file) saying that access_list might be required when executing transactions with Safe's having multiple signers
Why it might be required? It could be more optimal, and maybe required in an edge case, but in my personal case I was never forced to use it
What is needed?
In our repo we are using Safe as the entity holding funds for some of our agents. For some contract interactions, it failed since the parameter
accessList
was missing (see this for rationale, transaction fails due out-of-gas error).We realized it was missing from the references below:
I would suggest adding a reference to this in the README of this repo and/or somewhere else in the documentation.
Background
For reference, here is the relevant PR where we add
accessList
(link).The text was updated successfully, but these errors were encountered: