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

LIVE-1469 Handle onboarded devices in device actions #1738

Merged
merged 1 commit into from
Feb 25, 2022

Conversation

juan-cortes
Copy link
Contributor

Context (issues, jira)

https://ledgerhq.atlassian.net/browse/LIVE-1469

Description / Usage

The latest firmware version in the upcoming LNS+ and on existing devices allow us to detect a device before it's onboarded but we can't interact with it the normal way because, well, it's not seeded. So we need to handle that case and show a meaningful error message to our users and guide them to the onboarding.

Expectations

  • Test coverage: The changes of this PR are covered by test. Unit test were added with mocks when depending on a backend/device.
  • No impact: The changes of this PR have ZERO impact on the userland. Meaning, we can use these changes without modifying LLD/LLM at all. It will be a "noop" and the maintainers will be able to bump it without changing anything.

@juan-cortes juan-cortes requested review from a team as code owners February 24, 2022 15:10
@vercel
Copy link

vercel bot commented Feb 24, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/ledgerhq/ledger-live-common/5TercUEt1EqgS5jWiWhpKbeQryaq
✅ Preview: https://ledger-live-common-git-bugfix-live-1469-ledgerhq.vercel.app

@valpinkman valpinkman changed the base branch from develop to hotfix/onboarding-lns-LIVE-1469 February 25, 2022 10:51
@valpinkman valpinkman merged commit 7f11cd1 into hotfix/onboarding-lns-LIVE-1469 Feb 25, 2022
@valpinkman valpinkman deleted the bugfix/LIVE-1469 branch February 25, 2022 10:55
@valpinkman valpinkman mentioned this pull request Feb 25, 2022
valpinkman added a commit that referenced this pull request Feb 25, 2022
* hotfix/LIVE-1469:
  v21.32.6
  LIVE-1469 Handle onboarded devices in device actions (#1738)
valpinkman added a commit that referenced this pull request Mar 7, 2022
* release/21.34.0: (24 commits)
  v21.34.0
  Create dependencies.md
  Filecoin fixes (#1740)
  Bump url-parse from 1.5.7 to 1.5.10 in /mobile-test-app (#1757)
  Bump url-parse from 1.5.7 to 1.5.10 in /cli (#1758)
  Bump url-parse from 1.5.7 to 1.5.10 (#1759)
  Bump url-parse from 1.5.7 to 1.5.10 in /tools (#1760)
  Bump urijs from 1.19.7 to 1.19.8 in /cli (#1755)
  Bump urijs from 1.19.7 to 1.19.8 in /mobile-test-app (#1753)
  Bump urijs from 1.19.7 to 1.19.8 (#1754)
  Bump urijs from 1.19.7 to 1.19.8 in /tools (#1756)
  LIVE-1469 Handle onboarded devices in device actions (#1738)
  bot: filecoin spec: remove specific version
  Fixes elrond bot LIVE-1462 (#1739)
  Make node-ble transport optional again (#1736)
  Update FeatureToggle component types (#1737)
  Bot: ethereum specs to allow more testTimeout (#1733)
  Fixes elrond specs bot case of the eGLD ticker on device
  fix dcr fees (#1734)
  Update mock API for market tests to support filtering (#1728)
  ...
valpinkman added a commit that referenced this pull request Mar 18, 2022
* release/v21.35.0:
  v21.35.0
  v21.34.0
  v21.32.6
  LIVE-1469 Handle onboarded devices in device actions (#1738)
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.

3 participants