metamask-mobile
metamask-mobile copied to clipboard
feat: Base Wallet Details Screen
Description
This PR implements the basic wallet details screen. This feature is hidden behind a remote feature flag. This screen shows all the accounts that are tied to a wallet along with some other information...
- wallet name
- Total wallet balance (aggregate balances of all accounts in the wallet). This is not the aggregate balance of all your accounts, just the accounts that belong to the wallet
- List of accounts that belong to that wallet with...
- the account name
- account icon (Blockies or Jazzicon)
- individual account balance
Designs:
Out of scope:
- Social recovery
- SRP
- Adding an account These changes will be made in subsequent PRs.
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/MUL-211
Manual testing steps
- This feature is hidden behind a remote feature flag so to enable it you will either need to use a proxy (message me for details) or have this function return true as well as this one.
- Create a wallet
- Open the account list.
- Since the multi-chain accounts state 1 feature flag is enabled, you should see your accounts grouped by wallet
- click on the blue details icon beside the wallet name
- it should take you to the wallet details screen
- this screen should contain your wallet name, total wallet balance, and a list of your accounts in that wallet with their individual balances.
- Click on an account should take you to that accounts details screen
- clicking back from that screen should take you back to wallet details
- Clicking back on the wallet details screen should take you back to the accounts list.
Screenshots/Recordings
Before
N/A
After
https://github.com/user-attachments/assets/b7e49e6f-587f-4470-8435-8b0f27ed2114
Pre-merge author checklist
- [ ] I’ve followed MetaMask Contributor Docs and MetaMask Mobile 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.