diff options
| author | David Symonds <dsymonds@golang.org> | 2012-07-30 14:41:04 +1000 |
|---|---|---|
| committer | David Symonds <dsymonds@golang.org> | 2012-07-30 14:41:04 +1000 |
| commit | 6a5773bf4e7c1de4248ce98a5505b0286d825566 (patch) | |
| tree | 7f4c6cfc3197226b99b2e9f62f67339cc29264c8 /misc/dashboard | |
| parent | c4779f9efdb93c71f4f1818f6d68e006ae6b1344 (diff) | |
| download | go-6a5773bf4e7c1de4248ce98a5505b0286d825566.tar.gz | |
misc/dashboard/codereview: pass user information to gobot when assigning reviewer.
R=bradfitz
CC=gobot, golang-dev, rsc
http://codereview.appspot.com/6457052
Diffstat (limited to 'misc/dashboard')
| -rw-r--r-- | misc/dashboard/codereview/dashboard/cl.go | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/misc/dashboard/codereview/dashboard/cl.go b/misc/dashboard/codereview/dashboard/cl.go index 866594776..181d2e40f 100644 --- a/misc/dashboard/codereview/dashboard/cl.go +++ b/misc/dashboard/codereview/dashboard/cl.go @@ -128,7 +128,8 @@ func handleAssign(w http.ResponseWriter, r *http.Request) { } u := user.Current(c) - if _, ok := emailToPerson[u.Email]; !ok { + person, ok := emailToPerson[u.Email] + if !ok { http.Error(w, "Not allowed", http.StatusUnauthorized) return } @@ -183,7 +184,7 @@ func handleAssign(w http.ResponseWriter, r *http.Request) { if !found { c.Infof("Adding %v as a reviewer of CL %v", rev, n) - url := fmt.Sprintf("%s?cl=%s&r=%s", gobotBase, n, rev) + url := fmt.Sprintf("%s?cl=%s&r=%s&obo=%s", gobotBase, n, rev, person) resp, err := urlfetch.Client(c).Get(url) if err != nil { c.Errorf("Gobot GET failed: %v", err) |