Skip to content
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

Pushover notification service #5

Merged
merged 6 commits into from
Nov 8, 2017
Merged

Conversation

danialmoghaddam
Copy link
Contributor

Just added Pushover notification service to send out notification when there is a force reboot.

@Kasmetski
Copy link
Owner

Thank you for the pull request.
I need to make a few changes before commit.
I assume you already tested it?

@danialmoghaddam
Copy link
Contributor Author

@Kasmetski First let me thank you for sharing this application with the community.

Yes I did some extensive tests and everything is working pretty well.

img_6918

@Kasmetski
Copy link
Owner

Perfect. Can I use this photo for the documentation when I have more time to write it?

I've implemented Telegram remote control but I forgot to push it to github. Now I think to use the Notif() function to send notifications for all implemented services - pushover, telegram (slack soon & etc.)

Thanks again for the commit.

@danialmoghaddam
Copy link
Contributor Author

Absolutely. Feel free to use it if it suits the need. I'll help to push more info on "how to connect the PW switch to the relay". Noobs like me would get confused on NO, NC ports on relay.

@Kasmetski Kasmetski merged commit 08ccef8 into Kasmetski:master Nov 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants