fix: non evm bridge activity type cp-13-36.0#43557
Conversation
|
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/swaps-engineers (1 files, +1 -1)
|
| selectNonEvmTransactionsForActivity, | ||
| getAssetsMetadata, | ||
| getInternalAccountsObject, | ||
| selectBridgeHistoryItem, |
There was a problem hiding this comment.
- include the bridge item, if existing
| const bridgeStatus = bridgeHistory.status.status; | ||
|
|
||
| return { | ||
| type: 'bridge', |
Builds ready [89595b0]
⚡ Performance Benchmarks (Total: 🟢 18 pass · 🟡 6 warn · 🔴 0 fail)
Bundle size diffs [🚀 Bundle size reduced!]
|
Builds ready [6120af4] [reused from 89595b0]
⚡ Performance Benchmarks (Total: 🟢 18 pass · 🟡 6 warn · 🔴 0 fail)
Bundle size diffs [🚀 Bundle size reduced!]
|
Description
Keyring transactions mark these as "send" and need a lookup to render as a bridge activity
Fixes #43498
Note: activity redesign is behind a flag
Changelog
CHANGELOG entry: null
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist