Fix userid for certs

cmu-sei · Jan 22, 2025 · 150cb8c · 150cb8c
1 parent 8fcdb0e
commit 150cb8c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/...ents/game-center/game-center-team-context-menu/game-center-team-context-menu.component.ts b/...ents/game-center/game-center-team-context-menu/game-center-team-context-menu.component.ts
@@ -66,7 +66,7 @@ export class GameCenterTeamContextMenuComponent {
       this.observeTeamUrl = this.routerService.getObserveTeamsUrl(this.game.id, this.team.id);
     }
 
-    this.certificateUrl = this.routerService.getCertificatePrintableUrl(PlayerMode.competition, this.game.id);
+    this.certificateUrl = this.routerService.getCertificatePrintableUrl(PlayerMode.competition, this.game.id, this.team.captain.userId);
   }
 
   protected confirmReset(request: TeamSessionResetRequest) {

diff --git a/projects/gameboard-ui/src/app/services/router.service.ts b/projects/gameboard-ui/src/app/services/router.service.ts
@@ -48,14 +48,14 @@ export class RouterService implements OnDestroy {
     return this.buildAppUrlWithQueryParams({ queryParams: { search: teamId } }, "admin", "game", gameId, "teams").toString();
   }
 
-  public getCertificatePrintableUrl(mode: PlayerMode, challengeSpecOrGameId: string) {
-    const localUserId = this.localUser.user$.value?.id;
+  public getCertificatePrintableUrl(mode: PlayerMode, challengeSpecOrGameId: string, ownerUserId?: string) {
+    const userId = ownerUserId || this.localUser.user$.value?.id;
 
-    if (!localUserId) {
+    if (!userId) {
       throw new Error("Can't navigate to printable certificate if not authenticated.");
     }
 
-    return `user/${localUserId}/certificates/${mode}/${challengeSpecOrGameId}`;
+    return `user/${userId}/certificates/${mode}/${challengeSpecOrGameId}`;
   }
 
   public getCertificateListUrl() {