feat:implement loading student summary using nestjs

client/src/api/api.ts

@@ -3800,6 +3800,91 @@ export interface MentorStudentDto {
      */
     'repoUrl': string | null;
 }
+/**
+ * 
+ * @export
+ * @interface MentorStudentSummaryDto
+ */
+export interface MentorStudentSummaryDto {
+    /**
+     * 
+     * @type {number}
+     * @memberof MentorStudentSummaryDto
+     */
+    'id': number;
+    /**
+     * 
+     * @type {string}
+     * @memberof MentorStudentSummaryDto
+     */
+    'githubId': string;
+    /**
+     * 
+     * @type {string}
+     * @memberof MentorStudentSummaryDto
+     */
+    'name': string;
+    /**
+     * 
+     * @type {boolean}
+     * @memberof MentorStudentSummaryDto
+     */
+    'isActive': boolean;
+    /**
+     * 
+     * @type {string}
+     * @memberof MentorStudentSummaryDto
+     */
+    'cityName': string;
+    /**
+     * 
+     * @type {string}
+     * @memberof MentorStudentSummaryDto
+     */
+    'countryName': string;
+    /**
+     * 
+     * @type {Array<string>}
+     * @memberof MentorStudentSummaryDto
+     */
+    'students': Array<string>;
+    /**
+     * 
+     * @type {string}
+     * @memberof MentorStudentSummaryDto
+     */
+    'contactsEmail': string | null;
+    /**
+     * 
+     * @type {string}
+     * @memberof MentorStudentSummaryDto
+     */
+    'contactsPhone': string | null;
+    /**
+     * 
+     * @type {string}
+     * @memberof MentorStudentSummaryDto
+     */
+    'contactsSkype': string | null;
+    /**
+     * 
+     * @type {string}
+     * @memberof MentorStudentSummaryDto
+     */
+    'contactsTelegram': string | null;
+    /**
+     * 
+     * @type {string}
+     * @memberof MentorStudentSummaryDto
+     */
+    'contactsNotes': string | null;
+    /**
+     * 
+     * @type {string}
+     * @memberof MentorStudentSummaryDto
+     */
+    'contactsWhatsApp': string | null;
+}
 /**
  * 
  * @export
@@ -5125,6 +5210,49 @@ export interface StudentId {
      */
     'id': number;
 }
+/**
+ * 
+ * @export
+ * @interface StudentSummaryDto
+ */
+export interface StudentSummaryDto {
+    /**
+     * 
+     * @type {number}
+     * @memberof StudentSummaryDto
+     */
+    'totalScore': number;
+    /**
+     * 
+     * @type {Array<string>}
+     * @memberof StudentSummaryDto
+     */
+    'results': Array<string>;
+    /**
+     * 
+     * @type {boolean}
+     * @memberof StudentSummaryDto
+     */
+    'isActive': boolean;
+    /**
+     * 
+     * @type {MentorStudentSummaryDto}
+     * @memberof StudentSummaryDto
+     */
+    'mentor': MentorStudentSummaryDto | null;
+    /**
+     * 
+     * @type {number}
+     * @memberof StudentSummaryDto
+     */
+    'rank': number;
+    /**
+     * 
+     * @type {string}
+     * @memberof StudentSummaryDto
+     */
+    'repository': string | null;
+}
 /**
  * 
  * @export
@@ -15375,6 +15503,43 @@ export const StudentsApiAxiosParamCreator = function (configuration?: Configurat
 
 
 
+            setSearchParams(localVarUrlObj, localVarQueryParameter);
+            let headersFromBaseOptions = baseOptions && baseOptions.headers ? baseOptions.headers : {};
+            localVarRequestOptions.headers = {...localVarHeaderParameter, ...headersFromBaseOptions, ...options.headers};
+
+            return {
+                url: toPathString(localVarUrlObj),
+                options: localVarRequestOptions,
+            };
+        },
+        /**
+         * 
+         * @param {number} courseId 
+         * @param {string} githubId 
+         * @param {*} [options] Override http request option.
+         * @throws {RequiredError}
+         */
+        getStudentSummary: async (courseId: number, githubId: string, options: AxiosRequestConfig = {}): Promise<RequestArgs> => {
+            // verify required parameter 'courseId' is not null or undefined
+            assertParamExists('getStudentSummary', 'courseId', courseId)
+            // verify required parameter 'githubId' is not null or undefined
+            assertParamExists('getStudentSummary', 'githubId', githubId)
+            const localVarPath = `/courses/{courseId}/students/{githubId}/summary`
+                .replace(`{${"courseId"}}`, encodeURIComponent(String(courseId)))
+                .replace(`{${"githubId"}}`, encodeURIComponent(String(githubId)));
+            // use dummy base URL string because the URL constructor only accepts absolute URLs.
+            const localVarUrlObj = new URL(localVarPath, DUMMY_BASE_URL);
+            let baseOptions;
+            if (configuration) {
+                baseOptions = configuration.baseOptions;
+            }
+
+            const localVarRequestOptions = { method: 'GET', ...baseOptions, ...options};
+            const localVarHeaderParameter = {} as any;
+            const localVarQueryParameter = {} as any;
+
+
+
             setSearchParams(localVarUrlObj, localVarQueryParameter);
             let headersFromBaseOptions = baseOptions && baseOptions.headers ? baseOptions.headers : {};
             localVarRequestOptions.headers = {...localVarHeaderParameter, ...headersFromBaseOptions, ...options.headers};
@@ -15404,6 +15569,17 @@ export const StudentsApiFp = function(configuration?: Configuration) {
             const localVarAxiosArgs = await localVarAxiosParamCreator.getStudent(studentId, options);
             return createRequestFunction(localVarAxiosArgs, globalAxios, BASE_PATH, configuration);
         },
+        /**
+         * 
+         * @param {number} courseId 
+         * @param {string} githubId 
+         * @param {*} [options] Override http request option.
+         * @throws {RequiredError}
+         */
+        async getStudentSummary(courseId: number, githubId: string, options?: AxiosRequestConfig): Promise<(axios?: AxiosInstance, basePath?: string) => AxiosPromise<StudentSummaryDto>> {
+            const localVarAxiosArgs = await localVarAxiosParamCreator.getStudentSummary(courseId, githubId, options);
+            return createRequestFunction(localVarAxiosArgs, globalAxios, BASE_PATH, configuration);
+        },
     }
 };
 
@@ -15423,6 +15599,16 @@ export const StudentsApiFactory = function (configuration?: Configuration, baseP
         getStudent(studentId: number, options?: any): AxiosPromise<StudentDto> {
             return localVarFp.getStudent(studentId, options).then((request) => request(axios, basePath));
         },
+        /**
+         * 
+         * @param {number} courseId 
+         * @param {string} githubId 
+         * @param {*} [options] Override http request option.
+         * @throws {RequiredError}
+         */
+        getStudentSummary(courseId: number, githubId: string, options?: any): AxiosPromise<StudentSummaryDto> {
+            return localVarFp.getStudentSummary(courseId, githubId, options).then((request) => request(axios, basePath));
+        },
     };
 };
 
@@ -15443,6 +15629,18 @@ export class StudentsApi extends BaseAPI {
     public getStudent(studentId: number, options?: AxiosRequestConfig) {
         return StudentsApiFp(this.configuration).getStudent(studentId, options).then((request) => request(this.axios, this.basePath));
     }
+
+    /**
+     * 
+     * @param {number} courseId 
+     * @param {string} githubId 
+     * @param {*} [options] Override http request option.
+     * @throws {RequiredError}
+     * @memberof StudentsApi
+     */
+    public getStudentSummary(courseId: number, githubId: string, options?: AxiosRequestConfig) {
+        return StudentsApiFp(this.configuration).getStudentSummary(courseId, githubId, options).then((request) => request(this.axios, this.basePath));
+    }
 }
 
 

client/src/services/course.ts

@@ -16,6 +16,7 @@ import {
   CriteriaDto,
   CrossCheckMessageDto,
   CrossCheckCriteriaDataDto,
+  StudentsApi,
 } from 'api';
 import { optionalQueryString } from 'utils/optionalQueryString';
 import { Decision } from 'data/interviews/technical-screening';
@@ -140,6 +141,7 @@ export type SearchStudent = UserBasic & { mentor: UserBasic | null };
 const courseTasksApi = new CoursesTasksApi();
 const courseEventsApi = new CoursesEventsApi();
 const studentsScoreApi = new StudentsScoreApi();
+const studentsApi = new StudentsApi();
 
 export class CourseService {
   private axios: AxiosInstance;
@@ -527,8 +529,8 @@ export class CourseService {
   }
 
   async getStudentSummary(githubId: string | 'me') {
-    const result = await this.axios.get(`/student/${githubId}/summary`);
-    return result.data.data as StudentSummary;
+    const result = await studentsApi.getStudentSummary(this.courseId, githubId);
+    return result.data as unknown as StudentSummary;
   }
 
   async getStudentScore(githubId: string) {

nestjs/src/courses/course-schedule/course-schedule.service.ts

@@ -304,7 +304,7 @@ export class CourseScheduleService {
         ...(technicalScreeningResults
           .find(task => task.courseTaskId === courseTaskId)
           ?.stageInterviewFeedbacks.map(feedback => JSON.parse(feedback.json))
-          .map((json: any) => json?.resume?.score ?? 0) ?? []),
+          .map((json: any) => (json?.resume?.score || json?.steps?.decision?.values?.finalScore) ?? 0) ?? []),
       );
     const currentScore = isFinite(scoreRaw) ? scoreRaw : null;
     return currentScore;

nestjs/src/courses/course-students/course-students.controller.ts

@@ -0,0 +1,48 @@
+import { Controller, Get, Param, Req, UseGuards } from '@nestjs/common';
+import { ApiBadRequestResponse, ApiForbiddenResponse, ApiOkResponse, ApiOperation, ApiTags } from '@nestjs/swagger';
+import { CurrentRequest, DefaultGuard } from '../../auth';
+import { StudentSummaryDto } from './dto/student-summary.dto';
+import { CourseStudentsService } from './course-students.service';
+
+@Controller('courses/:courseId/students')
+@ApiTags('students')
+@UseGuards(DefaultGuard)
+export class CourseStudentsController {
+  constructor(private courseStudentService: CourseStudentsService) {}
+
+  @Get(':githubId/summary')
+  @ApiForbiddenResponse()
+  @ApiBadRequestResponse()
+  @ApiOkResponse({
+    type: StudentSummaryDto,
+  })
+  @ApiOperation({ operationId: 'getStudentSummary' })
+  public async getStudentSummary(
+    @Param('courseId') courseId: number,
+    @Param('githubId') githubId: string,
+    @Req() req: CurrentRequest,
+  ) {
+    let studentGithubId;
+    if (githubId === 'me') {
+      studentGithubId = req.user.githubId;
+    } else {
+      studentGithubId = githubId;
+    }
+
+    const student = await this.courseStudentService.getStudentByGithubId(courseId, studentGithubId);
+
+    const [score, mentor] = await Promise.all([
+      this.courseStudentService.getStudentScore(student?.id || 0),
+      student?.mentorId ? await this.courseStudentService.getMentorWithContacts(student.mentorId) : null,
+    ]);
+
+    return new StudentSummaryDto({
+      totalScore: score?.totalScore,
+      results: score?.results,
+      rank: score?.rank,
+      isActive: !student?.isExpelled && !student?.isFailed,
+      mentor,
+      repository: student?.repository || null,
+    });
+  }
+}