Skip to content

fix(ios): scene notify first launch#11149

Merged
NathanWalker merged 2 commits intomainfrom
fix/scene-launch
Mar 30, 2026
Merged

fix(ios): scene notify first launch#11149
NathanWalker merged 2 commits intomainfrom
fix/scene-launch

Conversation

@NathanWalker
Copy link
Copy Markdown
Contributor

  • improves conditions across all frameworks for scene first launch notification

@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Mar 30, 2026

View your CI Pipeline Execution ↗ for commit cdcbcb3

Command Status Duration Result
nx test apps-automated -c=android ✅ Succeeded 3m 50s View ↗
nx run-many --target=test --configuration=ci --... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-03-30 16:58:00 UTC

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adjusts iOS scene-lifecycle startup logic so the “scene first launch” notification is emitted exactly once on initial app startup, even when the primary scene is assigned during scene setup.

Changes:

  • Compute a isFirstScene flag before _setupWindowForScene can assign the primary scene.
  • Gate _notifySceneAppStarted() on both “no primary scene yet” and “app not launched yet” to prevent duplicate launch notifications.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@NathanWalker NathanWalker merged commit f1678ed into main Mar 30, 2026
7 checks passed
@NathanWalker NathanWalker deleted the fix/scene-launch branch March 30, 2026 16:55
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