minor fixes to types and allowing null values

Author: gavxue

backend/typescript/graphql/types/reviewDashboardType.ts

@@ -24,6 +24,7 @@ const reviewDashboardType = gql`
     desireToLearn: Int
     skill: Int
     skillCategory: String
+    comments: String
   }
 
   type ReviewDetails {
@@ -39,7 +40,7 @@ const reviewDashboardType = gql`
     program: String!
     resumeUrl: String!
     applicationStatus: String!
-    skillCategory: String!
+    skillCategory: String
     reviewDetails: [ReviewDetails!]!
   }
 

backend/typescript/services/implementations/reviewDashboardService.ts

@@ -41,7 +41,7 @@ function toSidePanelDTO(model: ApplicantRecord): ReviewDashboardSidePanelDTO {
     program: model.applicant!.program,
     resumeUrl: model.applicant!.resumeUrl,
     applicationStatus: model.status,
-    skillCategory: model.skillCategory || "Junior",   // seems skill category can be null in the applicant record db right now
+    skillCategory: model.skillCategory,
     reviewDetails: reviewDetails,
   };
 }

backend/typescript/types.ts

@@ -198,6 +198,7 @@ export type Review = {
   desireToLearn?: number;
   skill?: number;
   skillCategory?: SkillCategory;
+  comments?: string;
 };
 
 export type ReviewedApplicantRecordDTO = {
@@ -222,7 +223,7 @@ export type ReviewDashboardSidePanelDTO = {
   program: string;
   resumeUrl: string;
   applicationStatus: ApplicationStatus;
-  skillCategory: SkillCategory;
+  skillCategory: SkillCategory | null;
   reviewDetails: ReviewDetails[];
 };