-
Notifications
You must be signed in to change notification settings - Fork 115
Update live-common #1808
Update live-common #1808
Conversation
Hey there, upon trying to build this PR on Android, I ran into a syntax error in node_modules/pdkdf, let me know if you need additional details, can't provide a screenshot here @gre |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok i think i had it one once, but i had it fixed by running with node 14 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm sorry it seems not the same case, we reproduced on node 14😔
thanks! we are on it. |
Hey guys, Ran into something dirty while performing regression testing. Steps to reproduce :
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be reproduced on iphone 6s plus ios 14.6
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems a Bug related to ERC-20, please have a check about it😔
We introduced a bug when tracking analytics, can't rely on account.currency when sub-accounts are possible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
* Update live-common * live-common 20.3.0 * latest url for zerion * fix * update latest platform wording * update dep * fix pbkdf2 error * Fix crash on op details swap history We introduced a bug when tracking analytics, can't rely on account.currency when sub-accounts are possible. * Use name, not id for tracking event * Actually pass the account down to the footer Co-authored-by: Valentin D. Pinkman <[email protected]> Co-authored-by: Juan Cortes Ross <[email protected]>
https://ledgerhq.atlassian.net/browse/LL-6308
https://github.com/LedgerHQ/ledger-live-common/releases/tag/v20.2.0
https://github.com/LedgerHQ/ledger-live-common/releases/tag/v20.3.0
see LedgerHQ/ledger-live-desktop#3962 for impact of the scope