LedgerHQ / ledger-live-desktop

⛔️ DEPRECATED - Ledger Live (Desktop)
https://www.ledger.com/live
MIT License
954 stars 297 forks source link

[LIVE-1899][LIVE-1900][LIVE-1903][LIVE-1934] Bugfix - NFTs stabilization #4942

Closed lambertkevin closed 2 years ago

lambertkevin commented 2 years ago

πŸ¦’ Context (issues, jira)

[LIVE-1899] [LIVE-1900] [LIVE-1903] [LIVE-1934]

πŸ’» Description / Demo (image or video)

https://user-images.githubusercontent.com/44363395/164739972-d41dff2d-91ac-41f8-b695-daff23e5f6b1.mov

https://user-images.githubusercontent.com/44363395/164740274-438fdf6d-1683-4df4-b3f4-5022947c1c9a.mov

Screenshot 2022-04-22 at 16 58 19

https://user-images.githubusercontent.com/44363395/164740751-2b70cfe1-9b04-4b56-a317-f9836bed8272.mov

πŸ–€ Expectations to reach

PR must pass CI, rebase develop if conflicts. Thanks!

github-actions[bot] commented 2 years ago

Thanks for your contribution. To be able to merge in develop branch, you need to:

github-actions[bot] commented 2 years ago

Coverage report

St.:grey_question:
Category Percentage Covered / Total
πŸ”΄ Statements 8.74% 9/103
πŸ”΄ Branches 0% 0/19
πŸ”΄ Functions 2.94% 1/34
πŸ”΄ Lines 8.33% 8/96

Test suite run success

1 tests passing in 1 suite.

Report generated by πŸ§ͺjest coverage report action from c951db99c5af8791dd69b0cceb0bd3a3956170ec

ghost commented 2 years ago

@gsoares-ledger adding you as reviewer just in case you have spare time ^^ as you're the only other NFT expert around.

ghost commented 2 years ago

Not for 2.41 branch, pending its merge into develop.