Feat/ledger device sdk
Description
Changelog
CHANGELOG entry:
Related issues
Fixes:
Manual testing steps
- Go to this page...
Screenshots/Recordings
Before
After
Pre-merge author checklist
- [ ] I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
- [ ] I've completed the PR template to the best of my ability
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using JSDoc format if applicable
- [ ] I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
Pre-merge reviewer checklist
- [ ] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.
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.
✨ Files requiring CODEOWNER review ✨
✅ @MetaMask/confirmations (1 files, +5 -0)
- 📁 ui/
- 📁 pages/
- 📁 confirmations/
- 📁 components/
- 📁 confirm/
- 📁 ledger-info/
- 📄
ledger-info.tsx+5 -0
- 📄
- 📁 ledger-info/
- 📁 confirm/
- 📁 components/
- 📁 confirmations/
- 📁 pages/
📜 @MetaMask/policy-reviewers (1 files, +345 -370)
- 📁 lavamoat/
- 📁 build-system/
- 📄
policy.json+345 -370
- 📄
- 📁 build-system/
[!TIP]
Follow the policy review process outlined in the LavaMoat Policy Review Process doc before expecting an approval from Policy Reviewers.
🔄 @MetaMask/swaps-engineers (1 files, +12 -1)
- 📁 ui/
- 📁 pages/
- 📁 swaps/
- 📁 prepare-swap-page/
- 📄
prepare-swap-page.js+12 -1
- 📄
- 📁 prepare-swap-page/
- 📁 swaps/
- 📁 pages/
[!CAUTION] MetaMask internal reviewing guidelines:
- Do not ignore-all
- Each alert has instructions on how to review if you don't know what it means. If lost, ask your Security Liaison or the supply-chain group
- Copy-paste ignore lines for specific packages or a group of one kind with a note on what research you did to deem it safe.
@SocketSecurity ignore npm/PACKAGE@VERSION
| Action | Severity | Alert (click "▶" to expand/collapse) |
|---|---|---|
| Block | Publisher changed: npm
|