Read the statement by Michael Teeuw here.
Phone Notifications
-
@rchase android seems to work fine. As long as there’s a pushbullet app for iPhone it should work. I don’t even use the app on my phone.
-
How can I reset?
Notifications always keep.
Also The second notification is output hundreds or thousands of times.
What should I do?? -
Hi,
I’ve just installed your module for use with an iPhone. As I understand it, push-functionality is somewhat limited with Apple, but what can I expect this module to receive? Facebook messenger? Email? Calls/texts? Currently it doesn’t look like it’s receiving anything, but that may be because i’ve configured it wrong!
Cheers!
-
So I had a few issues with this module that I think I was able to fix. The first issue I had was that the second notification gets posted several times. Second, when a notification from an app is already on the list, if a new notification from that app comes in it does not update or push it to the top. Finally, the top of the list is the oldest notification, not the newest. I came up with a solution that seems to work for me so far.
In the phone_notification.js file update the cleanPayload function to look like this
cleanPayload: function(newPayload){ var application_name = newPayload.application_name; var that = this; var dupIndex = 0; if(this.payload.length > 0) { this.payload.forEach(function (m) { // If application_name already exists, increment notification count if(m.application_name === application_name) { m.count++; that.payload.splice(dupIndex,1); } dupIndex++; }); } this.payload.unshift(newPayload); this.payload.slice(0, this.config.displayCount -1); },
Let me know if this helps or if you have any issues with it.
-
@tnorling Hello there! I had almost given up hope on anyone turning up to help. Thanks so much. But just to follow-up I´ve replaced the CleanPayload function with yours and it seems to only allow one notification to appear, while ignoring others. The notification count also does not seem to increase despite giving myself missed calls. Thanks once again. Perhaps there are other segments that you have edited to make this work?
-
@SypherDaemon Interesting…what did you set your numberOfNotifications to in the config file? I have mine set to 3 and I am indeed showing 3 notifications at a time. I also just finished writing a function to remove notifications once they’ve been swiped away from your phone. Maybe I changed something somewhere else as well that made it all work. I’ll post my current working code shortly for you to test out.
Edit: The count may no longer work because I am now replacing notifications coming from the same app. I’ll see what I can do to fix that next…
-
@tnorling It’s fully working now, it was a mistake on my part. And yes I fully understand why the notification count isn’t increasing now, I was just using it to debug and see if the program was still accepting multiple notifications. Not really a loss from my end, as I’m not planning on using the notification count. But again, thank you so much. I’m gonna keep testing it out to see if there aren’t any unforseen side-effects.
-
@SypherDaemon No problem! I’ve been wanting to use this for my own project so I’ve been trying to fix the issues I see with it. Most of the work was done by others, they deserve the credit on this one. I forked the original repo and uploaded my work if you’d like the dismissal feature as well.
-
@tnorling I’m doing the same over on this side. I will definitely check it out. Good luck in your project!
Edit: It turns out the problem earlier with the m.count++ was my own doing, my own mistake. Haha.
-
@tnorling excellent work! Glad you could figure it out.
Once you’re good to go, would you mind submitting a pull request?