Skip to content

feat: migrate legacy UIKit samples to UIScene lifecycle - #1883

Merged
ncooke3 merged 1 commit into
mainfrom
nc.uiscene-migration
Oct 28, 2026
Merged

ncooke3 merged 1 commit into
mainfrom
nc.uiscene-migration

Conversation

@ncooke3

@ncooke3 ncooke3 commented Oct 28, 2026

Copy link
Copy Markdown
Member

Starting with the iOS 27 SDK in Xcode 27, UIKit enforces UIScene lifecycle adoption at launch and terminates apps that still rely on the legacy UIApplicationDelegate window lifecycle (SIGTRAP in _UIApplicationEvaluateRuntimeIssueForNoSceneLifecycleAdoption).

This PR migrates the five remaining UIKit quickstart targets (installations, database, firestore, inappmessaging, and messaging) to UIWindowScene using a SceneDelegate and a static UIApplicationSceneManifest in Info.plist, preserving backward compatibility with Xcode 26 while unblocking Xcode 27 simulator tests.

@ncooke3
ncooke3 added this pull request to stack #1884 October 28, 2026 15:52

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request migrates multiple Firebase example applications (Database, Firestore, InAppMessaging, Installations, and Messaging) to support the UIScene-based lifecycle. It introduces a new SceneDelegate.swift file to each project, configures the UIApplicationSceneManifest in their respective Info.plist files, registers the new scene delegates in the Xcode project files, and removes the unused window property from the AppDelegate classes. There are no review comments to address, and the changes look correct and consistent across all projects.

@ncooke3
ncooke3 requested review from andrewheard and daymxn October 28, 2026 16:46
@ncooke3
ncooke3 force-pushed the nc.uiscene-migration branch from abc6f18 to 5eb4869 Compare October 28, 2026 17:05
Base automatically changed from nc.ci.xcode-matrix to main October 28, 2026 17:07
@ncooke3
ncooke3 force-pushed the nc.uiscene-migration branch from 5eb4869 to a11eb55 Compare October 28, 2026 17:07
@ncooke3

ncooke3 commented Oct 28, 2026

Copy link
Copy Markdown
Member Author

Firestore failure is expected.

#1882 (comment)

@ncooke3
ncooke3 enabled stacked auto-merge (squash) October 28, 2026 17:25
@ncooke3
ncooke3 merged commit edcd190 into main Oct 28, 2026
31 of 32 checks passed
@ncooke3
ncooke3 deleted the nc.uiscene-migration branch October 28, 2026 17:39
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.

2 participants