Skip to content

Add Trezor support #1

@jordaniza

Description

@jordaniza

Hey @ind-igo, thanks for this. I'd like to use it with my Trezor and happy to PR into the repo instead of forking.

Just reviewing the BatchScript.sol file and want to double check the implementation approach with you to confirm I'm not missing anything:

https://github.com/ind-igo/forge-safe/blob/main/src/BatchScript.sol#L68
Add hash of "trezor" as an option

https://github.com/ind-igo/forge-safe/blob/main/src/BatchScript.sol#L158
Add trezor to switch statement - I am assuming trezor supports the mnemonic index flag but I can't see documentation on it?

Cheers

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions