Issue 2035: Add email notifier with per show notification lists #428

Open
wants to merge 1 commit into
from

4 participants

@Slugger

This patch provides email notifications, with support for TLS and SMTP AUTH. It provides the regular notifications on snatch and download. It provides the ability to give a list of "global" emails, which receive notifications for all shows and it also allows configuration of email notification lists on a per show basis. This addresses Google Code issue 2035. Screenshot attached to that issue.

Tested using Gmail's SMTP server (TLS + SMTP AUTH) and my ISP's SMTP server (no TLS, no AUTH). Both worked as expected.

Tested on my (relatively old) Linux VM (Ubuntu) using python 2.6.5 and Cheetah 2.0.1. Also tested on WinXP using python 2.7.3 and Cheetah 2.4.4.

Changes are pretty standard except I added a new column to the tv_shows table, which stores the list of email addresses configured for each show.

@xombiemp

I just tested this with my gmail account and it works great! Please merge this!

@thezoggy

new code needs to be based off the development branch, not master

@Slugger
@Nyghto

Is email notification going to be implemented? really needing this:)

@xombiemp xombiemp referenced this pull request Jan 25, 2013
Open

Email Notifications #569

@xombiemp

I created a new pull request for this against the development branch. #569

@Nyghto

Much thanks, looking forward to it. couch potato also finally has email notification.

@thezoggy

why not just use sab's email notification ability?

@xombiemp

I don't use sab. I use rutorrent.

@Nyghto

i am using sabs mail notifications, but things are changing.. So want couch (it is possible with couch) and sickbeard only send mail upon complete download.

@jmichelgarcia jmichelgarcia referenced this pull request in xombiemp/Sick-Beard Oct 12, 2013
Closed

Email Notifications #1

@bricky bricky pushed a commit that referenced this pull request Nov 10, 2013
@xombiemp xombiemp Email Notifications
"This patch provides email notifications, with support for TLS and SMTP
AUTH. It provides the regular notifications on snatch and download. It
provides the ability to give a list of "global" emails, which receive
notifications for all shows and it also allows configuration of email
notification lists on a per show basis."

This is Sluggers pull request from
#428

I made the changes in the Development branch and changed a few things
to get it to work with the current version of SickBeard.  I've tested
it and it seems to be working great.
ee34671
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment