Notifications not fired at the right time. #1190

koreem · 2017-01-17T08:28:35Z

My Enviroment
Ionic
Plugin version: de.appplant.cordova.plugin.local-notification 0.8.4
Platform: Android
OS: macOS Sierra
Device manufacturer / model: Samsung Galaxy Tab A
Cordova version (cordova -v): 6.4.0
Cordova platform version (cordova platform ls): Installed platforms: android 6.0.1

Expected Behavior
I have scheduled 3 notifications: 09:10, 09:15, 09:20, they should fire at te right time.

Actual Behavior
They are not fired at the right time, sometimes the first one is fired, the second one isn't and at the last scheduled time it fires two times. It's completely random.

Steps to Reproduce
Scheduling code:

var pushAppointments = [];

pushAppointments.push({
    id: id++,
    title: translator.translate("Reminder appointment"),
    at: at,
    text: appointment.description + ": " + dateToShow,
    icon: "res:///not_icon",
})  

cordova.plugins.notification.local.schedule(pushAppointments);

When I check the pending notifications with:

cordova.plugins.notification.local.getAll(function (notifications) {
    console.log("pending notifications: ");
    console.log(notifications);
});

The 'at' property is correctly set, in my example 1484640600, 1484640900, 1484641200.

The text was updated successfully, but these errors were encountered:

rwillett · 2017-01-17T08:50:14Z

Please use the template in #1188 with more information and update your question.

Rob

koreem · 2017-01-17T09:14:58Z

@rwillett done.

rwillett · 2017-01-17T12:07:44Z

It looks pretty much the same as ours and ours works on Android 6.0.

Our differences:

We have a callback in cordova.plugins.notification.local.schedule(). It doesn't do anything but we have it :)
Exactly how are you creating the at for the at: field? We use

        var d          = new Date();
        var now        = d.getTime();
        var year       = d.getFullYear();
        var month      = d.getMonth();
        var day        = d.getDate();

        var reminder0  = new Date(now + 60 * 1000);
        var reminder1  = new Date(year , month , day , 19 , 30 , 0);
        var reminder2  = new Date(year , month , day , 20 , 30 , 0);
	var reminder3  = new Date(year , month , day , 21 , 0 , 0);

to create four reminders that are scheduled in 60 secs and at 19:30, 20:30 and 21:00 for today.

and we create the array using

	var notifications = [ { id: GetNotificationIdAndIncrement() ,
                                sound: "" , // Turn the sound off
	                        at: reminder0 ,
				data: { type: "CongestionChargeNotification" , original_schedule_time: reminder0.getTime()/1000 ,
                                       	reminder_type: "reminder0_congestion_charge" , order: 1} ,
	                        text: "You've crossed into the Congestion Zone and might need to pay. You'll get more reminders later today."
                             } ,
	                     { id: GetNotificationIdAndIncrement() ,
                               sound: "" , // Turn the sound off
	                       at: reminder1 ,
	                       data: { type: "CongestionChargeNotification" , original_schedule_time: reminder1.getTime()/1000  ,
                                       reminder_type: "reminder1_congestion_charge" , order: 2} ,
                               text: "Reminder 1 - Don't forget to pay the Congestion Charge before midnight."
                             } ,
                             { id: GetNotificationIdAndIncrement() ,
                               sound: "" , // Turn the sound off
                               at: reminder2 ,
                               data: { type: "CongestionChargeNotification" , original_schedule_time: reminder2.getTime()/1000  ,
                                       reminder_type: "reminder2_congestion_charge" , order: 3} ,
                               text: "Reminder 2 - Don't forget to pay the Congestion Charge before midnight."
                             } ,
                             { id:  GetNotificationIdAndIncrement() ,
                               sound: "" , // Turn the sound off
                               at: reminder3 ,
                               data: { type: "CongestionChargeNotification" , original_schedule_time: reminder3.getTime()/1000  ,
                                       reminder_type: "reminder3_congestion_charge" , order: 4} ,
                               text: "Final Reminder - Don't forget to pay the Congestion Charge before midnight."
                             }];

We add the icon separately to the above for a reason that now escapes me.

file://pw_notification.png";

Our icon is a slightly different URL.

My feeling is that you are doing something wrong with the at: value but I can't see what. Post your code to create the at: value and see what that does.

Rob

koreem · 2017-01-17T14:29:41Z

@rwillett thanks for your answer. For testing purposes I tried your example and I still have the same issue. The first reminder was pushed at the right time, the second and last reminders were pushed at 15:26. See below:

var year = d.getFullYear();
var month = d.getMonth();
var day = d.getDate();
            
var reminder1  = new Date(year , month , day , 15 , 22 , 0);
var reminder2  = new Date(year , month , day , 15 , 24 , 0);
var reminder3  = new Date(year , month , day , 15 , 26 , 0);
                    
var pushAppointments = [{
   id: id++,
   title: translator.translate("Reminder appointment"),
   at: reminder1,
   text: "reminder1",
   icon: "res:///not_icon",
   },
   {
   id: id++,
   title: translator.translate("Reminder appointment"),
   at: reminder2,
   text: "reminder2",
   icon: "res:///not_icon",
   },
   {
   id: id++,
   title: translator.translate("Reminder appointment"),
   at: reminder3,
   text: "reminder3",
   icon: "res:///not_icon",
   }
];
					   
cordova.plugins.notification.local.schedule(pushAppointments);

rwillett · 2017-01-17T15:02:03Z

At what time are you running the code?

Are you putting the scheduled time far enough in the future from when you run?

We are also running a later version of the plugin, however I don't think thats the issue as we had this working on Android 6.0 some versions ago.

Getting two alerts at the same time sounds as if something is being overwritten. If this was C I'd start looking for a memory leak :)

You have the code we use in production. I will check its still working on Android 6.0 later today or tomorrow but am busy at the moment.

Can't think of anything else at this time. I'll check its working on our app again and come back.

What other plugins are you running?

Rob

koreem · 2017-01-17T15:22:15Z

Specified code runs when the app goes to the background. For testing purposes I'm scheduling the notifications 5-10 min in the future. When I check the pending notifications, the "at" property specifies the right amount of seconds, so that's a bit strange.

I'm running the following plugins:

com.appgiraffe.plugins.applicationPreferences 0.1.0 "ApplicationPreferences"
com.ionic.keyboard 1.0.4 "Keyboard"
cordova-plugin-android-permissions 0.10.0 "Permissions"
cordova-plugin-app-event 1.2.0 "Application Events"
cordova-plugin-app-preferences 0.7.7 "AppPreferences"
cordova-plugin-app-version 0.1.8 "AppVersion"
cordova-plugin-ble-central 1.1.0 "BLE"
cordova-plugin-compat 1.0.0 "Compat"
cordova-plugin-console 1.0.3 "Console"
cordova-plugin-crosswalk-webview 2.2.0 "Crosswalk WebView Engine"
cordova-plugin-device 1.1.2 "Device"
cordova-plugin-file 4.2.0 "File"
cordova-plugin-file-opener2 2.0.2 "File Opener2"
cordova-plugin-file-transfer 1.5.1 "File Transfer"
cordova-plugin-moove 1.0 "Cordova Plugin MOOVE"
cordova-plugin-screen-orientation 1.4.2 "Screen Orientation"
cordova-plugin-splashscreen 4.0.1 "Splashscreen"
cordova-plugin-touchid 0.3.1 "Touch ID"
cordova-plugin-transport-security 0.1.2 "App Transport Security"
cordova-plugin-whitelist 1.2.2 "Whitelist"
de.appplant.cordova.plugin.local-notification 0.8.4 "LocalNotification"
ionic-plugin-keyboard 2.2.0 "Keyboard"
org.apache.cordova.file 1.3.3 "File"

I will try the same in a new blank Ionic App.

koreem · 2017-01-18T09:16:02Z

I just isolated the problem into a blank Ionic app. I got the exact same problem. I also build the same app on my Samsung Galaxy S7 (Android 6.0.1) and it surprisingly works...

So for now I think I will build my own timing functionality with $timeout.

rwillett · 2017-01-18T09:54:08Z

That's interesting and annoying.

The $timeout won't work on IOS though in the background.

Have you tried scheduling them one at a time.

rwillett · 2017-01-18T10:30:05Z

You could try the later version of the plugin as well.

koreem · 2017-01-18T10:55:55Z

I'm using 0.8.4, that's the latest right?

Scheduling one at a time results in the same behavior, scheduling one at a time in a forEach loop fires all push notifications at once at the last "at" time, despite the fact that the "at" date seems to be right.

rwillett · 2017-01-18T11:27:50Z

There is a 0.8.5 dev branch.

Scheduling one at a time results in the same behavior, scheduling one at a time in a forEach loop fires all push notifications at once at the last "at" time, despite the fact that the "at" date seems to be right.

sounds really odd to me. That sounds like the at: field isn't working or you are overwriting something.

However since it works on a Samsung Galaxy S7 (Android 6.0.1), it does seem to indicate that Samsung have changed something.

I'll test it on an Android 6.0 Nexus

rwillett · 2017-01-18T11:46:17Z

We've just rebuilt a test version on Android 6.0.1, Nexus 5.

We schedule four notifications and they all appeared.

Here is the entire production function directly cut from the source, no changes have been made. The const SEND_CONGESTION_ZONE_NOTIFICATIONS_IMMEDIATELY is used to send local notifications out now as opposed to waiting until the evening.

    function SetupCongestionNotificationReminders() {
        // We do three reminders from 18:30 to 21:00.
        var debugSetupCongestionNotificationReminders = debugFalse;
        var whatAmI = WhatAmI("SetupCongestionNotificationReminders");

        if (debugSetupCongestionNotificationReminders)
        {
            ConsoleLog("SetupCongestionNotificationReminders: called");
        }

        var d          = new Date();
        var now        = d.getTime();
        var year       = d.getFullYear();
        var month      = d.getMonth();
        var day        = d.getDate();

        var reminder0  = new Date(now + 60 * 1000);
        var reminder1  = new Date(year , month , day , 19 , 30 , 0);
        var reminder2  = new Date(year , month , day , 20 , 30 , 0);
        var reminder3  = new Date(year , month , day , 21 , 0 , 0);

        if (SEND_CONGESTION_ZONE_NOTIFICATIONS_IMMEDIATELY)
        {
            reminder0 = new Date(now + 5 * 1000);
            reminder1 = new Date(now + 90 * 1000);
            reminder2 = new Date(now + 180 * 1000);
            reminder3 = new Date(now + 300 * 1000);
        }

        if (debugSetupCongestionNotificationReminders)
        {
            ConsoleLog("Year = " + year + " month " + month + " day " + day);
            ConsoleLog("reminder 1 = " + JSON.stringify(reminder1));
            ConsoleLog("reminder 2 = " + JSON.stringify(reminder2));
            ConsoleLog("reminder 3 = " + JSON.stringify(reminder3));
        }

        var notifications = [ { id: GetNotificationIdAndIncrement() ,
                                sound: "" , // Turn the sound off
                                at: reminder0 ,
                                data: { type: "CongestionChargeNotification" , original_schedule_time: reminder0.getTime()/1000 ,
                                        reminder_type: "reminder0_congestion_charge" , order: 1} ,
                                text: "You've crossed into the Congestion Zone and might need to pay. You'll get more reminders later today."
                             } ,
                             { id: GetNotificationIdAndIncrement() ,
                               sound: "" , // Turn the sound off
                               at: reminder1 ,
                               data: { type: "CongestionChargeNotification" , original_schedule_time: reminder1.getTime()/1000  ,
                                       reminder_type: "reminder1_congestion_charge" , order: 2} ,
                               text: "Reminder 1 - Don't forget to pay the Congestion Charge before midnight."
                             } ,
                             { id: GetNotificationIdAndIncrement() ,
                               sound: "" , // Turn the sound off
                               at: reminder2 ,
                               data: { type: "CongestionChargeNotification" , original_schedule_time: reminder2.getTime()/1000  ,
                                       reminder_type: "reminder2_congestion_charge" , order: 3} ,
                               text: "Reminder 2 - Don't forget to pay the Congestion Charge before midnight."
                             } ,
                             { id:  GetNotificationIdAndIncrement() ,
                               sound: "" , // Turn the sound off
                               at: reminder3 ,
                               data: { type: "CongestionChargeNotification" , original_schedule_time: reminder3.getTime()/1000  ,
                                       reminder_type: "reminder3_congestion_charge" , order: 4} ,
                               text: "Final Reminder - Don't forget to pay the Congestion Charge before midnight."
                             }];

        if (whatAmI === "android_app" || whatAmI === "android_simulator")
        {
            for (var i = 0 ; i < notifications.length ; i++)
            {
                notifications[i]["icon"] = "file://pw_notification.png";
            }
        }

        if (debugSetupCongestionNotificationReminders)
            ConsoleLog("SetupCongestionNotificationReminders: " + JSON.stringify(notifications));

        cordova.plugins.notification.local.schedule(notifications , function () {
            if (debugSetupCongestionNotificationReminders)
                ConsoleLog("SetupCongestionNotificationReminders: schedule callback OK");
        });
    }

We've just simulated a trip into London's congestion zone and got the notifications appear on the notification shade

koreem · 2017-01-18T14:05:48Z

Where can I find the 0.8.5 dev branch?

rwillett · 2017-01-18T14:11:23Z

cordova plugin add https://github.com/katzer/cordova-plugin-local-notifications#ios10

koreem · 2017-01-18T14:42:58Z

Tested the 0.8.5 version. Same results. Working as it should be on my Samsung Galaxy S7, but not on the Samsung Galaxy Tab A.

$rootScope.scheduleNotifications = function() {
      
      var d          = new Date();
      var year       = d.getFullYear();
      var month      = d.getMonth();
      var day        = d.getDate();

      var reminder1  = new Date(year , month , day , 15 , 40 , 0);
      var reminder2  = new Date(year , month , day , 15 , 41 , 0);
      var reminder3  = new Date(year , month , day , 15 , 42 , 0);
      
      var pushAppointments = [{
          id: 1,
          title: "Reminder appointment",
          at: reminder1,
          text: "reminder1",
          },
          {
          id: 2,
          title: "Reminder appointment",
          at: reminder2,
          text: "reminder2",
          },
          {
          id: 3,
          title: "Reminder appointment",
          at: reminder3,
          text: "reminder3",
      }];
      
      cordova.plugins.notification.local.schedule(pushAppointments , function () {
        console.log("notifications set!")    
      });
		
      $timeout(function() {
        cordova.plugins.notification.local.getAll(function (notifications) {
          console.log("pending notifications: ");
          console.log(notifications);
        });
      }, 5000);
}

rwillett · 2017-01-18T14:48:13Z

This is a Samsung issue then.

Also we used to put stuff in $timeout functions. We found that it wasn't needed AND it caused race conditions when rescheduling and reworking the local notifications.

Rob

koreem · 2017-01-18T14:58:33Z

Ok, I will use my own timing functionality then. Thanks anyway! :)

nadirzia · 2019-04-22T13:05:24Z

Ionic Local Notification does not trigger sometimes or does not trigger other times. Sometimes it triggers before time set time.

I am facing a different scenario

When a schedule notification is set for 5 minute and app in background. Then sometime notification trigger before the finish time randomly e.g(4 minute or sometime 4.5 minute)

When a schedule notification set for 5 minute and the app is left running in background. Then sometimes the notification is triggered but sometimes it is not.

below is my code
this.myDate = new Date();
this.triggerDate = new Date(this.myDate.setSeconds(this.myDate.getSeconds() + 300));

this.localNotifications.schedule({
  id: 1,
  title: 'mylimit',
  text: 'Please provide eat enjoyment and guilt experience' + new Date(this.triggerDate),
  smallIcon: 'resources/icon.png',
  data: { "mydata": this.stateType, "createdTime": this.myDate },
  trigger: {at: new Date(this.triggerDate)},
});

rwillett mentioned this issue Jan 18, 2017

Just Fires Once #1192

Closed

rwillett mentioned this issue Jan 25, 2017

Notification not triggered on ios #1200

Closed

rwillett closed this as completed Feb 15, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Notifications not fired at the right time. #1190

Notifications not fired at the right time. #1190

koreem commented Jan 17, 2017 •

edited

Loading

rwillett commented Jan 17, 2017 •

edited

Loading

koreem commented Jan 17, 2017

rwillett commented Jan 17, 2017

koreem commented Jan 17, 2017 •

edited

Loading

rwillett commented Jan 17, 2017

koreem commented Jan 17, 2017 •

edited

Loading

koreem commented Jan 18, 2017 •

edited

Loading

rwillett commented Jan 18, 2017

rwillett commented Jan 18, 2017

koreem commented Jan 18, 2017 •

edited

Loading

rwillett commented Jan 18, 2017

rwillett commented Jan 18, 2017

koreem commented Jan 18, 2017

rwillett commented Jan 18, 2017

koreem commented Jan 18, 2017 •

edited

Loading

rwillett commented Jan 18, 2017

koreem commented Jan 18, 2017

nadirzia commented Apr 22, 2019 •

edited

Loading

Notifications not fired at the right time. #1190

Notifications not fired at the right time. #1190

Comments

koreem commented Jan 17, 2017 • edited Loading

rwillett commented Jan 17, 2017 • edited Loading

koreem commented Jan 17, 2017

rwillett commented Jan 17, 2017

koreem commented Jan 17, 2017 • edited Loading

rwillett commented Jan 17, 2017

koreem commented Jan 17, 2017 • edited Loading

koreem commented Jan 18, 2017 • edited Loading

rwillett commented Jan 18, 2017

rwillett commented Jan 18, 2017

koreem commented Jan 18, 2017 • edited Loading

rwillett commented Jan 18, 2017

rwillett commented Jan 18, 2017

koreem commented Jan 18, 2017

rwillett commented Jan 18, 2017

koreem commented Jan 18, 2017 • edited Loading

rwillett commented Jan 18, 2017

koreem commented Jan 18, 2017

nadirzia commented Apr 22, 2019 • edited Loading

koreem commented Jan 17, 2017 •

edited

Loading

rwillett commented Jan 17, 2017 •

edited

Loading

koreem commented Jan 17, 2017 •

edited

Loading

koreem commented Jan 17, 2017 •

edited

Loading

koreem commented Jan 18, 2017 •

edited

Loading

koreem commented Jan 18, 2017 •

edited

Loading

koreem commented Jan 18, 2017 •

edited

Loading

nadirzia commented Apr 22, 2019 •

edited

Loading