Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support Trezor for signing #270

Closed
Uxio0 opened this issue Oct 3, 2023 · 4 comments · Fixed by #318
Closed

Support Trezor for signing #270

Uxio0 opened this issue Oct 3, 2023 · 4 comments · Fixed by #318
Assignees
@moisses89
Labels
enhancement New feature or request

Comments

@Uxio0
Copy link
Member

Uxio0 commented Oct 3, 2023

What is needed?

Trezor must be supported for signing
@Uxio0 Uxio0 added the enhancement New feature or request label Oct 3, 2023
@Uxio0 Uxio0 self-assigned this Oct 3, 2023
@Uxio0 Uxio0 mentioned this issue Oct 3, 2023
Closed
@wminshew
Copy link

So if I have a safe on OP from proxy-factory 0xa6B71E26C5e0845f74c812102Ca7114b6a896AB2 (not supported by UI) with only trezor signers, do I have any workaround options today to execute txns?

@Uxio0 Uxio0 assigned moisses89 and unassigned Uxio0 Nov 27, 2023
@moisses89 moisses89 mentioned this issue Nov 29, 2023
Merged
@LayneHaber
Copy link

any idea when trezor support will be added here?

@Uxio0
Copy link
Member Author

Uxio0 commented Dec 5, 2023

@LayneHaber We are working on it as you can see in the linked PR, it should be ready next week 😉

@JagoFigueroa
Copy link

Tested and verified, reported a couple of small issues that will be tackled on separate tickets 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@moisses89 moisses89

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Trezor support safe-global/safe-cli
5 participants
@Uxio0 @moisses89 @wminshew @LayneHaber @JagoFigueroa