Skip to content

feat: ledger error handling. #39038

Draft
montelaidev wants to merge 224 commits intomainfrom
fix/hw-swaps-retry
Draft

feat: ledger error handling. #39038
montelaidev wants to merge 224 commits intomainfrom
fix/hw-swaps-retry

Conversation

@montelaidev
Copy link
Contributor

@montelaidev montelaidev commented Jan 6, 2026

Description

Open in GitHub Codespaces

Changelog

CHANGELOG entry:

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

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.

@montelaidev montelaidev self-assigned this Jan 6, 2026
@montelaidev montelaidev added the DO-NOT-MERGE Pull requests that should not be merged label Jan 6, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

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.

@metamaskbot metamaskbot added the team-accounts-framework Accounts Framework team label Jan 6, 2026
@metamaskbotv2
Copy link
Contributor

metamaskbotv2 bot commented Jan 6, 2026

✨ Files requiring CODEOWNER review ✨

📜 @MetaMask/policy-reviewers (12 files, +864 -144)
  • 📁 lavamoat/
    • 📁 browserify/
      • 📁 beta/
        • 📄 policy.json +38 -18
      • 📁 experimental/
        • 📄 policy.json +38 -18
      • 📁 flask/
        • 📄 policy.json +38 -18
      • 📁 main/
        • 📄 policy.json +38 -18
    • 📁 webpack/
      • 📁 mv2/
        • 📁 beta/
          • 📄 policy.json +23 -18
        • 📁 experimental/
          • 📄 policy.json +23 -18
        • 📁 flask/
          • 📄 policy.json +23 -18
        • 📁 main/
          • 📄 policy.json +23 -18
      • 📁 mv3/
        • 📁 beta/
          • 📄 policy.json +155 -0
        • 📁 experimental/
          • 📄 policy.json +155 -0
        • 📁 flask/
          • 📄 policy.json +155 -0
        • 📁 main/
          • 📄 policy.json +155 -0

Tip

Follow the policy review process outlined in the LavaMoat Policy Review Process doc before expecting an approval from Policy Reviewers.

@montelaidev
Copy link
Contributor Author

@metamaskbot update-policies

@metamaskbot
Copy link
Collaborator

Policies updated.
👀 Please review the diff for suspicious new powers.

🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff

👀 lavamoat/browserify/beta/policy.json changes differ from main/policy.json policy changes
👀 lavamoat/browserify/experimental/policy.json changes differ from main/policy.json policy changes
👀 lavamoat/browserify/flask/policy.json changes differ from main/policy.json policy changes
👀 lavamoat/webpack/mv2/beta/policy.json changes differ from mv2/main/policy.json policy changes
👀 lavamoat/webpack/mv2/experimental/policy.json changes differ from mv2/main/policy.json policy changes
👀 lavamoat/webpack/mv2/flask/policy.json changes differ from mv2/main/policy.json policy changes
👀 lavamoat/webpack/mv3/beta/policy.json changes differ from mv3/main/policy.json policy changes
👀 lavamoat/webpack/mv3/experimental/policy.json changes differ from mv3/main/policy.json policy changes
👀 lavamoat/webpack/mv3/flask/policy.json changes differ from mv3/main/policy.json policy changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO-NOT-MERGE Pull requests that should not be merged size-XL team-accounts-framework Accounts Framework team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants