katzer / cordova-plugin-local-notifications

Cordova Local-Notification Plugin
Apache License 2.0
2.57k stars 1.75k forks source link

Takes too long to show notification #1227

Closed deepakbandela closed 7 years ago

deepakbandela commented 7 years ago

Your Environment

Plugin version: 0.8.4 Platform: Android OS version: 6.0 Device manufacturer / model: Moto G3 / Motorola Cordova version (cordova -v): 6.4.0

Cordova platform version (cordova platform ls): Your system information: Cordova CLI: 6.5.0 Ionic Framework Version: 2.0.0-rc.5 Ionic CLI Version: 2.2.1 Ionic App Lib Version: 2.2.0 Ionic App Scripts Version: 1.0.0 ios-deploy version: 1.9.1 ios-sim version: 5.0.13 OS: macOS Sierra Node Version: v6.10.0 Xcode version: Xcode 8.2.1 Build version 8C1002

Plugin config

Expected Behavior Local Notification takes around 5-6 seconds to display

Tell us what should happen Local Notification should be displayed instantly

Actual Behavior slow notification

Tell us what happens instead

Steps to Reproduce just a notification scheule takes long time

Reproduce this issue; include code to reproduce, if relevant LocalNotifications.schedule({ id: 1, text: 'Single ILocalNotification', title: 'title', data: { secret: key } });

Context Want to show notifications on my events

rwillett commented 7 years ago

Fill out the template in #1188.

As an aside Android doesn't guarantee immediate delivery.

deepakbandela commented 7 years ago

Updated the defect description Looks like we are using an alarm manager to schedule a notification.

Can we have something which quickly trigger the notification without alarm manager or rescheduling etc.

rwillett commented 7 years ago

Thanks for this, but as I said before, Android doesn't guarantee immediate delivery.

It would be interesting to know what slow means. 0.1 sec late, 1 sec late, 10 secs late, 1 minute late, 10 minutes late etc. One person's slow is another persons fast.

We use the local notifications and we see Android coming in 1-3 minutes behind the same test on IOS. We're not that bothered as our use case doesn't need millisec accuracy. Since push notifications aren't guaranteed to be on time either, if you require a notification to be highly accurate (whatever that means), you might be out of luck.

rwillett commented 7 years ago

Closed as notification plugin working as designed.