Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/ledger integration #7004

Conversation

stanleyyconsensys
Copy link
Contributor

@stanleyyconsensys stanleyyconsensys commented Aug 17, 2023

Description
sync latest ledger integration code from fork

Checklist

  • There is a related GitHub issue
  • Tests are included if applicable
  • Any added code is fully documented

stanleyyconsensys and others added 16 commits July 26, 2023 13:03
- use isHardwareAccount instead isQRHardwareAccount
…nsaction

- remove duplicate function renderLedgerSignButton
- update latest renderLedgerSignButton function param for createStyles
- fix action view default label , defaultProps is not able to translate
- signLedgerTransaction is not being pass when using TransactionElement
…saction-in-ledger

fix(MetaMask#24):  uncaught error unsigned transaction in ledger
…g-transaction-via-browser

fix(MetaMask#34): incorrect label when signing transaction via browser
…object-error-

fix(MetaMask#33): fix Unhandled exception on null object error in transaction
…-user-tries-to-reveal-Ledger-private-key

fix(MetaMask#18): change to display same error message with keystone
- replace AppState.removeEventListener to remove
- replace TransactionController.cancelTransaction to ApprovalController.reject
- fix duplicate language block in en.json
- replace legacy approval transaction
fix (MetaMask#17): Unknown labels on transaction screen for MetaMask and imported accounts
fix: ledger label on wallet screen
…e-conflict-on-approval

fix (MetaMask#44): fix legacy code with merge conflict on approval
@stanleyyconsensys stanleyyconsensys requested a review from a team as a code owner August 17, 2023 06:14
@github-actions
Copy link
Contributor

github-actions bot commented Aug 17, 2023

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@stanleyyconsensys
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@stanleyyconsensys stanleyyconsensys merged commit 16fc6d0 into MetaMask:feat/ledger-integration Aug 17, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Aug 17, 2023
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.

2 participants