Unity-Technologies / com.unity.mobile.notifications

Mobile Notifications Package
https://docs.unity3d.com/Packages/com.unity.mobile.notifications@2.1/manual/index.html
Other
131 stars 42 forks source link

Minor iOS cleanup #324

Closed aurimasc closed 1 month ago

aurimasc commented 1 month ago

Notifications did have UnityAppController expansion that had no use. This has been removed along with header file. .mm file was renamed to match the class it contains. Also we no longer create and instance of UnityNotificationLifeCycleManager as we don't use it in any way. Only hook to lifecycle notifications when framework is loaded. The entire PR is a simple reduction of code. The only affected area is the initialization part. Tested manually on iPhone 12 Pro Max that local notifications still work. Tested on simulator that push notifications are still received correctly. iOS only, not specific to version and very isolated (initialization part that happens automatically on app start).