misc/dashboard/codereview: add LastUpdate field to CL

To be filled in by a later CL. I deployed a test version to
App Engine to work on setting the values, so there are
a few records that have this field set already.

That field is breaking the live version, so I have pushed
a new copy with this 1-line change to the live version

I assumed that appengine/datastore was like every other
marshaling and unmarshaling package we have in Go
(for example, encoding/gob, encoding/json, encoding/xml,
and protobuf) and that if it loaded an unknown field it would
just ignore it. Apparently not. Sorry.

R=dsymonds
TBR=dsymonds
CC=golang-dev
https://golang.org/cl/6454064

diff --git a/misc/dashboard/codereview/dashboard/cl.go b/misc/dashboard/codereview/dashboard/cl.go
index 181d2e40feb65917a3f520f1eb6478b0d23f9100..c9cee2452713fe775cd5c1c12db2754482925318 100644 (file)
--- a/misc/dashboard/codereview/dashboard/cl.go
+++ b/misc/dashboard/codereview/dashboard/cl.go
@@ -49,6 +49,7 @@ type CL struct {
        FirstLine   string `datastore:",noindex"`
        LGTMs       []string
        NotLGTMs    []string
+       LastUpdate  string
 
        // Mail information.
        Subject       string   `datastore:",noindex"`