Skip to content

fix: iOS navigation session attachment and detachment#497

Merged
jokerttu merged 2 commits into
mainfrom
fix/ios-memory-leak-issues
Nov 25, 2025
Merged

fix: iOS navigation session attachment and detachment#497
jokerttu merged 2 commits into
mainfrom
fix/ios-memory-leak-issues

Conversation

@jokerttu

@jokerttu jokerttu commented Nov 21, 2025

Copy link
Copy Markdown
Contributor

Fixes iOS memory leak issues with navigation sessions and view attachment and detachments.
Fixes also view visibility issues in edge cases on iOS

Related issue: #435
Related issue: #358
Related issue: #491 (iOS only)

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.

Comment thread ios/react-native-navigation-sdk/NavViewController.m

@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.

Looks good to me.

@jokerttu
jokerttu merged commit cc0aae1 into main Nov 25, 2025
12 checks passed
@jokerttu
jokerttu deleted the fix/ios-memory-leak-issues branch November 25, 2025 08:37
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.

3 participants