search

react-native-push-notification / ios

React Native Push Notification API for iOS.
MIT License
740 stars 284 forks source link

addNotificationRequest() only repeats daily #263

Open ninjz opened 3 years ago

ninjz commented 3 years ago

From the looks of it, addNotificationRequest() only allows repeating a notification daily. This means losing functionality to repeat weekly, monthly, yearly, etc., which I'm sure a lot of apps depend on.

Looking at the code it looks like it shouldn't be too hard to extend the capabilities of addNotificationRequest() to support the same parameter repeatInterval by only adding the necessary flags in the date components.

I'd attempt this if I had time (which unfortunately I don't) and am ok with using the deprecated methods for now. If someone here would be able to add this functionality that would be great.

daheeahn commented 3 years ago

This feature is really important!

jamesxabregas commented 3 years ago

From what I've tested daily repeating doesn't even work properly through addNotificationRequest(). Only works with scheduleLocalNotification() at the moment. It seems to me the challenge is in abstracting the UNNotificationTrigger defintion.