[PR #6638] PM-33411: bug: Defer early navigation until lifecycle is resumed #12883

Closed
opened 2026-04-11 03:52:49 -05:00 by GiteaMirror · 0 comments
Owner

Original Pull Request: https://github.com/bitwarden/android/pull/6638

State: closed
Merged: Yes


🎟️ Tracking

PM-33411

📔 Objective

This PR updates the EventsEffect to support deferred background events that will wait until the screen is resumed before firing the event. This us useful for events that are fired off during the init block of the VM.

This change should ensure that the graph is fully set on the NavHost before attempting to do a deeplink navigation. I believe this is the main place this sort of issue can occur because the navigation events are occurring at the same time the NavHost is being configured, leading to a race-condition.

**Original Pull Request:** https://github.com/bitwarden/android/pull/6638 **State:** closed **Merged:** Yes --- ## 🎟️ Tracking [PM-33411](https://bitwarden.atlassian.net/browse/PM-33411) ## 📔 Objective This PR updates the `EventsEffect` to support deferred background events that will wait until the screen is resumed before firing the event. This us useful for events that are fired off during the `init` block of the VM. This change should ensure that the graph is fully set on the `NavHost` before attempting to do a deeplink navigation. I believe this is the main place this sort of issue can occur because the navigation events are occurring at the same time the `NavHost` is being configured, leading to a race-condition. [PM-33411]: https://bitwarden.atlassian.net/browse/PM-33411?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
GiteaMirror added the pull-request label 2026-04-11 03:52:49 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/android#12883