Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(787)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: milo/api/resp/build.go

Issue 2982183002: Milo: Store console defs as their own entities (Closed)
Patch Set: tests Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « milo/api/config/project.pb.go ('k') | milo/buildsource/buildbot/pubsub_test.go » ('j') | milo/buildsource/buildbot/pubsub_test.go » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: milo/api/resp/build.go
diff --git a/milo/api/resp/build.go b/milo/api/resp/build.go
index 1cf550d120f6338d16d7dbb24ac4c38e93e931fa..95d505bb9c5deb9031ad513735d336e016e5179a 100644
--- a/milo/api/resp/build.go
+++ b/milo/api/resp/build.go
@@ -313,7 +313,7 @@ func (rb *MiloBuild) SummarizeTo(c context.Context, bs *model.BuildSummary) erro
// definitions will specify their manifest as "REVISION", and we'll do
// lookups with null URL fields.
bs.AddManifestRevisionIndex(
- con.ProjectID, con.Console.Name, "REVISION", "", revisionBytes)
+ con.GetProjectName(), con.ID, "REVISION", "", revisionBytes)
}
}
}
« no previous file with comments | « milo/api/config/project.pb.go ('k') | milo/buildsource/buildbot/pubsub_test.go » ('j') | milo/buildsource/buildbot/pubsub_test.go » ('J')

Powered by Google App Engine
This is Rietveld 408576698