[monorail go api] add IssuesList rpc

go/src/infra/monorail/monorail.proto

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file *partially* describes Monorail API.
 // It is derived from
 // https://chromium.googlesource.com/infra/infra/+/f2e2a4d/appengine/monorail/services/api_svc_v1.py
 // and
 // https://chromium.googlesource.com/infra/infra/+/f2e2a4d/appengine/monorail/proto/api_pb2_v1.py
 
 syntax = "proto3";
 
 package monorail;
 
 // Monorail service can manipulate issues.
 service Monorail {
     // Creates an issue.
     rpc InsertIssue(InsertIssueRequest) returns (InsertIssueResponse){};
     // Posts a comment to an issue. Can update issue attributes, such as status.
     rpc InsertComment(InsertCommentRequest) returns (InsertCommentResponse){};
+    // Lists issues from a project.
+    rpc IssuesList(IssuesListRequest) returns (IssuesListResponse){};
 }
 
 // A monorail issue.
 message Issue {
     // Reporter of the issue.
     AtomPerson author = 1;
     // Issues that must be fixed before this one can be fixed.
     repeated IssueRef blockedOn = 2;
     // People participating in the issue discussion.
     repeated AtomPerson cc = 6;
@@ skipping 62 matching lines @@
         Update updates = 8;
     }
     // Definition of the comment.
     Comment comment = 1;
     // The reference to post the comment to.
     IssueRef issue = 2;
 }
 
 message InsertCommentResponse{}
 
+message IssuesListRequest {

[nodir, 2016/06/09 04:09:28]

Comments world be great, otherwise a reader may have hard time understanding
what these fields mean. Other fields and messages in this file are documented

[seanmccullough1, 2016/06/09 15:01:33]

Done.

+  string projectId = 1;
+  repeated string additionalProject = 2;
+  enum CannedQuery {
+    ALL = 0;
+    NEW = 1;
+    OPEN = 2;
+    OWNED = 3;
+    REPORTED = 4;
+    STARRED = 5;
+    TO_VERIFY = 6;
+  }
+  CannedQuery can= 3;

[martiniss, 2016/06/09 02:30:13]

space around =

[seanmccullough1, 2016/06/09 15:01:33]

Done.

+  string label = 4;
+  int32 maxResults = 5;
+  string owner = 6;
+  int64 publishedMax = 7;
+  int64 publishedMin = 8;
+  string q = 9;
+  string sort = 10;
+  int32 startIndex = 11;
+  string status = 12;
+  int64 updatedMax = 13;
+  int64 updatedMin = 14;
+}
+
+message IssuesListResponse {
+  // error = messages.MessageField(ErrorMessage, 1)

[nodir, 2016/06/09 04:09:28]

Looks like an important field. Why not included?

[seanmccullough1, 2016/06/09 15:01:33]

It was ignored by other Request protos defined here. Added it for this one.

+  repeated Issue items = 2;
+  int32 totalResults = 3;
+  string kind = 4;

[martiniss, 2016/06/09 02:30:13]

what is this exactly?

[seanmccullough1, 2016/06/09 15:01:32]

I could defer to the original doc:
https://chromium.googlesource.com/infra/infra/+/f2e2a4d/appengine/monorail/pr...
:)

I believe it is an endpoints thing, for this response I assume it will always be
"monorail#issueList"

+}
+
 // Defines a mutation to an issue.
 // This message is partial.
 // Derived from Update type in api_pb2_v1.py.
 message Update {
     // If set, the new status of the issue.
     string status = 2;
 }
 
 // Identifies a Monorail user.
 message AtomPerson {
     // User email.
     string name = 1;
 }