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

Upgrade to Trezor Connect 8#46

Merged
darkwing merged 6 commits intoMetaMask:masterfrom
darkwing:upgrade-trezor-8
Jan 19, 2021
Merged

Upgrade to Trezor Connect 8#46
darkwing merged 6 commits intoMetaMask:masterfrom
darkwing:upgrade-trezor-8

Conversation

@darkwing
Copy link
Copy Markdown
Contributor

@darkwing darkwing commented Jan 10, 2021

Represents a work in progress update for Trezor Connect 8.

@darkwing darkwing marked this pull request as ready for review January 12, 2021 20:08
@darkwing darkwing changed the title Debug upgrading to trezor connect 8 Upgrade to Trezor Connect 8 Jan 13, 2021
@darkwing darkwing requested review from Gudahtt and removed request for a team January 13, 2021 18:09
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM! Tested on both the trezor one and model T.

@darkwing darkwing merged commit 88d46e2 into MetaMask:master Jan 19, 2021
Gudahtt added a commit that referenced this pull request Jan 19, 2021
This release includes various dependency updates and development
tooling changes. The only functional change is the upgrade of
`trezor-connect` from v7 to v8 (#46).
@Gudahtt Gudahtt mentioned this pull request Jan 19, 2021
Gudahtt added a commit that referenced this pull request Jan 19, 2021
This release includes various dependency updates and development
tooling changes. The only functional change is the upgrade of
`trezor-connect` from v7 to v8 (#46).
Gudahtt added a commit that referenced this pull request Jan 19, 2021
`sinon` was accidentally added as a `dependency` in #46. Instead it
should be declared as a `devDependency`, as it's only used in tests.
Gudahtt added a commit that referenced this pull request Jan 19, 2021
`sinon` was accidentally added as a `dependency` in #46. Instead it
should be declared as a `devDependency`, as it's only used in tests.
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