rainbow icon indicating copy to clipboard operation
rainbow copied to clipboard

Wallet list context menu w/ notification settings

Open benisgold opened this issue 2 years ago • 1 comments

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?

benisgold avatar Jul 20 '22 17:07 benisgold

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

linear[bot] avatar Jul 20 '22 17:07 linear[bot]