adds new fields to graphql query

uwblueprint · Dec 6, 2023 · fbe8bce · fbe8bce
1 parent 724056e
commit fbe8bce
Show file tree

Hide file tree

Showing 5 changed files with 47 additions and 22 deletions.
diff --git a/backend/typescript/graphql/resolvers/dashboardResolvers.ts b/backend/typescript/graphql/resolvers/dashboardResolvers.ts
@@ -39,9 +39,8 @@ const dashboardResolvers = {
       _parent: undefined,
       { secondChoice }: { secondChoice: ApplicantRole },
     ): Promise<Array<ApplicationDTO>> => {
-      const applications = await dashboardService.getApplicationsBySecondChoiceRole(
-        secondChoice,
-      );
+      const applications =
+        await dashboardService.getApplicationsBySecondChoiceRole(secondChoice);
       return applications;
     },
     dashboardsByApplicationId: async (
@@ -79,8 +78,10 @@ const dashboardResolvers = {
         teamPlayer,
         desireToLearn,
         skill,
+        totalScore,
         skillCategory,
         reviwerComments,
+        adminComments,
         recommendedSecondChoice,
       }: {
         reviewerEmail: string;
@@ -90,8 +91,10 @@ const dashboardResolvers = {
         teamPlayer: number;
         desireToLearn: number;
         skill: number;
+        totalScore: number;
         skillCategory: string;
         reviwerComments: string;
+        adminComments: string;
         recommendedSecondChoice: string;
       },
     ): Promise<ApplicationDashboardDTO> => {
@@ -103,8 +106,10 @@ const dashboardResolvers = {
         teamPlayer,
         desireToLearn,
         skill,
+        totalScore,
         skillCategory,
         reviwerComments,
+        adminComments,
         recommendedSecondChoice,
       );
     },

diff --git a/backend/typescript/graphql/types/dashboardType.ts b/backend/typescript/graphql/types/dashboardType.ts
@@ -90,8 +90,10 @@ const dashboardType = gql`
       teamPlayer: Int!
       desireToLearn: Int!
       skill: Int!
+      totalScore: Int!
       skillCategory: String!
       reviwerComments: String!
+      adminComments: String!
       recommendedSecondChoice: String!
     ): ApplicationDashboardDTO!
   }

diff --git a/backend/typescript/services/implementations/appDashboardService.ts b/backend/typescript/services/implementations/appDashboardService.ts
@@ -47,8 +47,10 @@ class AppDashboardService implements IAppDashboardService {
       teamPlayer: dashboard.teamPlayer,
       desireToLearn: dashboard.desireToLearn,
       skill: dashboard.skill,
+      totalScore: dashboard.totalScore,
       skillCategory: dashboard.skillCategory,
       reviewerComments: dashboard.reviewerComments,
+      adminComments: dashboard.adminComments,
       recommendedSecondChoice: dashboard.recommendedSecondChoice,
       reviewerId: dashboard.reviewerId,
       applicationId: dashboard.applicationId,
@@ -166,8 +168,8 @@ class AppDashboardService implements IAppDashboardService {
           );
         },
       );
-      applicationsBySecondChoiceRoleDTO = await applicationsBySecondChoiceRole.map(
-        (application) => {
+      applicationsBySecondChoiceRoleDTO =
+        await applicationsBySecondChoiceRole.map((application) => {
           return {
             id: application.id,
             academicOrCoop: application.academicOrCoop,
@@ -191,8 +193,7 @@ class AppDashboardService implements IAppDashboardService {
             timesApplied: application.timesApplied,
             timestamp: application.timestamp,
           };
-        },
-      );
+        });
     } catch (error: unknown) {
       Logger.error(
         `Failed to get applications by this second choice role = ${role}. Reason = ${getErrorMessage(
@@ -221,8 +222,10 @@ class AppDashboardService implements IAppDashboardService {
           teamPlayer: dashboard.teamPlayer,
           desireToLearn: dashboard.desireToLearn,
           skill: dashboard.skill,
+          totalScore: dashboard.totalScore,
           skillCategory: dashboard.skillCategory,
           reviewerComments: dashboard.reviewerComments,
+          adminComments: dashboard.adminComments,
           recommendedSecondChoice: dashboard.recommendedSecondChoice,
           reviewerId: dashboard.reviewerId,
           applicationId: dashboard.applicationId,
@@ -243,15 +246,13 @@ class AppDashboardService implements IAppDashboardService {
     role: ApplicantRole,
   ): Promise<ApplicationDashboardRowDTO[]> {
     // get all the applications for the role
-    const applications: Array<ApplicationDTO> = await this.getApplicationsByRole(
-      role,
-    );
+    const applications: Array<ApplicationDTO> =
+      await this.getApplicationsByRole(role);
     // get the dashboards associated with the applications
     const appDashRows: Array<ApplicationDashboardRowDTO> = await Promise.all(
       applications.map(async (application) => {
-        const reviewDashboards: Array<ApplicationDashboardDTO> = await this.getDashboardsByApplicationId(
-          application.id,
-        );
+        const reviewDashboards: Array<ApplicationDashboardDTO> =
+          await this.getDashboardsByApplicationId(application.id);
         const reviewers: Array<UserDTO> = await Promise.all(
           reviewDashboards.map(async (dash) => {
             return userService.getUserByEmail(dash.reviewerEmail);
@@ -271,15 +272,13 @@ class AppDashboardService implements IAppDashboardService {
     role: ApplicantRole,
   ): Promise<ApplicationDashboardRowDTO[]> {
     // get all the applications for the role
-    const applications: Array<ApplicationDTO> = await this.getApplicationsBySecondChoiceRole(
-      role,
-    );
+    const applications: Array<ApplicationDTO> =
+      await this.getApplicationsBySecondChoiceRole(role);
     // get the dashboards associated with the applications
     const appDashRows: Array<ApplicationDashboardRowDTO> = await Promise.all(
       applications.map(async (application) => {
-        const reviewDashboards: Array<ApplicationDashboardDTO> = await this.getDashboardsByApplicationId(
-          application.id,
-        );
+        const reviewDashboards: Array<ApplicationDashboardDTO> =
+          await this.getDashboardsByApplicationId(application.id);
         const reviewers: Array<UserDTO> = await Promise.all(
           reviewDashboards.map(async (dash) => {
             return userService.getUserByEmail(dash.reviewerEmail);
@@ -331,8 +330,10 @@ class AppDashboardService implements IAppDashboardService {
       teamPlayer: dashboard.teamPlayer,
       desireToLearn: dashboard.desireToLearn,
       skill: dashboard.skill,
+      totalScore: dashboard.totalScore,
       skillCategory: dashboard.skillCategory,
       reviewerComments: dashboard.reviewerComments,
+      adminComments: dashboard.adminComments,
       recommendedSecondChoice: dashboard.recommendedSecondChoice,
       reviewerId: dashboard.reviewerId,
       applicationId: dashboard.applicationId,
@@ -347,8 +348,10 @@ class AppDashboardService implements IAppDashboardService {
     teamPlayer: number,
     desireToLearn: number,
     skill: number,
+    totalScore: number,
     skillCategory: string,
     reviewerComments: string,
+    adminComments: string,
     recommendedSecondChoice: string,
   ): Promise<ApplicationDashboardDTO> {
     try {
@@ -363,8 +366,10 @@ class AppDashboardService implements IAppDashboardService {
         teamPlayer,
         desireToLearn,
         skill,
+        totalScore,
         skillCategory,
         reviewerComments,
+        adminComments,
         recommendedSecondChoice,
       });
       Logger.error(`the data: ${JSON.stringify(dashboard)}`);
@@ -375,8 +380,10 @@ class AppDashboardService implements IAppDashboardService {
         teamPlayer: dashboard.teamPlayer,
         desireToLearn: dashboard.desireToLearn,
         skill: dashboard.skill,
+        totalScore: dashboard.totalScore,
         skillCategory: dashboard.skillCategory,
         reviewerComments: dashboard.reviewerComments,
+        adminComments: dashboard.adminComments,
         recommendedSecondChoice: dashboard.recommendedSecondChoice,
         reviewerId: dashboard.reviewerId,
         applicationId: dashboard.applicationId,
@@ -413,8 +420,10 @@ class AppDashboardService implements IAppDashboardService {
       teamPlayer: dashboard.teamPlayer,
       desireToLearn: dashboard.desireToLearn,
       skill: dashboard.skill,
+      totalScore: dashboard.totalScore,
       skillCategory: dashboard.skillCategory,
       reviewerComments: dashboard.reviewerComments,
+      adminComments: dashboard.adminComments,
       recommendedSecondChoice: dashboard.recommendedSecondChoice,
       reviewerId: dashboard.reviewerId,
       applicationId: dashboard.applicationId,
@@ -447,8 +456,10 @@ class AppDashboardService implements IAppDashboardService {
       teamPlayer: dashboard.teamPlayer,
       desireToLearn: dashboard.desireToLearn,
       skill: dashboard.skill,
+      totalScore: dashboard.totalScore,
       skillCategory: dashboard.skillCategory,
       reviewerComments: dashboard.reviewerComments,
+      adminComments: dashboard.adminComments,
       recommendedSecondChoice: dashboard.recommendedSecondChoice,
       reviewerId: dashboard.reviewerId,
       applicationId: dashboard.applicationId,

diff --git a/backend/typescript/services/interfaces/appDashboardService.ts b/backend/typescript/services/interfaces/appDashboardService.ts
@@ -36,11 +36,14 @@ interface IAppDashboardService {
    * @Param reviewerEmail the email of the reviewer
    * @param applicationId the id of the application (seperate from the postgres id field of the application)
    * @param reviewerAuthId the Firebase auth id of the user (This is NOT the same as the postgress id field of the user)
-   * @param passionFSG passion for social good rating of applicatn
-   * @param teamPlayer teamwork rating of applicatn
-   * @param skill skill rating of applicatn
+   * @param passionFSG passion for social good rating of application
+   * @param teamPlayer teamwork rating of application
+   * @param desireToLearn willingness to learn rating of application
+   * @param skill skill rating of application
+   * @param totalScore sum of passionFSG, teamPlayer, desireToLearn, and skill
    * @param skillCategory whether applicant is viewed as junior, intermediate, or senior
    * @param reviewerComments comments of the application from reviewer
+   * @param adminComments comments of the application from admin
    * @param recommendedSecondChoice an indication of whether 2nd choice is recommended
    * @param reviewComplete whether the reviewer has finished the review
    * @returns an array of the updated dashboard entry ids
@@ -54,8 +57,10 @@ interface IAppDashboardService {
     teamPlayer: number,
     desireToLearn: number,
     skill: number,
+    totalScore: number,
     skillCategory: string,
     reviewerComments: string,
+    adminComments: string,
     recommendedSecondChoice: string,
   ): Promise<ApplicationDashboardDTO>;
 

diff --git a/backend/typescript/types.ts b/backend/typescript/types.ts
@@ -39,8 +39,10 @@ export type ApplicationDashboardDTO = {
   teamPlayer: number;
   desireToLearn: number;
   skill: number;
+  totalScore: number;
   skillCategory: string;
   reviewerComments: string;
+  adminComments: string;
   recommendedSecondChoice: string;
   reviewerId: number;
   applicationId: number;