tlon-apps icon indicating copy to clipboard operation
tlon-apps copied to clipboard

onboarding: add notification prompt

Open jamesacklin opened this issue 1 month ago • 3 comments

Summary

Fixes TLON-5038 by allowing the user to customize their notification settings during the onboarding flow.

Changes

  • Adds a new screen, SetNotificationsScreen, to the onboarding flow between nickname and ship reservation steps
  • Stores the desired %volume level in the onboarding provider, injects after ship auth
  • Moves the device notification prompt to this screen rather than on SetNickname
  • Also fixes TLON-5054 by moving that line of text to a new line. There's something up with text inside of a Pressable inside of text that a framework upgrade may have affected.
  • Also adds a no-op store to the app Cosmos fixture wrapper, similar to the packages/app one works.

How did I test?

  • Sign up as a new user
  • Complete this new step
  • Navigate to your global notification settings in the app
  • Confirm the setting reflects that which you set during onboarding

Risks and impact

  • Safe to rollback without consulting PR author? Yes
  • Affects important code area:
    • [x] Onboarding
    • [ ] State / providers
    • [ ] Message sync
    • [ ] Channel display
    • [ ] Notifications
    • [ ] Other:

Rollback plan

git revert

Screenshots / videos

Screenshot 2025-11-17 at 8 36 19 AM

jamesacklin avatar Nov 17 '25 13:11 jamesacklin

@dnbrwstr PTAL; I found an opportunity to unify the options themselves with a hook to be used in ChatOptionsSheets even if it's a different interface (ActionGroup vs radio buttons).

jamesacklin avatar Nov 18 '25 17:11 jamesacklin

Moving back to draft so we can have a discussion with design.

jamesacklin avatar Nov 18 '25 22:11 jamesacklin

Ready to go once more.

jamesacklin avatar Dec 03 '25 15:12 jamesacklin