Skip to content

Conversation

ghgoodreau
Copy link
Contributor

@ghgoodreau ghgoodreau commented Sep 10, 2025

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.

@metamaskbot metamaskbot added the team-swaps-and-bridge Swaps and Bridge team label Sep 10, 2025
@metamaskbot
Copy link
Collaborator

✨ Files requiring CODEOWNER review ✨

🔄 @MetaMask/swaps-engineers (9 files, +277 -52)
  • 📁 ui/
    • 📁 ducks/
      • 📁 bridge/
        • 📄 selectors.test.ts +2 -2
        • 📄 selectors.ts +143 -13
        • 📄 utils.ts +14 -1
    • 📁 hooks/
      • 📁 bridge/
        • 📁 __snapshots__/
          • 📄 useTokensWithFiltering.test.ts.snap +1 -1
          • 📄 useBridgeChainInfo.ts +31 -11
          • 📄 useTokensWithFiltering.ts +30 -9
    • 📁 pages/
      • 📁 bridge/
        • 📁 hooks/
          • 📄 useExternalAccountResolution.ts +35 -8
        • 📁 prepare/
          • 📁 components/
            • 📄 destination-account-picker.tsx +14 -0
            • 📄 prepare-bridge-page.tsx +7 -7

Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size-M team-swaps-and-bridge Swaps and Bridge team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants