HomePhabricator
Diffusion Comm d17d71bec3ff

[native] Fix crash in NUX

Description

[native] Fix crash in NUX

Summary:
issue: ENG-9375
If a dev kills the app while it is showing the NUX, then after they reopen they will get this error: "button should be registered with nuxTipContext"
This happens because for dev we remember the navigation state. When the navigation state is rehydrated, the buttons have not yet been mounted, so they are not registered with the context, resulting in the error.
Because this is for dev only, and because we expect the NUX tips to only be shown at the start of the app over the chat list, I think we should just reset the nav state if NUX tips are present.

Test Plan:
Created a new user -> saw the NUX tips -> navigated to a tip that points to a button -> closed the app -> opened the app
Before this resulted in an error on the screen. After this change the error doesn't appear. The user sees the chat list.

Reviewers: tomek, kamil, ashoat

Reviewed By: ashoat

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D13520

Details

Provenance
inkaAuthored on Mon, Sep 30, 5:44 AM
Reviewer
ashoat
Differential Revision
D13520: [native] Fix crash in NUX
Parents
rCOMM369199f83879: [terraform] Bump keyserver image to 1.0.124
Branches
Unknown
Tags
Unknown