Skip to content

fix!: native state issues on Android when multiple engines are used - #525

Merged
jokerttu merged 10 commits into
mainfrom
refactor/session-manager-lifecycle-architecture
Nov 19, 2025
Merged

jokerttu merged 10 commits into
mainfrom
refactor/session-manager-lifecycle-architecture

Conversation

@jokerttu

@jokerttu jokerttu commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

This PR fixes how navigation instances an navigation manager instances are handled when multiple flutter engines are used for example when foreground services are used.
This change will allow multiple engine intances to communicate with navigation sessions.

Fixes #308
Closes #169

Related #211
Related #274
Related #397

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation
  • I added new tests to check the change I am making
  • All existing and new tests are passing.

@jokerttu jokerttu changed the title fix: native state issues when multiple engines are used fix: native state issues on Android when multiple engines are used Nov 14, 2025
@jokerttu
jokerttu marked this pull request as ready for review November 14, 2025 11:29
@jokerttu
jokerttu force-pushed the refactor/session-manager-lifecycle-architecture branch 2 times, most recently from a680664 to dbe63bd Compare November 14, 2025 14:18
@jokerttu
jokerttu force-pushed the refactor/session-manager-lifecycle-architecture branch from dbe63bd to f146116 Compare November 17, 2025 10:11

@illuminati1911 illuminati1911 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Tests are bit unstable though but I don't think it's related to this PR.

@jokerttu jokerttu changed the title fix: native state issues on Android when multiple engines are used fix!: native state issues on Android when multiple engines are used Nov 19, 2025
@jokerttu
jokerttu merged commit cb7d61e into main Nov 19, 2025
43 of 49 checks passed
@jokerttu
jokerttu deleted the refactor/session-manager-lifecycle-architecture branch November 19, 2025 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Instance not created, create with createInstance() NavInfoEvent not triggered in android when application is in background/minimized

3 participants