Add PR count to leaderboard activity

ls1intum · Sep 25, 2024 · 92c07fb · 92c07fb
1 parent 7b9c90c
commit 92c07fb
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 1 deletion.
diff --git a/server/application-server/openapi.yaml b/server/application-server/openapi.yaml
@@ -465,6 +465,9 @@ components:
         rank:
           type: integer
           format: int32
+        numberOfReviewedPRs:
+          type: integer
+          format: int32
         changesRequested:
           type: array
           items:

diff --git a/...lication-server/src/main/java/de/tum/in/www1/hephaestus/leaderboard/LeaderboardEntry.java b/...lication-server/src/main/java/de/tum/in/www1/hephaestus/leaderboard/LeaderboardEntry.java
@@ -21,6 +21,7 @@ public class LeaderboardEntry {
         private UserType type;
         private int score;
         private int rank;
+        private int numberOfReviewedPRs;
         private PullRequestReviewDTO[] changesRequested;
         private PullRequestReviewDTO[] approvals;
         private PullRequestReviewDTO[] comments;

diff --git a/...cation-server/src/main/java/de/tum/in/www1/hephaestus/leaderboard/LeaderboardService.java b/...cation-server/src/main/java/de/tum/in/www1/hephaestus/leaderboard/LeaderboardService.java
@@ -56,6 +56,7 @@ public List<LeaderboardEntry> createLeaderboard(Optional<LocalDate> after, Optio
                 return null;
             }
             AtomicInteger score = new AtomicInteger(0);
+            Set<Integer> reviewedPRs = new HashSet<>();
             Set<PullRequestReviewDTO> changesRequestedSet = new HashSet<>();
             Set<PullRequestReviewDTO> approvedSet = new HashSet<>();
             Set<PullRequestReviewDTO> commentSet = new HashSet<>();
@@ -69,6 +70,7 @@ public List<LeaderboardEntry> createLeaderboard(Optional<LocalDate> after, Optio
                         PullRequestReviewDTO reviewDTO = new PullRequestReviewDTO(review.getId(), review.getCreatedAt(),
                                 review.getUpdatedAt(), review.getSubmittedAt(), review.getState());
 
+                        reviewedPRs.add(review.getPullRequest().getNumber());
                         switch (review.getState()) {
                             case CHANGES_REQUESTED:
                                 changesRequestedSet.add(reviewDTO);
@@ -88,6 +90,7 @@ public List<LeaderboardEntry> createLeaderboard(Optional<LocalDate> after, Optio
             return new LeaderboardEntry(user.getLogin(), user.getAvatarUrl(), user.getName(), user.getType(),
                     score.get(),
                     0, // preliminary rank
+                    reviewedPRs.size(),
                     changesRequestedSet.toArray(new PullRequestReviewDTO[changesRequestedSet.size()]),
                     approvedSet.toArray(new PullRequestReviewDTO[approvedSet.size()]),
                     commentSet.toArray(new PullRequestReviewDTO[commentSet.size()]));

diff --git a/webapp/src/app/core/modules/openapi/model/leaderboard-entry.ts b/webapp/src/app/core/modules/openapi/model/leaderboard-entry.ts
@@ -19,6 +19,7 @@ export interface LeaderboardEntry {
     type?: LeaderboardEntry.TypeEnum;
     score?: number;
     rank?: number;
+    numberOfReviewedPRs?: number;
     changesRequested?: Array<PullRequestReviewDTO>;
     approvals?: Array<PullRequestReviewDTO>;
     comments?: Array<PullRequestReviewDTO>;

diff --git a/webapp/src/app/home/leaderboard/leaderboard.component.html b/webapp/src/app/home/leaderboard/leaderboard.component.html
@@ -25,6 +25,13 @@
         <td appTableCell class="text-center">{{ entry.score }}</td>
         <td appTableCell class="py-2">
           <div class="flex items-center gap-3">
+            <div class="flex items-center gap-1 text-github-foreground" title="Total reviewed PRs">
+              <ng-icon [svg]="octGitPullRequest" size="16" />
+              {{ entry.numberOfReviewedPRs }}
+            </div>
+            <div class="flex items-center text-github-neutral-foreground">
+              <ng-icon [svg]="octChevronRight" size="16" />
+            </div>
             @if (entry.changesRequested && entry.changesRequested.length > 0) {
               <div class="flex items-center gap-1 text-github-danger-foreground" title="Changes Requested">
                 <ng-icon [svg]="octFileDiff" size="16" />

diff --git a/webapp/src/app/home/leaderboard/leaderboard.component.ts b/webapp/src/app/home/leaderboard/leaderboard.component.ts
@@ -1,6 +1,6 @@
 import { Component, input } from '@angular/core';
 import { NgIconComponent } from '@ng-icons/core';
-import { octFileDiff, octCheck, octComment } from '@ng-icons/octicons';
+import { octFileDiff, octCheck, octComment, octGitPullRequest, octChevronRight } from '@ng-icons/octicons';
 import { LeaderboardEntry } from 'app/core/modules/openapi';
 import { AvatarFallbackComponent } from 'app/ui/avatar/avatar-fallback.component';
 import { AvatarImageComponent } from 'app/ui/avatar/avatar-image.component';
@@ -37,6 +37,8 @@ export class LeaderboardComponent {
   protected octFileDiff = octFileDiff;
   protected octCheck = octCheck;
   protected octComment = octComment;
+  protected octGitPullRequest = octGitPullRequest;
+  protected octChevronRight = octChevronRight;
 
   leaderboard = input<LeaderboardEntry[]>();
 }