darkkirb/harness-drone
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge pull request #277 from aadavids/email_subject

Browse source 
Gmail threading of email notifications
This commit is contained in:
Brad Rydzewski 2014-04-18 11:22:38 -07:00
commit b005576068
2 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
pkg/mail/mail.go
View file

@ -74,9 +74,9 @@ func SendPassword(to string, data interface{}) error {
}
// Sends a build success email to the user.
func SendSuccess(repo, to string, data interface{}) error {
func SendSuccess(repo, sha, to string, data interface{}) error {
msg := Message{}
msg.Subject = "[SUCCESS] " + repo
msg.Subject = fmt.Sprintf("[%s] SUCCESS building %s", repo, sha)
msg.To = to
var buf bytes.Buffer
@ -91,9 +91,9 @@ func SendSuccess(repo, to string, data interface{}) error {
}
// Sends a build failure email to the user.
func SendFailure(repo, to string, data interface{}) error {
func SendFailure(repo, sha, to string, data interface{}) error {
msg := Message{}
msg.Subject = "[FAILURE] " + repo
msg.Subject = fmt.Sprintf("[%s] FAILURE building %s", repo, sha)
msg.To = to
var buf bytes.Buffer

4
pkg/plugin/notify/email.go
View file

@ -26,7 +26,7 @@ func (e *Email) Send(context *Context) error {
func (e *Email) sendFailure(context *Context) error {
// loop through and email recipients
for _, email := range e.Recipients {
if err := mail.SendFailure(context.Repo.Name, email, context); err != nil {
if err := mail.SendFailure(context.Repo.Name, context.Commit.HashShort(), email, context); err != nil {
return err
}
}
@ -38,7 +38,7 @@ func (e *Email) sendFailure(context *Context) error {
func (e *Email) sendSuccess(context *Context) error {
// loop through and email recipients
for _, email := range e.Recipients {
if err := mail.SendSuccess(context.Repo.Name, email, context); err != nil {
if err := mail.SendSuccess(context.Repo.Name, context.Commit.HashShort(), email, context); err != nil {
return err
}
}