Skip to content

Remove app state and add logs push navigation #4310

Martin Santangelo requested to merge fix/push-navigation-android into develop

Ticket(s) / Related Merge Requests

#4310 (closed)

Summary of Changes

According to sentry reports, the issue seems to be happening when a push notification is opened after the app is killed by Android because of the device's low memory.

I removed the app state event handler that is not used anymore and that is causing re-renders of the whole app when moving the app background/foreground. (This makes the navigator ref be set many times)

This may solve the issue, in case it is not, I added some additional logs to check when the navigator reference is set and when the push navigation is handled to help us to debug it further

Testing Considerations

  • The app should not re-render anymore when moving it to the background/foreground (can be only be tested on dev mode)

Regression Scope

none

Platform Affected (web, mobile, etc)

Android

Developer Testing Completed

Checked that the app is not re-rendering when moving it to the background/foreground

Screenshots

none

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Merge request reports