Skip to content

Solidity account with Multisig - incorrect names in generated comment #712

@ericglau

Description

@ericglau

From the Account tab, enable Modules and choose Signer with Multisig. The contract imports are:

contract MyAccount is Account, IERC1271, AccountERC7579, MultiSignerERC7913 {

but the generated comment has the wrong name for MultiSignerERC7913:

    // IMPORTANT: Make sure SignerMultisig is most derived than AccountERC7579
    // in the inheritance chain (i.e. contract ... is AccountERC7579, ..., SignerMultisig)

Similarly, Signer with Multisig Weighted has contract imports:

contract MyAccount is Account, IERC1271, AccountERC7579, MultiSignerERC7913Weighted {

but the generated comment has the wrong name for MultiSignerERC7913Weighted:

    // IMPORTANT: Make sure SignerMultisigWeighted is most derived than AccountERC7579
    // in the inheritance chain (i.e. contract ... is AccountERC7579, ..., SignerMultisigWeighted)

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workinggood first issueGood for newcomers

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions