Replace deprecated local notification methods on iOS

[nbolender]

As of push-notification-ios@v1.7.0, these iOS methods are deprecated:

• presentLocalNotification
• scheduleLocalNotification
• cancelAllLocalNotifications
• getScheduledLocalNotifications

This PR replaces those methods appropriately on iOS without changing the API of react-native-push-notification.

This fixes #1716

2 Breaking Changes

alertAction is deprecated as of iOS 10 and is not available in the new addNotificationRequest method.

It should also be noted that push-notification-ios@v1.4.1 had an unmentioned breaking change, requiring users to update their AppDelegate to use UNUserNotificationCenter and to stop using application:didReceiveLocalNotification: to receive notifications (this caused undefined notification details similar to #1488). I opened an issue over there to hopefully update their documentation.

These changes will require the same update, so it should be mentioned as a breaking change (though this has been true since 8/18/2020)

Depends on update to push-notification-ios

This depends on react-native-push-notification/ios#237 so I have this marked as draft for now.

[Dallas62]

Hi @nbolender

Thanks for this PR !
This is really helpful !

[nbolender]

This is now ready for review; Requires @react-native-community/push-notification-ios@^1.7.4

[Dallas62]

This parameter doesn't exist in the iOS library, it's repeats now but it's not the same thing 😞

[nbolender]

I changed it to repeats, and updated the documentation to state that 'day' is the only available option for repeatType on iOS. This would be another breaking change.

It is possible for the iOS library to specify other repeat intervals, but as explained here it is not as straightforward. For example, I could not set a fire date of 8 days from now and a weekly repeat; it would instead start firing 1 day from now.

[Dallas62]

Will wait for an answer for this issue, there is no way to cancel a notification request by the id:
react-native-push-notification/ios#239

Nice work, this is working great !
I will also look for other PR such as #612 which is now available on iOS.

[Dallas62]

Hi,
The version 1.8.0 is available with the missing method:
react-native-push-notification/ios#242

https://github.com/react-native-push-notification-ios/push-notification-ios#removependingnotificationrequests

Just an error in the exemple code to take care.

[nbolender]

I've made this change, but also noted the documentation for cancelAllLocalNotifications. The docs say that this cancels all pending and delivered notifications. When using notification requests, it now cancels only pending notifications. There is another method to cancel delivered notifications; should we call this as well within cancelAllLocalNotifications?

I'm just not sure of the behavior on the Android side.

[Dallas62]

I will dig into it this week-end 😉

[Dallas62]

Thanks for allo these changes !
Awesome 🙌

Will look to another PR for Android and release a new version ASAP