-
Notifications
You must be signed in to change notification settings - Fork 26
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.
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
feat: Add support for Ledger hardware wallet #68
Conversation
src/constants.js
Outdated
@@ -95,6 +95,7 @@ export const SENTRY_DSN = process.env.SENTRY_DSN || 'https://69c067d1587c465cac8 | |||
* Server options for the user to choose which one to connect | |||
*/ | |||
export const DEFAULT_SERVERS = [ | |||
'http://localhost:8080/v1a/', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove localhost from servers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The implementation is heavily dependent on Ledger. We will need a major refactor to add support to more hardware wallets. Anyway, I think it is good for this PR and we should refactor it in another PR. What do you think?
Co-authored-by: Yan Martins <yan.martins@gmail.com>
21b1c70
to
ae6d8c1
Compare
…t/ledger-comunication
…t/ledger-comunication
Closes #50 and #51