Milo: Fix ?limit= param

milo/frontend/view_builder.go

Index: milo/frontend/view_builder.go
diff --git a/milo/frontend/view_builder.go b/milo/frontend/view_builder.go
index 901bbeebe2c4a5c9829cecd85fdb862eb06f69e3..1472fada0a9ddfda33162429763e19cf04c1ff2a 100644
--- a/milo/frontend/view_builder.go
+++ b/milo/frontend/view_builder.go
@@ -13,7 +13,11 @@ import (
 // BuilderHandler is responsible for taking a universal builder ID and rendering
 // the builder page (defined in ./appengine/templates/pages/builder.html).
 func BuilderHandler(c *router.Context, builderID buildsource.BuilderID) {
-	builder, err := builderID.Get(c.Context, c.Params.ByName("limit"), c.Params.ByName("cursor"))
+	limit := 25
+	if tLimit, err := GetLimit(c.Request); err == nil {

[iannucci, 2017/07/19 00:04:57]

make this take a default and not return error

[Ryan Tseng, 2017/07/19 00:12:30]

Done.

+		limit = tLimit
+	}
+	builder, err := builderID.Get(c.Context, limit, c.Request.FormValue("cursor"))
 	// TODO(iannucci, hinoka): make MiloBuild refer to annotation stream by
 	// host/prefix/path instead of by directly pulling it. Do all annotation
 	// stream rendering in the frontend.