Enhancement - Support for Pushover.net

[scottbindas]

For those IOS users, PushBullet no longer offers an IOS App. Pushover.net supports both IOS/Google apps.

It would be desirable that to also expose their "Message Priority" function so that for a larger county that has many FD's you can push FD Alerts for surrounding departments but allow them to be configured where a silent alert is all that is needed.

[Joey40]

Unfortunate that PushBullet decided to drop support for iOS and apparently never revisited this decision. Would love to see support for an alternative.

[kdolan]

This is planned

[ComputerComa]

Just wanted to note I have kinda started working on this as it is a feature that I would like to see.
You can follow my progress here.
I'm not sure if I am going about it correctly or not, so feel free to contact with suggestions or advice.
Really love the project!
Thanks, ComputerComa

[kdolan]

Looks good so far! You will just need to finish implementing notifiers/index.js and add some tests and I would be happy to get that merged!

[ComputerComa]

Yeah, that was where I kinda hit a wall, I'm not sure I quite understand how to actually finish implementing it.

[kdolan]

@ComputerComa You are SOO close.

1. Add a method to https://github.com/ComputerComa/fd-tone-notify/blob/daa2e3c7e22c5e771920a6cf9c6b9a80f4f18aff/notifiers/index.js called _pushoverNotifications(params, prePostType)
2. Add method to https://github.com/ComputerComa/fd-tone-notify/blob/daa2e3c7e22c5e771920a6cf9c6b9a80f4f18aff/obj/NotificationParams.js for getPushovers()
3. Finish implementing the _pushoverNotifications(params, prePostType) method. You should be able to pretty closely copy the pattern for PushBullet.
4. Test your code and see if it works!

[ComputerComa]

I think I'm pretty much finished. I'm not quite sure I have it implemented right though. Pushover registers as a notifier but never actually fires off. I did a test with the included tone examples and it catches it, (I edited the included example config to use pushover as the only notifier) but it just says in the debug logs that there are no PRE or POST notifications for pushover.

[kdolan]

@ComputerComa It looks like you just forgot to add the new methods to the pre/post notifications methods.
If you add here and here those notifications should start processing.

[ghost]

I think I'm pretty much finished. I'm not quite sure I have it implemented right though. Pushover registers as a notifier but never actually fires off. I did a test with the included tone examples and it catches it, (I edited the included example config to use pushover as the only notifier) but it just says in the debug logs that there are no PRE or POST notifications for pushover.

Do you still have the code? I am trying to setup my whole county however I realized that pushbullet is not on iOS

[ComputerComa]

I think I'm pretty much finished. I'm not quite sure I have it implemented right though. Pushover registers as a notifier but never actually fires off. I did a test with the included tone examples and it catches it, (I edited the included example config to use pushover as the only notifier) but it just says in the debug logs that there are no PRE or POST notifications for pushover.

Do you still have the code? I am trying to setup my whole county however I realized that pushbullet is not on iOS

I unfortunately don't anymore. It was on an old laptop that I no longer have