LedgerHQ / ledgerjs

⛔️ MOVED to monorepo "ledger-live"
https://github.com/LedgerHQ/ledger-live
Apache License 2.0
573 stars 381 forks source link

Bugfix/live 1799: Nano S Plus storage - Adjust size value #835

Closed alexandremgo closed 2 years ago

alexandremgo commented 2 years ago

Type

Bugfix

Context

https://ledgerhq.atlassian.net/jira/software/c/projects/LIVE/boards/462?modal=detail&selectedIssue=LIVE-1799

The Nano S Plus capacity is 1536 KB and not 1533 KB

What is affected / Test plan

The memorySize property is used to compute the remaining free storage space in the nano + display to the user in the LLD and LLM (the computation is done in live-common).

The new devices package has been tested on LLD:

previous version with 1533 KB **Look at the "free" storage section on the bottom right** ![LLD_LIVE-1599_1533_0](https://user-images.githubusercontent.com/15096913/162767188-8610d65a-07d0-48c5-a465-381d3a0ac80d.png)
new version with 1536 KB **Look at the "free" storage section on the bottom right** ![LLD_LIVE-1599_1536_0](https://user-images.githubusercontent.com/15096913/162767258-2efe9158-081d-4121-8346-68bf3f693e16.png)

Note

Also bump to node v14 in the .nvmrc file (same as the version used in LLD and LLM)

codecov[bot] commented 2 years ago

Codecov Report

Merging #835 (f153230) into master (f9554d5) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #835   +/-   ##
=======================================
  Coverage   45.60%   45.60%           
=======================================
  Files          82       82           
  Lines        5004     5004           
  Branches      890      890           
=======================================
  Hits         2282     2282           
  Misses       2526     2526           
  Partials      196      196           
Impacted Files Coverage Δ
packages/devices/src/index.ts 0.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f9554d5...f153230. Read the comment docs.