rainbow
rainbow copied to clipboard
Wallet list context menu w/ notification settings
Fixes TEAM2-80
Figma link (if any): https://www.figma.com/file/Xc4Pq3CLJNVG52z1K0URvP/notifications?node-id=1010:7719
What changed (plus any additional context for devs)
- replaced action sheet with context menu on wallet list edit
- added notification settings option
Screen recordings / screenshots
https://user-images.githubusercontent.com/15272675/180043795-8047208d-fffc-45d7-81e2-9c95029d9db7.mp4
What to test
- all the options work properly
- android (since it doesnt support context menus)
Final checklist
- [ ] Assigned individual reviewers?
- [ ] Added labels? (
team1/team2
,critical path
,release
,dev QA
) - [ ] Did you test both iOS and Android?
- [ ] If your changes are visual, did you check both the light and dark themes?
- [ ] Added e2e tests? If not, please specify why
- [ ] If you added new critical path files, did you update the CODEOWNERS file?
- [ ] If no
dev QA
label, did you add the PR to the QA Queue?
TEAM2-80 Add "Notification Settings" deeplink to wallet selection sheet
PixelSnap 2022-06-01 at 6.55.28 [email protected]
- [ ] Notification Settings deeplink that opens up the Settings→Notifications→[wallet] section where you can manage all notification settings for this wallet
- [ ] Include wallet name on top of the menu for context