Closed wegotourselvesareader closed 2 years ago
I figured I'd give it a go to see if I could work out how to do it. I edited sabnzbd-notify.py to add a few lines into the notification map, between 'startup' and 'download':
SABNZBD_NOTIFICATION_MAP = {
# Startup/Shutdown
'startup': (
'Startup/Shutdown',
'info',
),
# Pause / Resume
'pause_resume': (
'Pause/Resume',
'info',
),
# Added NZB
'download': (
'Added NZB',
'info',
),
# Post-processing started
[...]
I tested it and got both "Paused" and "Resuming" notifications.
Great work! If you want to do a PR you can have all of the credit! I also don't mind doing one myself, but it won't probably be until this weekend. :slightly_smiling_face:
thanks for sharing all of your success! :rocket:
I've never done a pull or push in GitHub (rank novice, me), but I can give it a go. I've got some time towards the end of the week; I'll take a look at it then.
I've never done a pull or push in GitHub (rank novice, me), but I can give it a go. I've got some time towards the end of the week; I'll take a look at it then.
That was easier and quicker than I thought it might be. Done!
Fantastic! It's now merged into the master branch! Great work! Thanks for the PR!
:bulb: The Idea
SABnzbd offers the option to send a notification on pausing/resuming the queue:
However, sabnzbd-notify.py throws an error if the "Pause / Resume" option is enabled:
Would it be possible to add support for the
pause_resume
notification, please?:hammer: Breaking Feature
I don't believe this would break anything, only augment the existing functionality. All the other notification types are supported by sabnzbd-notify.py; it's just
pause_resume
which isn't.