Conversation
CLA Assistant Lite All Contributors have signed the CLA. |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
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.
Quite the photographer 😉
Pull Request Test Coverage Report for Build 1724088469
💛 - Coveralls |
E2E Tests Failed Failed tests:
|
The Image is there. Looks good Note: I still have an "old legder" so the original image would still work for people like me 😄 |
@francovenica think we should add smith like "(formerly known as Display Transaction Data)"? |
Probably good to cover all bases. |
On the other hand, the docs I linked mention what this setting used to be called, so I think it's fine. |
If you want to add something like "(Called Display Transaction Data in older models)". But I'm pretty sure is not that big of a deal, there isn't that many options inside the Eth app for ledger, not much room for the user to mess that one up |
What it solves
Resolves #2876
How this PR fixes it
I've updated the image (took it from Ledger's docs), added links to their docs and updated the name of the setting (it's now called Blind Signing).
Screenshots