-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enhancement - Support for Pushover.net #6
Comments
Unfortunate that PushBullet decided to drop support for iOS and apparently never revisited this decision. Would love to see support for an alternative. |
This is planned |
Just wanted to note I have kinda started working on this as it is a feature that I would like to see. |
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! |
Yeah, that was where I kinda hit a wall, I'm not sure I quite understand how to actually finish implementing it. |
@ComputerComa You are SOO close.
|
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. |
@ComputerComa It looks like you just forgot to add the new methods to the pre/post notifications methods. |
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 |
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.
The text was updated successfully, but these errors were encountered: