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

LIVE-545 Add USB Firmware updates on Android #2432

Open
wants to merge 76 commits into
base: develop
Choose a base branch
from

Conversation

grsoares21
Copy link
Contributor

@grsoares21 grsoares21 commented Apr 14, 2022

This adds the possibility of updating the firmware of the device via USB OTG on Android.
Not all firmware updates are supported. These are the devices than can be updated:
Nano S -> Version >= 1.6.1
Nano X -> Version >= 1.2.4-6
Nano S+ -> all
These refer to the current version of the device prior to the update.

screen-recording-20220428-163840-ll-dev-1_WYzfQ6tn.mp4

Type

Feature

Context

LIVE-545

Parts of the app affected / Test plan

Both the Portfolio page and the Manager page, present the firmware banner to the user if their device is out of date.
If the version is not supported, or the device is not connected via USB on Android, the user will be told to go to Desktop to update their device when clicking the update button. Otherwise the user will be directed to the update flow.
Updates to be tested:
LNS+:

  • 1.0.0 -> 1.0.2

LNS:

  • 2.0.0 -> 2.1.0
  • 1.6.1 -> 2.1.0

LNX:

  • 2.0.0 -> 2.0.2
  • 2.0.1 -> 2.0.2
  • 1.3.0 -> 2.0.2
  • 1.2.4-6 -> 2.0.0

If possible, to test with non supported versions to verify that the user is not able to access it.

@grsoares21 grsoares21 requested a review from a team April 14, 2022 12:24
@grsoares21 grsoares21 requested review from a team as code owners April 14, 2022 12:24
@grsoares21 grsoares21 changed the base branch from develop to release/v3.0.x April 14, 2022 14:38
@juan-cortes
Copy link
Contributor

You need to update the dependencies @grsoares21 so that CI passes, I think rebasing develop and bumping everything over there should work. If we are relying on a custom live-common, perhaps you need to align things over there too, or if things are merged already, use the right version

@juan-cortes juan-cortes self-requested a review May 10, 2022 15:23
Copy link
Contributor

@juan-cortes juan-cortes left a comment

Choose a reason for hiding this comment

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

Make sure CI passes and you dont have dependencies 🙏🏼

@juan-cortes
Copy link
Contributor

You didn't manually fix yarn.lock right? This is the result of a yarn deduplicate or similar process?
@grsoares21

@grsoares21
Copy link
Contributor Author

You didn't manually fix yarn.lock right? This is the result of a yarn deduplicate or similar process? @grsoares21

@juan-cortes it's the result of yarn-deduplicate

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants