Skip to content

Commit

Permalink
fix(messaging, ios): resolve getAPNSToken promise in all cases
Browse files Browse the repository at this point in the history
Fixes #7272 with thanks to @appa-gomi !
  • Loading branch information
mikehardy committed Feb 7, 2024
1 parent 65a9763 commit e4f2541
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions packages/messaging/ios/RNFBMessaging/RNFBMessagingModule.m
Original file line number Diff line number Diff line change
Expand Up @@ -173,8 +173,8 @@ - (NSDictionary *)constantsToExport {
resolve([NSNull null]);
return;
#endif
DLog(@"RNFBMessaging getAPNSToken - ARM64 Simulator detected, but no APNS token set. Assuming "
@"APNS token is possible. macOS13+ / iOS16+ / M1 mac required for assumption to be valid. "
DLog(@"RNFBMessaging getAPNSToken - ARM64 Simulator detected, but no APNS token available. "
@"APNS token may be possible. macOS13+ / iOS16+ / M1 mac required for assumption to be valid. "
@"Use setAPNSToken in testing if needed.");
#endif
if ([UIApplication sharedApplication].isRegisteredForRemoteNotifications == NO) {
Expand All @@ -188,6 +188,7 @@ - (NSDictionary *)constantsToExport {
}];
return;
}
resolve([NSNull null]);
}
}

Expand Down

0 comments on commit e4f2541

Please sign in to comment.