Skip to content
This repository has been archived by the owner on Jun 5, 2019. It is now read-only.

trezorctl: update url to releases.json #242

Closed
wants to merge 1 commit into from

Conversation

bithobbes
Copy link
Contributor

No description provided.

@prusnak
Copy link
Member

prusnak commented Mar 28, 2018

I think we can implement the following improvements:

a) use connect().features.bootloader_mode to check whether the device is in the bootloader mode first and throw an exception if not

b) use connect().features.major_version to determine whether we should put 1 or 2 in the JSON url

Would you like to try it out?

@prusnak prusnak added this to the v0.9.2 milestone Apr 11, 2018
@prusnak
Copy link
Member

prusnak commented Jun 1, 2018

Closing as unmerged. This needs to be handled in a more systematic way.

Let's deal with that in #269

@prusnak prusnak closed this Jun 1, 2018
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