qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-22 11:14:47 -07:00
Code Releases Activity

misc/dashboard/codereview: set In-Reply-To header to properly thread mail.

Browse Source 
R=adg
CC=golang-dev
https://golang.org/cl/6208051
This commit is contained in:
David Symonds 2012-05-14 10:05:39 +10:00
parent be15a8f2dc
commit 58bcec62c0
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
misc/dashboard/codereview/dashboard/cl.go
View File

@ -12,6 +12,7 @@ import (
"html/template" "html/template"
"io" "io"
"net/http" "net/http"
netmail "net/mail"
"net/url" "net/url"
"regexp" "regexp"
"sort" "sort"
@ -192,8 +193,11 @@ func handleAssign(w http.ResponseWriter, r *http.Request) {
Subject: cl.Subject + " (issue " + n + ")", Subject: cl.Subject + " (issue " + n + ")",
Body: "R=" + rev + "\n\n(sent by gocodereview)", Body: "R=" + rev + "\n\n(sent by gocodereview)",
} }
// TODO(dsymonds): Use cl.LastMessageID as the In-Reply-To header if cl.LastMessageID != "" {
// when the appengine/mail package supports that. msg.Headers = netmail.Header{
"In-Reply-To": []string{cl.LastMessageID},
}
}
if err := mail.Send(c, msg); err != nil { if err := mail.Send(c, msg); err != nil {
c.Errorf("mail.Send: %v", err) c.Errorf("mail.Send: %v", err)
} }