Skip to content

Conversation

@jokerttu
Copy link
Contributor

@jokerttu jokerttu commented Nov 14, 2025

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
Copy link
Contributor

@illuminati1911 illuminati1911 left a comment

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