Added handling for cases where the user in a PR review is missing (#998)

Author: avasconcelos114

server/plugin/api.go

@@ -859,7 +859,15 @@ func fetchReviews(c *UserContext, client *github.Client, repoOwner string, repoN
 		return []*github.PullRequestReview{}, errors.Wrap(err, "could not list reviews")
 	}
 
-	return reviewsList, nil
+	// Filtering reviews where user is nil
+	filtered := make([]*github.PullRequestReview, 0, len(reviewsList))
+	for _, review := range reviewsList {
+		if review != nil && review.User != nil {
+			filtered = append(filtered, review)
+		}
+	}
+
+	return filtered, nil
 }
 
 func getRepoOwnerAndNameFromURL(rawURL string) (string, string, error) {

webapp/src/components/sidebar_right/github_items.tsx

@@ -467,7 +467,11 @@ function getReviewText(item: GithubItem, style: any, secondLine: boolean) {
     };
 
     const lastReviews = item.reviews.reduce(reverse, []).filter((v) => {
-        if (v.user.login === item.user.login) {
+        if (!v.user) {
+            return false;
+        }
+
+        if (item.user && v.user.login === item.user.login) {
             return false;
         }
 

webapp/src/types/github_types.ts

@@ -17,7 +17,7 @@ type GitHubUser = {
 
 export type Review = {
     state: string;
-    user: GitHubUser;
+    user?: GitHubUser;
 }
 
 export type GithubItem = PrsDetailsData & {