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

Replace email functions with apprise notification functions. #29

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

eperdeme
Copy link

@eperdeme eperdeme commented Mar 7, 2020

Just playing around but through I'd share this minor change.

It should allow most methods of notifications (email,slack,pushbullet,etc).

Need a bit more work still.

@xontik
Copy link

xontik commented May 11, 2020

PLaying with apprise and this script too !

@eperdeme
Copy link
Author

eperdeme commented May 12, 2020

I've been using my fork for a while, works fine using it with pushbullet but the apprise yml should work with most providers, but I've not tested every one, so can't be 100% sure on the change.

Also my pythons mostly 'glue-level' python (copy and paste expert) :)

@Chronial Chronial added notfication-service out-of-scope This feature is of scope for the (narrow) goals of snapraid-runner labels Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notfication-service out-of-scope This feature is of scope for the (narrow) goals of snapraid-runner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants