Skip to content

Clarify swizzling requirement for APNs token registration in AppDelegate #1740

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
+3 −1
Open

Clarify swizzling requirement for APNs token registration in AppDelegate #1740

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3d57509
Clarify swizzling requirement for APNs token registration in AppDelegate
Preacher999987 Jul 30, 2025
1c40c1a
fix: Swift file formatting
Preacher999987 Jul 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions messaging/MessagingExampleSwift/AppDelegate.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,8 +99,11 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
didFailToRegisterForRemoteNotificationsWithError error: Error) {
print("Unable to register for remote notifications: \(error.localizedDescription)")
}

// This function is added here only for debugging purposes, and can be removed if swizzling is enabled.

Copy link

Choose a reason for hiding this comment

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

medium

This line contains a tab character, which is likely unintentional for a blank line. Swift style guides, such as the Google Swift Style Guide, recommend using spaces over tabs for consistency. Mixed whitespace can lead to formatting issues across different editors and environments. Please remove the tab character to make this a truly empty line.

// NOTE: When method swizzling is enabled (default), this method must still be explicitly declared.
// Firebase Messaging uses it to intercept and register the APNs device token internally.
// If omitted, the device token will not be received.

// If swizzling is disabled then this function must be implemented so that the APNs token can be paired to
// the FCM registration token.
func application(_ application: UIApplication,
Expand Down