Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

Upgrade to TrezorConnect v6 #12

Merged
merged 13 commits into from
Nov 21, 2018
Merged

Upgrade to TrezorConnect v6 #12

merged 13 commits into from
Nov 21, 2018

Conversation

brunobar79
Copy link
Contributor

@brunobar79 brunobar79 commented Nov 21, 2018

  • Removed local version of TrezorConnect
  • require trezor-connect from npm
  • Migrated code to V6 API
  • Tests updated
  • Bumped version to 0.2.0

This PR give us cross browser support for Trezor.

@brunobar79 brunobar79 changed the title Upgrade to TrezorConnect v5 Upgrade to TrezorConnect v6 Nov 21, 2018
Copy link

@estebanmino estebanmino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and I can confirm that the extension in this PR, MetaMask/metamask-extension#5797 working on both Brave and Firefox.

@brunobar79
Copy link
Contributor Author

Thanks @estebanmino!

@brunobar79 brunobar79 merged commit 64426c6 into master Nov 21, 2018
@whymarrh whymarrh deleted the trezor-v6 branch February 24, 2020 20:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants