[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[ ] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native
Summary:
This change is part of a series of cherry-picks intended to bring visionOS support to React Native macOS. Thes
Prior to these commits, `RCTPushNotificationManager used UILocalNotification on iOS, and we ifdef'ed macOS to use to use NSUserNotification. Both are deprecated and replaced with the cross-apple-platform UNNotification API. Let's cherry pick these changes to make adding visionOS support easier.
Notably excluded from these PRs was properly supporting remote notification on macOS. I didn't see an obvious equivalent for that, and I don't think we supported it before either. I'll leave that as a future task.
Test Plan:
RNTester actually doesn't have a test page for PushNotificationManager, so I don't have a good way to test this. I'll leave the build passing as a good sign
Please select one of the following
Summary:
This change is part of a series of cherry-picks intended to bring visionOS support to React Native macOS. Thes
Prior to these commits, `RCTPushNotificationManager used UILocalNotification on iOS, and we ifdef'ed macOS to use to use NSUserNotification. Both are deprecated and replaced with the cross-apple-platform UNNotification API. Let's cherry pick these changes to make adding visionOS support easier.
Notably excluded from these PRs was properly supporting remote notification on macOS. I didn't see an obvious equivalent for that, and I don't think we supported it before either. I'll leave that as a future task.
Test Plan:
RNTester actually doesn't have a test page for PushNotificationManager, so I don't have a good way to test this. I'll leave the build passing as a good sign