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
@overcat, there is a chance that updated Trezor will not work through js-stellar-wallets here if it uses WebUSB. We had to implement Ledger connect locally for the Account Viewer because of WebUSB restrictions. I don't remember the details, but it was something to do with only direct action that could launch something—using a library sort of created a 3rd party in the middle, which wasn't allowed. Maybe Trezor handles it differently, wanted to bring this up just in case.
We could just go ahead and implement Trezor connect locally as well, to remove dependency on js-stellar-wallets.
Hi @quietbits, it looks like I forgot to remove the reference to trezor-connect, but I have now fixed it. I have also tested it on the latest versions of Chrome and Firefox and it seems to be running smoothly.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Trezor v8 related npm packages are now in maintenance mode, future updates will be based on Trezor v9
trezor/trezor-suite#5358