Only fetch IssueComment and PullRequestCommit items

jcansdale · jcansdale · commit d66aac352d88 · 2020-03-18T17:41:46.000Z
diff --git a/src/GitHub.InlineReviews/Services/PullRequestSessionService.cs b/src/GitHub.InlineReviews/Services/PullRequestSessionService.cs
@@ -306,6 +306,8 @@ public virtual Task<PullRequestDetailModel> ReadPullRequestDetail(HostAddress ad
         async Task<PullRequestDetailModel> ReadPullRequestDetailWithResolved(HostAddress address, string owner,
             string name, int number, bool refresh)
         {
+            var itemTypes = new[] { PullRequestTimelineItemsItemType.IssueComment, PullRequestTimelineItemsItemType.PullRequestCommit };
+
             if (readPullRequestWithResolved == null)
             {
                 readPullRequestWithResolved = new Query()
@@ -383,7 +385,7 @@ async Task<PullRequestDetailModel> ReadPullRequestDetailWithResolved(HostAddress
                                 AvatarUrl = review.Author.AvatarUrl(null)
                             }
                         }).ToList(),
-                        Timeline = pr.TimelineItems(null, null, null, null, null, null, null).AllPages().Select(item => item.Switch<object>(when =>
+                        Timeline = pr.TimelineItems(null, null, null, null, itemTypes, null, null).AllPages().Select(item => item.Switch<object>(when =>
                             when.PullRequestCommit(commit => new CommitModel
                             {
                                 AbbreviatedOid = commit.Commit.AbbreviatedOid,
@@ -490,6 +492,8 @@ async Task<PullRequestDetailModel> ReadPullRequestDetailWithResolved(HostAddress
         async Task<PullRequestDetailModel> ReadPullRequestDetailWithoutResolved(HostAddress address, string owner,
             string name, int number, bool refresh)
         {
+            var itemTypes = new[] { PullRequestTimelineItemsItemType.IssueComment, PullRequestTimelineItemsItemType.PullRequestCommit };
+
             if (readPullRequestWithoutResolved == null)
             {
                 readPullRequestWithoutResolved = new Query()
@@ -562,7 +566,7 @@ async Task<PullRequestDetailModel> ReadPullRequestDetailWithoutResolved(HostAddr
                                 Url = comment.Url,
                             }).ToList(),
                         }).ToList(),
-                        Timeline = pr.TimelineItems(null, null, null, null, null, null, null).AllPages().Select(item => item.Switch<object>(when =>
+                        Timeline = pr.TimelineItems(null, null, null, null, itemTypes, null, null).AllPages().Select(item => item.Switch<object>(when =>
                             when.PullRequestCommit(commit => new CommitModel
                             {
                                 AbbreviatedOid = commit.Commit.AbbreviatedOid,

Original file line number	Diff line number	Diff line change
`@@ -306,6 +306,8 @@ public virtual Task<PullRequestDetailModel> ReadPullRequestDetail(HostAddress ad`
`306`	`306`	`async Task<PullRequestDetailModel> ReadPullRequestDetailWithResolved(HostAddress address, string owner,`
`307`	`307`	`string name, int number, bool refresh)`
`308`	`308`	`{`
	`309`	`+ var itemTypes = new[] { PullRequestTimelineItemsItemType.IssueComment, PullRequestTimelineItemsItemType.PullRequestCommit };`
	`310`	`+`
`309`	`311`	`if (readPullRequestWithResolved == null)`
`310`	`312`	`{`
`311`	`313`	`readPullRequestWithResolved = new Query()`
`@@ -383,7 +385,7 @@ async Task<PullRequestDetailModel> ReadPullRequestDetailWithResolved(HostAddress`
`383`	`385`	`AvatarUrl = review.Author.AvatarUrl(null)`
`384`	`386`	`}`
`385`	`387`	`}).ToList(),`
`386`		`- Timeline = pr.TimelineItems(null, null, null, null, null, null, null).AllPages().Select(item => item.Switch<object>(when =>`
	`388`	`+ Timeline = pr.TimelineItems(null, null, null, null, itemTypes, null, null).AllPages().Select(item => item.Switch<object>(when =>`
`387`	`389`	`when.PullRequestCommit(commit => new CommitModel`
`388`	`390`	`{`
`389`	`391`	`AbbreviatedOid = commit.Commit.AbbreviatedOid,`
`@@ -490,6 +492,8 @@ async Task<PullRequestDetailModel> ReadPullRequestDetailWithResolved(HostAddress`
`490`	`492`	`async Task<PullRequestDetailModel> ReadPullRequestDetailWithoutResolved(HostAddress address, string owner,`
`491`	`493`	`string name, int number, bool refresh)`
`492`	`494`	`{`
	`495`	`+ var itemTypes = new[] { PullRequestTimelineItemsItemType.IssueComment, PullRequestTimelineItemsItemType.PullRequestCommit };`
	`496`	`+`
`493`	`497`	`if (readPullRequestWithoutResolved == null)`
`494`	`498`	`{`
`495`	`499`	`readPullRequestWithoutResolved = new Query()`
`@@ -562,7 +566,7 @@ async Task<PullRequestDetailModel> ReadPullRequestDetailWithoutResolved(HostAddr`
`562`	`566`	`Url = comment.Url,`
`563`	`567`	`}).ToList(),`
`564`	`568`	`}).ToList(),`
`565`		`- Timeline = pr.TimelineItems(null, null, null, null, null, null, null).AllPages().Select(item => item.Switch<object>(when =>`
	`569`	`+ Timeline = pr.TimelineItems(null, null, null, null, itemTypes, null, null).AllPages().Select(item => item.Switch<object>(when =>`
`566`	`570`	`when.PullRequestCommit(commit => new CommitModel`
`567`	`571`	`{`
`568`	`572`	`AbbreviatedOid = commit.Commit.AbbreviatedOid,`