fix: fix account list in the reveal SRP flow#24906
Conversation
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsThe changes modify the SRPListItem component to use account groups (multichain accounts) instead of individual account addresses. This is a significant refactoring that affects:
The E2E tests in Additionally,
The risk is medium because:
|
|
ccharly
left a comment
There was a problem hiding this comment.
Left some minor suggestions + Tested, works fine 💪
|
All nitpicks are resolved as part of: #25064 |



Description
The account list displayed on the SRP reveal flow was outdated by showing the UI pre-BIP44. This PR updates the UI to show the correct multichain account list.
Changelog
CHANGELOG entry: Update SRP flow to display multichain accounts
Related issues
Fixes: #22950
Manual testing steps
Screenshots/Recordings
Before
Check #22950
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Updates SRP reveal to reflect multichain account model.
accountGroupsinSRPListItem; toggle shows group count and expands aFlatListof group names with avatars seeded byselectIconSeedAddressByAccountGroupId; addmaxHeighttoaccountsListstyleselectAccountGroupsByKeyringIdselector and tests; integrate intoSRPListItemselectIconSeedAddressByAccountGroupIdresilient (returns empty string when no accounts); adjust tests accordinglySRPListItemandAddNewAccountto assert group-based counts, rendering, and analyticsWritten by Cursor Bugbot for commit 239b576. This will update automatically on new commits. Configure here.