Skip to content
This repository has been archived by the owner on Nov 30, 2020. It is now read-only.

Add Trezor support #70

Open
zulucrypto opened this issue May 24, 2018 · 7 comments · May be fixed by #75
Open

Add Trezor support #70

zulucrypto opened this issue May 24, 2018 · 7 comments · May be fixed by #75

Comments

@zulucrypto
Copy link

I'm planning on implementing Trezor support but I'm opening an issue for initial discussion and feedback before I begin coding.

This will also depend on an update to the Trezor Connect project which is being tracked by this issue: trezor/connect#127

As far as the actual implementation, I was planning on adding a "Sign in with Trezor" button under the "Sign in with Ledger" one and essentially copying the Ledger functionality.

@zulucrypto zulucrypto linked a pull request Sep 17, 2018 that will close this issue
@spacesailor24
Copy link

Hi there,

I'm having difficulty trying to sign a transaction using Trezor model T and Stellar Account Viewer.

Please review my reddit post for more details, thank you.

https://www.reddit.com/r/Stellar/comments/abynfg/send_fails_with_stellar_account_viewer/?utm_source=reddit-android

@spacesailor24
Copy link

Sorry, I'm just now seeing that the PR that implements Trezor support hasn't been merged.

@reqlez
Copy link

reqlez commented Jan 7, 2019

I tested sending with Trezor from the Account Viewer ( trezor.io/stellar/ ) and you are right! I get the following message:

**"Could not sign transaction: Trezor: Parameter "amount" has invalid type. "string" expected.

Warning We submitted your transaction to the network but because of the network conditions we are not certain about its status: it could either succeed or fail. Instead of recreating the transaction you should use the button below to safely resubmit the transaction:
"**
I was able to send funds into the Trezor account and I can see them fine, they update in real time too. What happens is that when I press send, a new chrome tab opens asking to connect to Trezor to "prepare transaction", I click allow, but nothing actually happens on the Trezor it does not show that I have to allow it on the device itself... and right after that, I get the error. So it seems that the request either never makes it to the Trezor, or I don't know what's up.

However, when I moved my funds to the Trezor account, I saw the balance but there was no option to set inflation destination, so I went ahead and used this: https://kolontsov.github.io/stellar-trezor-inflation/ and it connected to the Trezor T just fine and submitted the request to the network no problem with my signature. By the way, may I recommend to integrate this feature into Account Viewer? i'm sure lots of people will be asking.

So not sure, you say PR is not merged but how come trezor.io/stellar/ has the page up and running? They have forked the code here too... https://github.com/trezor/stellar-account-viewer/issues

@prusnak
Copy link

prusnak commented Jan 9, 2019

@spacesailor24 @reqlez should be fixed on https://trezor.io/stellar

@spacesailor24
Copy link

@prusnak Thank you for the update! Will have to test myself at a later time.

@reqlez
Copy link

reqlez commented Jan 9, 2019

Actually, just tested with T and Chrome ( wihout the bridge software installed, if that matters ). Works great ! May I suggest getting the inflation destination code on the page going as well ;-)

@leaenbinario
Copy link

I get Error:"Failed to sign with Ledger device: U2F DEVICE_INELIGIBLE" when I try to use Sign with BIP Path

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants