From 49dca2dc33fddf7baaf926d5c6e0b41516aa6b6e Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Thu, 4 May 2023 19:51:38 -0400
Subject: [PATCH 01/12] initial commit

---
 .../apps/api/src/app.institutions.module.ts   |  12 ++
 .../assessment/assessment.aest.controller.ts  |  42 +-----
 .../assessment.controller.service.ts          |  57 ++++++++
 .../assessment.institutions.controller.ts     | 119 +++++++++++++++
 .../apps/api/src/route-controllers/index.ts   |   1 +
 .../application-exception.service.ts          |   7 +
 .../education-program-offering.service.ts     |  14 +-
 .../aest/students/assessment/Request.vue      |  13 +-
 .../src/constants/routes/RouteConstants.ts    |   1 +
 sources/packages/web/src/router/AESTRoutes.ts |   2 +-
 .../web/src/router/InstitutionRoutes.ts       |  20 ++-
 .../src/services/StudentAssessmentsService.ts |   9 +-
 .../src/services/http/StudentAssessmentApi.ts |  10 +-
 sources/packages/web/src/types/AppRoutes.ts   |  15 +-
 .../web/src/views/aest/AssessmentsSummary.vue |   4 +-
 .../InstitutionStudentApplications.vue        |  19 +++
 .../InstitutionAssessmentsSummary.vue         | 135 ++++++++++++++++++
 17 files changed, 418 insertions(+), 62 deletions(-)
 create mode 100644 sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
 create mode 100644 sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue

diff --git a/sources/packages/backend/apps/api/src/app.institutions.module.ts b/sources/packages/backend/apps/api/src/app.institutions.module.ts
index c85a45927d..26a4809f68 100644
--- a/sources/packages/backend/apps/api/src/app.institutions.module.ts
+++ b/sources/packages/backend/apps/api/src/app.institutions.module.ts
@@ -23,6 +23,10 @@ import {
   StudentService,
   EducationProgramOfferingImportCSVService,
   EducationProgramOfferingValidationService,
+  DisbursementReceiptService,
+  StudentAppealService,
+  ApplicationExceptionService,
+  StudentAppealRequestsService,
 } from "./services";
 import {
   DesignationAgreementInstitutionsController,
@@ -44,6 +48,8 @@ import {
   ConfirmationOfEnrollmentControllerService,
   StudentInstitutionsController,
   StudentControllerService,
+  AssessmentInstitutionsController,
+  AssessmentControllerService,
 } from "./route-controllers";
 import { AuthModule } from "./auth/auth.module";
 import {
@@ -77,6 +83,7 @@ import { UserControllerService } from "./route-controllers/user/user.controller.
     EducationProgramOfferingInstitutionsController,
     UserInstitutionsController,
     StudentInstitutionsController,
+    AssessmentInstitutionsController,
   ],
   providers: [
     WorkflowClientService,
@@ -123,6 +130,11 @@ import { UserControllerService } from "./route-controllers/user/user.controller.
     NoteSharedService,
     StudentControllerService,
     RestrictionSharedService,
+    AssessmentControllerService,
+    DisbursementReceiptService,
+    StudentAppealService,
+    ApplicationExceptionService,
+    StudentAppealRequestsService,
   ],
 })
 export class AppInstitutionsModule {}
diff --git a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.aest.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.aest.controller.ts
index 9b1f5247dc..e85abaf01c 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.aest.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.aest.controller.ts
@@ -39,50 +39,16 @@ export class AssessmentAESTController extends BaseController {
   /**
    * Get all pending and declined requests related to an application which would result
    * a new assessment when that request is approved.
-   * @param applicationId application number.
+   * @param applicationId application id.
    * @returns assessment requests or exceptions for a student application.
    */
   @Get("application/:applicationId/requests")
   async getRequestedAssessmentSummary(
     @Param("applicationId", ParseIntPipe) applicationId: number,
   ): Promise<RequestAssessmentSummaryAPIOutDTO[]> {
-    const requestAssessmentSummary: RequestAssessmentSummaryAPIOutDTO[] = [];
-    const offeringChange =
-      await this.educationProgramOfferingService.getOfferingRequestsByApplicationId(
-        applicationId,
-      );
-    if (offeringChange) {
-      requestAssessmentSummary.push({
-        id: offeringChange.id,
-        submittedDate: offeringChange.submittedDate,
-        status: offeringChange.offeringStatus,
-        requestType: RequestAssessmentTypeAPIOutDTO.OfferingRequest,
-        programId: offeringChange.educationProgram.id,
-      });
-    }
-    const applicationExceptions =
-      await this.applicationExceptionService.getExceptionsByApplicationId(
-        applicationId,
-        ApplicationExceptionStatus.Pending,
-        ApplicationExceptionStatus.Declined,
-      );
-    // When a pending or denied application exception exist then no other request can exist
-    // for the given application.
-    if (applicationExceptions.length > 0) {
-      const applicationExceptionArray: RequestAssessmentSummaryAPIOutDTO[] =
-        applicationExceptions.map((applicationException) => ({
-          id: applicationException.id,
-          submittedDate: applicationException.createdAt,
-          status: applicationException.exceptionStatus,
-          requestType: RequestAssessmentTypeAPIOutDTO.StudentException,
-        }));
-      return requestAssessmentSummary.concat(applicationExceptionArray);
-    }
-    const appeals =
-      await this.assessmentControllerService.getPendingAndDeniedAppeals(
-        applicationId,
-      );
-    return requestAssessmentSummary.concat(appeals);
+    return this.assessmentControllerService.requestedStudentAssessmentSummary(
+      applicationId,
+    );
   }
 
   /**
diff --git a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.controller.service.ts b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.controller.service.ts
index aa7f6163ca..fbc7d3b0fd 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.controller.service.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.controller.service.ts
@@ -3,6 +3,7 @@ import {
   COEStatus,
   DisbursementReceipt,
   AssessmentTriggerType,
+  ApplicationExceptionStatus,
 } from "@sims/sims-db";
 import {
   Injectable,
@@ -14,6 +15,8 @@ import {
   DisbursementReceiptService,
   StudentAppealService,
   StudentScholasticStandingsService,
+  EducationProgramOfferingService,
+  ApplicationExceptionService,
 } from "../../services";
 import {
   AssessmentNOAAPIOutDTO,
@@ -33,6 +36,8 @@ export class AssessmentControllerService {
     private readonly disbursementReceiptService: DisbursementReceiptService,
     private readonly studentAppealService: StudentAppealService,
     private readonly studentScholasticStandingsService: StudentScholasticStandingsService,
+    private readonly educationProgramOfferingService: EducationProgramOfferingService,
+    private readonly applicationExceptionService: ApplicationExceptionService,
   ) {}
 
   /**
@@ -282,4 +287,56 @@ export class AssessmentControllerService {
 
     return history;
   }
+
+  /**
+   * Get all pending and declined requests related to an application which would result
+   * a new assessment when that request is approved.
+   * @param applicationId application id.
+   * @param studentId student id.
+   * @returns assessment requests or exceptions for the student application.
+   */
+  async requestedStudentAssessmentSummary(
+    applicationId: number,
+    studentId?: number,
+  ): Promise<RequestAssessmentSummaryAPIOutDTO[]> {
+    const requestAssessmentSummary: RequestAssessmentSummaryAPIOutDTO[] = [];
+    const offeringChange =
+      await this.educationProgramOfferingService.getOfferingRequestsByApplicationId(
+        applicationId,
+        studentId,
+      );
+    if (offeringChange) {
+      requestAssessmentSummary.push({
+        id: offeringChange.id,
+        submittedDate: offeringChange.submittedDate,
+        status: offeringChange.offeringStatus,
+        requestType: RequestAssessmentTypeAPIOutDTO.OfferingRequest,
+        programId: offeringChange.educationProgram.id,
+      });
+    }
+    const applicationExceptions =
+      await this.applicationExceptionService.getExceptionsByApplicationId(
+        applicationId,
+        studentId,
+        ApplicationExceptionStatus.Pending,
+        ApplicationExceptionStatus.Declined,
+      );
+    // When a pending or denied application exception exist then no other request can exist
+    // for the given application.
+    if (applicationExceptions.length > 0) {
+      const applicationExceptionArray: RequestAssessmentSummaryAPIOutDTO[] =
+        applicationExceptions.map((applicationException) => ({
+          id: applicationException.id,
+          submittedDate: applicationException.createdAt,
+          status: applicationException.exceptionStatus,
+          requestType: RequestAssessmentTypeAPIOutDTO.StudentException,
+        }));
+      return requestAssessmentSummary.concat(applicationExceptionArray);
+    }
+    const appeals = await this.getPendingAndDeniedAppeals(
+      applicationId,
+      studentId,
+    );
+    return requestAssessmentSummary.concat(appeals);
+  }
 }
diff --git a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
new file mode 100644
index 0000000000..dc31387798
--- /dev/null
+++ b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
@@ -0,0 +1,119 @@
+import { Controller, Get, Param, ParseIntPipe } from "@nestjs/common";
+import BaseController from "../BaseController";
+import {
+  AllowAuthorizedParty,
+  Groups,
+  HasStudentDataAccess,
+  IsBCPublicInstitution,
+} from "../../auth/decorators";
+import { AuthorizedParties } from "../../auth/authorized-parties.enum";
+import { ClientTypeBaseRoute } from "../../types";
+import { UserGroups } from "../../auth/user-groups.enum";
+import {
+  AssessmentHistorySummaryAPIOutDTO,
+  AssessmentNOAAPIOutDTO,
+  RequestAssessmentSummaryAPIOutDTO,
+  AwardDetailsAPIOutDTO,
+  RequestAssessmentTypeAPIOutDTO,
+} from "./models/assessment.dto";
+import {
+  ApiNotFoundResponse,
+  ApiTags,
+  ApiUnprocessableEntityResponse,
+} from "@nestjs/swagger";
+import { AssessmentControllerService } from "./assessment.controller.service";
+import {
+  EducationProgramOfferingService,
+  ApplicationExceptionService,
+} from "../../services";
+import { ApplicationExceptionStatus } from "@sims/sims-db";
+
+/**
+ * Assessment controller for institutions.
+ */
+@AllowAuthorizedParty(AuthorizedParties.institution)
+@Controller("assessment")
+@ApiTags(`${ClientTypeBaseRoute.Institution}-assessment`)
+export class AssessmentInstitutionsController extends BaseController {
+  constructor(
+    private readonly assessmentControllerService: AssessmentControllerService,
+  ) //   private readonly educationProgramOfferingService: EducationProgramOfferingService,
+  //   private readonly applicationExceptionService: ApplicationExceptionService,
+  {
+    super();
+  }
+
+  /**
+   * Get all pending and declined requests related to an application which would result
+   * a new assessment when that request is approved.
+   * @param studentId student id.
+   * @param applicationId application id.
+   * @returns assessment requests or exceptions for the student application.
+   */
+  // @IsBCPublicInstitution()
+  // @HasStudentDataAccess("studentId")
+  @Get("/student/:studentId/application/:applicationId/requests")
+  async getRequestedAssessmentSummary(
+    @Param("studentId", ParseIntPipe) studentId: number,
+    @Param("applicationId", ParseIntPipe) applicationId: number,
+  ): Promise<RequestAssessmentSummaryAPIOutDTO[]> {
+    return this.assessmentControllerService.requestedStudentAssessmentSummary(
+      applicationId,
+      studentId,
+    );
+  }
+
+  // /**
+  //  * Method to get history of assessments for an application,
+  //  * i.e, this will have original assessment for the
+  //  * student application, and all approved student
+  //  * appeal and scholastic standings for the application
+  //  * which will have different assessment status.
+  //  * @param applicationId, application number.
+  //  * @returns summary of the assessment history for a student application.
+  //  */
+  // @Get("application/:applicationId/history")
+  // async getAssessmentHistorySummary(
+  //   @Param("applicationId", ParseIntPipe) applicationId: number,
+  // ): Promise<AssessmentHistorySummaryAPIOutDTO[]> {
+  //   return this.assessmentControllerService.getAssessmentHistorySummary(
+  //     applicationId,
+  //   );
+  // }
+
+  // /**
+  //  * Get the NOA values for a student application on a particular assessment.
+  //  * @param assessmentId assessment id to get the NOA values.
+  //  * @returns NOA and application data.
+  //  */
+  // @Get(":assessmentId/noa")
+  // @ApiNotFoundResponse({
+  //   description: "Assessment id not found.",
+  // })
+  // @ApiUnprocessableEntityResponse({
+  //   description: "Notice of assessment data is not present.",
+  // })
+  // async getAssessmentNOA(
+  //   @Param("assessmentId", ParseIntPipe) assessmentId: number,
+  // ): Promise<AssessmentNOAAPIOutDTO> {
+  //   return this.assessmentControllerService.getAssessmentNOA(assessmentId);
+  // }
+
+  // /**
+  //  * Get estimated and actual(if present) award details of an assessment.
+  //  * @param assessmentId assessment to which awards details belong to.
+  //  * @returns estimated and actual award details.
+  //  */
+  // @Get(":assessmentId/award")
+  // @ApiNotFoundResponse({
+  //   description: "Assessment not found.",
+  // })
+  // async getAssessmentAwardDetails(
+  //   @Param("assessmentId", ParseIntPipe) assessmentId: number,
+  // ): Promise<AwardDetailsAPIOutDTO> {
+  //   return this.assessmentControllerService.getAssessmentAwardDetails(
+  //     assessmentId,
+  //     true,
+  //   );
+  // }
+}
diff --git a/sources/packages/backend/apps/api/src/route-controllers/index.ts b/sources/packages/backend/apps/api/src/route-controllers/index.ts
index 0284c81e90..de298042d9 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/index.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/index.ts
@@ -57,3 +57,4 @@ export * from "./overaward/overaward.controller.service";
 export * from "./overaward/overaward.aest.controller";
 export * from "./overaward/overaward.students.controller";
 export * from "./student/student.institutions.controller";
+export * from "./assessment/assessment.institutions.controller";
diff --git a/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts b/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts
index 9b0adb3801..d1801192aa 100644
--- a/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts
+++ b/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts
@@ -66,10 +66,12 @@ export class ApplicationExceptionService extends RecordDataModelService<Applicat
    * submitted, for instance, when there are documents to be reviewed.
    * @param applicationId application associated with the exception.
    * @param status statuses to be filtered.
+   * @param studentId student id.
    * @returns student application exception information.
    */
   async getExceptionsByApplicationId(
     applicationId: number,
+    studentId?: number,
     ...status: ApplicationExceptionStatus[]
   ): Promise<ApplicationException[]> {
     const query = this.repo
@@ -84,6 +86,11 @@ export class ApplicationExceptionService extends RecordDataModelService<Applicat
     if (status) {
       query.andWhere("exception.exceptionStatus IN (:...status)", { status });
     }
+    if (studentId) {
+      query
+        .innerJoin("application.student", "student")
+        .andWhere("student.id = :studentId", { studentId });
+    }
     return query.getMany();
   }
 
diff --git a/sources/packages/backend/apps/api/src/services/education-program-offering/education-program-offering.service.ts b/sources/packages/backend/apps/api/src/services/education-program-offering/education-program-offering.service.ts
index 1498181303..3f7b3860f5 100644
--- a/sources/packages/backend/apps/api/src/services/education-program-offering/education-program-offering.service.ts
+++ b/sources/packages/backend/apps/api/src/services/education-program-offering/education-program-offering.service.ts
@@ -1204,12 +1204,14 @@ export class EducationProgramOfferingService extends RecordDataModelService<Educ
   /**
    * Get changed offering for an application Id.
    * @param applicationId Application id
+   * @param studentId student id.
    * @returns Offering and program details.
    */
   async getOfferingRequestsByApplicationId(
     applicationId: number,
+    studentId?: number,
   ): Promise<EducationProgramOffering> {
-    return this.repo
+    const offeringRequest = this.repo
       .createQueryBuilder("offering")
       .select([
         "offering.id",
@@ -1228,8 +1230,14 @@ export class EducationProgramOfferingService extends RecordDataModelService<Educ
       .where("application.id = :applicationId", { applicationId })
       .andWhere("precedingOffering.offeringStatus = :offeringStatus", {
         offeringStatus: OfferingStatus.ChangeUnderReview,
-      })
-      .getOne();
+      });
+
+    if (studentId) {
+      offeringRequest
+        .innerJoin("application.student", "student")
+        .andWhere("student.id = :studentId", { studentId });
+    }
+    return offeringRequest.getOne();
   }
 
   /**
diff --git a/sources/packages/web/src/components/aest/students/assessment/Request.vue b/sources/packages/web/src/components/aest/students/assessment/Request.vue
index 580661453c..76c2be976d 100644
--- a/sources/packages/web/src/components/aest/students/assessment/Request.vue
+++ b/sources/packages/web/src/components/aest/students/assessment/Request.vue
@@ -20,13 +20,13 @@
             <Column
               field="submittedDate"
               header="Submitted date"
-              sortable="true"
+              :sortable="true"
               ><template #body="slotProps">{{
                 dateOnlyLongString(slotProps.data.submittedDate)
               }}</template></Column
             >
-            <Column field="requestType" header="Type" sortable="true"></Column>
-            <Column header="Request form" sortable="false"
+            <Column field="requestType" header="Type" :sortable="true"></Column>
+            <Column header="Request form" :sortable="false"
               ><template #body="{ data }"
                 ><v-btn
                   @click="viewRequestForm(data)"
@@ -39,7 +39,7 @@
                 ></template
               ></Column
             >
-            <Column field="status" header="Status" sortable="true">
+            <Column field="status" header="Status" :sortable="true">
               <template #body="slotProps"
                 ><status-chip-requested-assessment
                   :status="slotProps.data.status" /></template
@@ -80,6 +80,10 @@ export default defineComponent({
       required: false,
       default: true,
     },
+    studentId: {
+      type: Number,
+      required: false,
+    },
   },
   setup(props, context) {
     const { dateOnlyLongString } = useFormatters();
@@ -89,6 +93,7 @@ export default defineComponent({
       requestedAssessment.value =
         await StudentAssessmentsService.shared.getAssessmentRequest(
           props.applicationId,
+          props.studentId,
         );
     });
 
diff --git a/sources/packages/web/src/constants/routes/RouteConstants.ts b/sources/packages/web/src/constants/routes/RouteConstants.ts
index d2443924ed..79eddb45ed 100644
--- a/sources/packages/web/src/constants/routes/RouteConstants.ts
+++ b/sources/packages/web/src/constants/routes/RouteConstants.ts
@@ -75,6 +75,7 @@ export const InstitutionRoutesConst = {
   STUDENT_RESTRICTIONS: Symbol(),
   STUDENT_FILE_UPLOADS: Symbol(),
   STUDENT_NOTES: Symbol(),
+  ASSESSMENTS_SUMMARY: Symbol(),
 };
 
 /**
diff --git a/sources/packages/web/src/router/AESTRoutes.ts b/sources/packages/web/src/router/AESTRoutes.ts
index b8da6b46c5..82fa631e2a 100644
--- a/sources/packages/web/src/router/AESTRoutes.ts
+++ b/sources/packages/web/src/router/AESTRoutes.ts
@@ -107,7 +107,7 @@ export const aestRoutes: Array<RouteRecordRaw> = [
         },
         children: [
           {
-            path: AppRoutes.AESTStudentProfile,
+            path: AppRoutes.StudentProfile,
             name: AESTRoutesConst.STUDENT_PROFILE,
             props: true,
             component: StudentProfile,
diff --git a/sources/packages/web/src/router/InstitutionRoutes.ts b/sources/packages/web/src/router/InstitutionRoutes.ts
index 7f55f3e6e4..72bba0be32 100644
--- a/sources/packages/web/src/router/InstitutionRoutes.ts
+++ b/sources/packages/web/src/router/InstitutionRoutes.ts
@@ -43,6 +43,7 @@ import InstitutionStudentApplications from "@/views/institution/student/Institut
 import InstitutionStudentRestrictions from "@/views/institution/student/InstitutionStudentRestrictions.vue";
 import InstitutionStudentFileUploads from "@/views/institution/student/InstitutionStudentFileUploads.vue";
 import InstitutionStudentNotes from "@/views/institution/student/InstitutionStudentNotes.vue";
+import InstitutionAssessmentsSummary from "@/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue";
 
 export const institutionRoutes: Array<RouteRecordRaw> = [
   {
@@ -447,7 +448,7 @@ export const institutionRoutes: Array<RouteRecordRaw> = [
         },
       },
       {
-        path: AppRoutes.InstitutionStudentDetail,
+        path: AppRoutes.StudentDetail,
         name: InstitutionRoutesConst.STUDENT_DETAILS,
         props: true,
         components: {
@@ -464,7 +465,7 @@ export const institutionRoutes: Array<RouteRecordRaw> = [
         },
         children: [
           {
-            path: AppRoutes.InstitutionStudentProfile,
+            path: AppRoutes.StudentProfile,
             name: InstitutionRoutesConst.STUDENT_PROFILE,
             props: true,
             component: InstitutionStudentProfile,
@@ -530,6 +531,21 @@ export const institutionRoutes: Array<RouteRecordRaw> = [
           },
         ],
       },
+      // todo: move the move inside application details as a child after guru, merge his PR
+
+      {
+        path: AppRoutes.InstitutionAssessmentSummary,
+        name: InstitutionRoutesConst.ASSESSMENTS_SUMMARY,
+        props: true,
+        component: InstitutionAssessmentsSummary,
+        meta: {
+          clientType: ClientIdType.Institution,
+          institutionUserTypes: [
+            InstitutionUserTypes.admin,
+            InstitutionUserTypes.user,
+          ],
+        },
+      },
     ],
     beforeEnter: (to, _from, next) => {
       AuthService.shared
diff --git a/sources/packages/web/src/services/StudentAssessmentsService.ts b/sources/packages/web/src/services/StudentAssessmentsService.ts
index 9e7f07727e..04e0fb3eed 100644
--- a/sources/packages/web/src/services/StudentAssessmentsService.ts
+++ b/sources/packages/web/src/services/StudentAssessmentsService.ts
@@ -21,13 +21,18 @@ export class StudentAssessmentsService {
    * Get all requested assessments for an student application,
    * i.e, this will fetch the combination of pending and denied
    * student appeal and scholastic standings for an application.
-   * @param applicationId, application number.
+   * @param applicationId, application id.
+   * @param studentId, student id.
    * @returns assessment requests for a student application.
    */
   async getAssessmentRequest(
     applicationId: number,
+    studentId?: number,
   ): Promise<RequestAssessmentSummaryAPIOutDTO[]> {
-    return ApiClient.StudentAssessmentApi.getAssessmentRequest(applicationId);
+    return ApiClient.StudentAssessmentApi.getAssessmentRequest(
+      applicationId,
+      studentId,
+    );
   }
   /**
    * Get history of assessments for an application,
diff --git a/sources/packages/web/src/services/http/StudentAssessmentApi.ts b/sources/packages/web/src/services/http/StudentAssessmentApi.ts
index 70ca73be56..68f93019ed 100644
--- a/sources/packages/web/src/services/http/StudentAssessmentApi.ts
+++ b/sources/packages/web/src/services/http/StudentAssessmentApi.ts
@@ -14,14 +14,20 @@ export class StudentAssessmentApi extends HttpBaseClient {
    * Get all requested assessments for an student application,
    * i.e, this will fetch the combination of pending and denied
    * student appeal and scholastic standings for an application.
-   * @param applicationId, application number.
+   * @param applicationId, application id.
+   * @param studentId, student id.
    * @returns Request assessment summary list.
    */
   public async getAssessmentRequest(
     applicationId: number,
+    studentId?: number,
   ): Promise<RequestAssessmentSummaryAPIOutDTO[]> {
+    const endpoint = studentId
+      ? `assessment/student/${studentId}/application/${applicationId}/requests`
+      : `assessment/application/${applicationId}/requests`;
+
     return this.getCall<RequestAssessmentSummaryAPIOutDTO[]>(
-      this.addClientRoot(`assessment/application/${applicationId}/requests`),
+      this.addClientRoot(endpoint),
     );
   }
   /**
diff --git a/sources/packages/web/src/types/AppRoutes.ts b/sources/packages/web/src/types/AppRoutes.ts
index 488d7f942b..5325e8d380 100644
--- a/sources/packages/web/src/types/AppRoutes.ts
+++ b/sources/packages/web/src/types/AppRoutes.ts
@@ -47,17 +47,14 @@ export enum AppRoutes {
   InstitutionManageUsers = "manage-users",
   ActiveApplicationScholasticStandingView = "active-application/location/:locationId/scholastic-standing/:scholasticStandingId",
   OfferingsUpload = "offerings-upload",
-  InstitutionStudentDetail = "student/:studentId",
-  InstitutionStudentProfile = "profile",
+  InstitutionAssessmentSummary = "assessment-summary",
   // AEST
   AESTRoot = "/ministry",
   AESTDashboard = "dashboard",
   SearchStudents = "search-students",
-  StudentDetail = "student/:studentId",
   ProgramDetail = "institution/:institutionId/location/:locationId/program-detail/:programId",
   SearchInstitutions = "search-institutions",
   InstitutionDetail = "institution-detail/:institutionId",
-  AESTStudentProfile = "profile",
   AESTEditInstitutionLocation = "institution-location/institution/:institutionId/location/:locationId/edit",
   AESTInstitutionProfile = "institution-profile",
   AESTInstitutionProfileEdit = "institution-profile/:institutionId/edit",
@@ -85,11 +82,7 @@ export enum AppRoutes {
   Locations = "locations",
   Users = "users",
   Designation = "designation",
-  Restrictions = "restrictions",
-  FileUploads = "files",
-  Notes = "notes",
   SINManagement = "sin-management",
-  Applications = "applications",
   ApplicationDetail = "student/:studentId/application/:applicationId",
   SupportingUserDetail = "supporting-user/:supportingUserId",
   // Supporting Users
@@ -106,4 +99,10 @@ export enum AppRoutes {
   StudentNotes = "student-notes/:studentId",
   StudentRestrictions = "student-restrictions/:studentId",
   Overawards = "overawards",
+  StudentDetail = "student/:studentId",
+  StudentProfile = "profile",
+  Applications = "applications",
+  Restrictions = "restrictions",
+  FileUploads = "files",
+  Notes = "notes",
 }
diff --git a/sources/packages/web/src/views/aest/AssessmentsSummary.vue b/sources/packages/web/src/views/aest/AssessmentsSummary.vue
index 06db14bdc6..c75e55f6dd 100644
--- a/sources/packages/web/src/views/aest/AssessmentsSummary.vue
+++ b/sources/packages/web/src/views/aest/AssessmentsSummary.vue
@@ -10,14 +10,14 @@
         subTitle="Assessment"
       />
     </template>
-    <RequestAssessment
+    <request-assessment
       class="mb-5"
       :applicationId="applicationId"
       @viewStudentAppeal="goToStudentAppeal"
       @viewApplicationException="goToApplicationException"
       @viewOfferingRequest="goToOfferingRequest"
     />
-    <HistoryAssessment
+    <history-assessment
       class="mb-5"
       :applicationId="applicationId"
       :viewRequestTypes="assessmentRequestTypes"
diff --git a/sources/packages/web/src/views/institution/student/InstitutionStudentApplications.vue b/sources/packages/web/src/views/institution/student/InstitutionStudentApplications.vue
index 5a390580aa..f1b958b7dc 100644
--- a/sources/packages/web/src/views/institution/student/InstitutionStudentApplications.vue
+++ b/sources/packages/web/src/views/institution/student/InstitutionStudentApplications.vue
@@ -3,6 +3,7 @@
     <student-applications
       :studentId="studentId"
       :enable-view-application="true"
+      @goToApplication="goToApplicationDetails"
     />
   </tab-container>
 </template>
@@ -10,6 +11,8 @@
 <script lang="ts">
 import { defineComponent } from "vue";
 import StudentApplications from "@/components/common/students/StudentApplications.vue";
+import { useRouter } from "vue-router";
+import { InstitutionRoutesConst } from "@/constants/routes/RouteConstants";
 
 export default defineComponent({
   components: { StudentApplications },
@@ -19,5 +22,21 @@ export default defineComponent({
       required: true,
     },
   },
+  setup() {
+    const router = useRouter();
+    // todo: This logic will be remove in guru's PR. ASSESSMENTS_SUMMARY is assessed by the new sidebar.
+    const goToApplicationDetails = (id: number) => {
+      console.log(id, "___________");
+      return router.push({
+        name: InstitutionRoutesConst.ASSESSMENTS_SUMMARY,
+        params: {
+          id: id,
+        },
+      });
+    };
+    return {
+      goToApplicationDetails,
+    };
+  },
 });
 </script>
diff --git a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
new file mode 100644
index 0000000000..8eec978324
--- /dev/null
+++ b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
@@ -0,0 +1,135 @@
+<template>
+  <full-page-container layout-template="centered">
+    <template #header>
+      <header-navigator
+        title="Student applications"
+        :routeLocation="{
+          name: AESTRoutesConst.STUDENT_APPLICATIONS,
+          params: { studentId },
+        }"
+        subTitle="Assessment"
+      />
+    </template>
+    <request-assessment
+      class="mb-5"
+      :applicationId="applicationId"
+      :studentId="studentId"
+      @viewStudentAppeal="goToStudentAppeal"
+      @viewApplicationException="goToApplicationException"
+      @viewOfferingRequest="goToOfferingRequest"
+    />
+    <history-assessment
+      :applicationId="applicationId"
+      :studentId="studentId"
+      :viewRequestTypes="assessmentRequestTypes"
+      @viewStudentAppeal="goToStudentAppeal"
+      @viewAssessment="gotToViewAssessment"
+      @viewOfferingRequest="goToOfferingRequest"
+      @viewApplicationException="goToApplicationException"
+      @viewScholasticStandingChange="goToScholasticStanding"
+    />
+  </full-page-container>
+</template>
+
+<script lang="ts">
+import { AESTRoutesConst } from "@/constants/routes/RouteConstants";
+import { useRouter } from "vue-router";
+import { defineComponent } from "vue";
+import { AssessmentTriggerType } from "@/types";
+import RequestAssessment from "@/components/aest/students/assessment/Request.vue";
+import HistoryAssessment from "@/components/aest/students/assessment/History.vue";
+// todo: remove the default value for student and institution id
+// todo: set required: true
+export default defineComponent({
+  components: {
+    RequestAssessment,
+    HistoryAssessment,
+  },
+  props: {
+    studentId: {
+      type: Number,
+      required: false,
+      default: 275,
+    },
+    applicationId: {
+      type: Number,
+      required: false,
+      default: 1734,
+    },
+  },
+  setup(props) {
+    // todo: check with jason if we need all form and view assessment for all types.
+    const router = useRouter();
+    // The assessment trigger types for which the request form must be visible by default.
+    const assessmentRequestTypes = [
+      AssessmentTriggerType.StudentAppeal,
+      AssessmentTriggerType.OfferingChange,
+      AssessmentTriggerType.ScholasticStandingChange,
+      AssessmentTriggerType.OriginalAssessment,
+    ];
+
+    const goToStudentAppeal = (appealId: number) => {
+      router.push({
+        name: AESTRoutesConst.STUDENT_APPEAL_REQUESTS_APPROVAL,
+        params: {
+          studentId: props.studentId,
+          applicationId: props.applicationId,
+          appealId,
+        },
+      });
+    };
+
+    const goToApplicationException = (exceptionId: number) => {
+      router.push({
+        name: AESTRoutesConst.APPLICATION_EXCEPTIONS_APPROVAL,
+        params: {
+          studentId: props.studentId,
+          applicationId: props.applicationId,
+          exceptionId,
+        },
+      });
+    };
+
+    const gotToViewAssessment = (assessmentId: number) => {
+      router.push({
+        name: AESTRoutesConst.ASSESSMENT_AWARD_VIEW,
+        params: {
+          studentId: props.studentId,
+          applicationId: props.applicationId,
+          assessmentId,
+        },
+      });
+    };
+
+    const goToScholasticStanding = (scholasticStandingId: number) => {
+      router.push({
+        name: AESTRoutesConst.SCHOLASTIC_STANDING_VIEW,
+        params: {
+          studentId: props.studentId,
+          applicationId: props.applicationId,
+          scholasticStandingId,
+        },
+      });
+    };
+    const goToOfferingRequest = (offeringId: number, programId: number) => {
+      router.push({
+        name: AESTRoutesConst.OFFERING_CHANGE_REQUEST_VIEW,
+        params: {
+          offeringId,
+          programId,
+        },
+      });
+    };
+
+    return {
+      AESTRoutesConst,
+      goToStudentAppeal,
+      gotToViewAssessment,
+      goToApplicationException,
+      goToScholasticStanding,
+      goToOfferingRequest,
+      assessmentRequestTypes,
+    };
+  },
+});
+</script>

From cb9d3978e003cb545c603293443ff61cc20efea9 Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Mon, 8 May 2023 20:31:15 -0400
Subject: [PATCH 02/12] exception and appeal

---
 .../backend/apps/api/src/app.aest.module.ts   |  4 +
 .../apps/api/src/app.institutions.module.ts   | 14 ++-
 .../application-exception.aest.controller.ts  | 22 +----
 ...pplication-exception.controller.service.ts | 44 +++++++++
 ...ation-exception.institutions.controller.ts | 42 ++++++++
 .../assessment.controller.service.ts          |  3 +-
 .../assessment.institutions.controller.ts     | 65 +++++-------
 ...ucation-program.institutions.controller.ts |  2 +-
 .../apps/api/src/route-controllers/index.ts   |  4 +
 .../student-appeal.aest.controller.ts         | 25 +----
 .../student-appeal.controller.service.ts      | 46 +++++++++
 .../student-appeal.institutions.controller.ts | 52 ++++++++++
 .../student-appeal.students.controller.ts     |  1 +
 .../application-exception.service.ts          | 19 +++-
 .../student-assessment.service.ts             |  2 +-
 .../student-scholastic-standings.service.ts   | 15 ++-
 .../aest/students/assessment/History.vue      |  5 +
 .../ApplicationExceptionsApproval.vue         | 79 ++++++---------
 .../StudentAppealRequestsApproval.vue         | 72 ++++----------
 .../sidebar/InstitutionApplicationSideBar.vue | 13 +++
 .../src/constants/routes/RouteConstants.ts    |  2 +
 sources/packages/web/src/router/AESTRoutes.ts |  8 +-
 .../web/src/router/InstitutionRoutes.ts       | 20 +++-
 .../services/ApplicationExceptionService.ts   |  9 +-
 .../web/src/services/StudentAppealService.ts  |  9 +-
 .../src/services/StudentAssessmentsService.ts |  9 +-
 .../services/http/ApplicationExceptionApi.ts  |  9 +-
 .../web/src/services/http/StudentAppealApi.ts | 10 +-
 .../src/services/http/StudentAssessmentApi.ts |  9 +-
 sources/packages/web/src/types/AppRoutes.ts   |  8 +-
 .../ApplicationExceptionsApproval.vue         | 77 +++++++++++++++
 .../StudentAppealRequestsApproval.vue         | 75 ++++++++++++++
 .../ApplicationExceptions.vue                 | 46 +++++++++
 .../InstitutionAssessmentsSummary.vue         | 98 ++++++++++---------
 .../StudentAppealRequest.vue                  | 44 +++++++++
 35 files changed, 706 insertions(+), 256 deletions(-)
 create mode 100644 sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.controller.service.ts
 create mode 100644 sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
 create mode 100644 sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
 create mode 100644 sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
 rename sources/packages/web/src/{views/aest => components/common/students/applicationDetails}/ApplicationExceptionsApproval.vue (67%)
 rename sources/packages/web/src/{views/aest => components/common/students/applicationDetails}/StudentAppealRequestsApproval.vue (68%)
 create mode 100644 sources/packages/web/src/views/aest/student/applicationDetails/ApplicationExceptionsApproval.vue
 create mode 100644 sources/packages/web/src/views/aest/student/applicationDetails/StudentAppealRequestsApproval.vue
 create mode 100644 sources/packages/web/src/views/institution/student/applicationDetails/ApplicationExceptions.vue
 create mode 100644 sources/packages/web/src/views/institution/student/applicationDetails/StudentAppealRequest.vue

diff --git a/sources/packages/backend/apps/api/src/app.aest.module.ts b/sources/packages/backend/apps/api/src/app.aest.module.ts
index 1461428adc..2a60ab30c7 100644
--- a/sources/packages/backend/apps/api/src/app.aest.module.ts
+++ b/sources/packages/backend/apps/api/src/app.aest.module.ts
@@ -64,6 +64,8 @@ import {
   ConfirmationOfEnrollmentControllerService,
   OverawardAESTController,
   OverawardControllerService,
+  ApplicationExceptionControllerService,
+  StudentAppealControllerService,
 } from "./route-controllers";
 import { AuthModule } from "./auth/auth.module";
 import {
@@ -160,6 +162,8 @@ import {
     NoteSharedService,
     OverawardControllerService,
     RestrictionSharedService,
+    ApplicationExceptionControllerService,
+    StudentAppealControllerService,
   ],
 })
 export class AppAESTModule {}
diff --git a/sources/packages/backend/apps/api/src/app.institutions.module.ts b/sources/packages/backend/apps/api/src/app.institutions.module.ts
index 7329a14c1d..874557b4ab 100644
--- a/sources/packages/backend/apps/api/src/app.institutions.module.ts
+++ b/sources/packages/backend/apps/api/src/app.institutions.module.ts
@@ -53,6 +53,13 @@ import {
   AssessmentInstitutionsController,
   AssessmentControllerService,
   OverawardControllerService,
+  ApplicationExceptionInstitutionsController,
+  ApplicationExceptionControllerService,
+  UserInstitutionsController,
+  ApplicationInstitutionsController,
+  UserControllerService,
+  StudentAppealInstitutionsController,
+  StudentAppealControllerService,
 } from "./route-controllers";
 import { AuthModule } from "./auth/auth.module";
 import {
@@ -69,9 +76,6 @@ import {
   SFASApplicationService,
   SFASPartTimeApplicationsService,
 } from "@sims/services/sfas";
-import { UserInstitutionsController } from "./route-controllers/user/user.institutions.controller";
-import { UserControllerService } from "./route-controllers/user/user.controller.service";
-import { ApplicationInstitutionsController } from "./route-controllers/application/application.institutions.controller";
 
 @Module({
   imports: [AuthModule],
@@ -90,6 +94,8 @@ import { ApplicationInstitutionsController } from "./route-controllers/applicati
     StudentInstitutionsController,
     AssessmentInstitutionsController,
     OverawardInstitutionsController,
+    ApplicationExceptionInstitutionsController,
+    StudentAppealInstitutionsController,
   ],
   providers: [
     ApplicationControllerService,
@@ -143,6 +149,8 @@ import { ApplicationInstitutionsController } from "./route-controllers/applicati
     ApplicationExceptionService,
     StudentAppealRequestsService,
     OverawardControllerService,
+    ApplicationExceptionControllerService,
+    StudentAppealControllerService,
   ],
 })
 export class AppInstitutionsModule {}
diff --git a/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.aest.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.aest.controller.ts
index 9f0ce01b36..537e83da4c 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.aest.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.aest.controller.ts
@@ -44,6 +44,7 @@ import {
 } from "../models/pagination.dto";
 import { Role } from "../../auth/roles.enum";
 import { WorkflowClientService } from "@sims/services";
+import { ApplicationExceptionControllerService } from "./application-exception.controller.service";
 
 @AllowAuthorizedParty(AuthorizedParties.aest)
 @Groups(UserGroups.AESTUser)
@@ -53,6 +54,7 @@ export class ApplicationExceptionAESTController extends BaseController {
   constructor(
     private readonly applicationExceptionService: ApplicationExceptionService,
     private readonly workflowClientService: WorkflowClientService,
+    private readonly applicationExceptionControllerService: ApplicationExceptionControllerService,
   ) {
     super();
   }
@@ -70,23 +72,9 @@ export class ApplicationExceptionAESTController extends BaseController {
   async getExceptionById(
     @Param("exceptionId", ParseIntPipe) exceptionId: number,
   ): Promise<ApplicationExceptionAPIOutDTO> {
-    const applicationException =
-      await this.applicationExceptionService.getExceptionById(exceptionId);
-    if (!applicationException) {
-      throw new NotFoundException("Student application exception not found.");
-    }
-    return {
-      exceptionStatus: applicationException.exceptionStatus,
-      submittedDate: applicationException.createdAt,
-      noteDescription: applicationException.exceptionNote?.description,
-      assessedByUserName: getUserFullName(applicationException.assessedBy),
-      assessedDate: applicationException.assessedDate,
-      exceptionRequests: applicationException.exceptionRequests.map(
-        (request) => ({
-          exceptionName: request.exceptionName,
-        }),
-      ),
-    };
+    return this.applicationExceptionControllerService.getExceptionDetails(
+      exceptionId,
+    );
   }
 
   /**
diff --git a/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.controller.service.ts b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.controller.service.ts
new file mode 100644
index 0000000000..9c54185066
--- /dev/null
+++ b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.controller.service.ts
@@ -0,0 +1,44 @@
+import { Injectable, NotFoundException } from "@nestjs/common";
+import { ApplicationExceptionService } from "../../services";
+import { getUserFullName } from "../../utilities";
+import { ApplicationExceptionAPIOutDTO } from "./models/application-exception.dto";
+
+@Injectable()
+export class ApplicationExceptionControllerService {
+  constructor(
+    private readonly applicationExceptionService: ApplicationExceptionService,
+  ) {}
+
+  /**
+   * Get a student application exception detected after the student application was
+   * submitted, for instance, when there are documents to be reviewed.
+   * @param exceptionId exception to be retrieved.
+   * @param studentId student id.
+   * @returns student application exception information.
+   */
+  async getExceptionDetails(
+    exceptionId: number,
+    studentId?: number,
+  ): Promise<ApplicationExceptionAPIOutDTO> {
+    const applicationException =
+      await this.applicationExceptionService.getExceptionDetails(
+        exceptionId,
+        studentId,
+      );
+    if (!applicationException) {
+      throw new NotFoundException("Student application exception not found.");
+    }
+    return {
+      exceptionStatus: applicationException.exceptionStatus,
+      submittedDate: applicationException.createdAt,
+      noteDescription: applicationException.exceptionNote?.description,
+      assessedByUserName: getUserFullName(applicationException.assessedBy),
+      assessedDate: applicationException.assessedDate,
+      exceptionRequests: applicationException.exceptionRequests.map(
+        (request) => ({
+          exceptionName: request.exceptionName,
+        }),
+      ),
+    };
+  }
+}
diff --git a/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
new file mode 100644
index 0000000000..e9e7803db2
--- /dev/null
+++ b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
@@ -0,0 +1,42 @@
+import { Controller, Get, Param, ParseIntPipe } from "@nestjs/common";
+import { AllowAuthorizedParty } from "../../auth/decorators";
+import { AuthorizedParties } from "../../auth/authorized-parties.enum";
+import { ClientTypeBaseRoute } from "../../types";
+import { ApiNotFoundResponse, ApiTags } from "@nestjs/swagger";
+import BaseController from "../BaseController";
+import { ApplicationExceptionAPIOutDTO } from "./models/application-exception.dto";
+import { ApplicationExceptionControllerService } from "./application-exception.controller.service";
+
+/**
+ * Application exception controller for institutions.
+ */
+@AllowAuthorizedParty(AuthorizedParties.institution)
+@Controller("application-exception")
+@ApiTags(`${ClientTypeBaseRoute.Institution}-application-exception`)
+export class ApplicationExceptionInstitutionsController extends BaseController {
+  constructor(
+    private readonly applicationExceptionControllerService: ApplicationExceptionControllerService,
+  ) {
+    super();
+  }
+
+  /**
+   * Get a student application exception of a student.
+   * @param studentId student id.
+   * @param exceptionId exception to be retrieved.
+   * @returns student application exception information.
+   */
+  @Get("student/:studentId/exception/:exceptionId")
+  @ApiNotFoundResponse({
+    description: "Student application exception not found.",
+  })
+  async getException(
+    @Param("studentId", ParseIntPipe) studentId: number,
+    @Param("exceptionId", ParseIntPipe) exceptionId: number,
+  ): Promise<ApplicationExceptionAPIOutDTO> {
+    return this.applicationExceptionControllerService.getExceptionDetails(
+      exceptionId,
+      studentId,
+    );
+  }
+}
diff --git a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.controller.service.ts b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.controller.service.ts
index fbc7d3b0fd..dd7132e404 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.controller.service.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.controller.service.ts
@@ -244,7 +244,7 @@ export class AssessmentControllerService {
    * Get history of approved assessment requests and
    * unsuccessful scholastic standing change requests(which will not create new assessment)
    * for an application.
-   * @param applicationId, application number.
+   * @param applicationId, application id.
    * @param studentId applicant student.
    * @returns summary of the assessment history for a student application.
    */
@@ -256,6 +256,7 @@ export class AssessmentControllerService {
       this.assessmentService.assessmentHistorySummary(applicationId, studentId),
       this.studentScholasticStandingsService.getUnsuccessfulScholasticStandings(
         applicationId,
+        studentId,
       ),
     ]);
     const history: AssessmentHistorySummaryAPIOutDTO[] = assessments.map(
diff --git a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
index dc31387798..3cde305224 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
@@ -2,44 +2,29 @@ import { Controller, Get, Param, ParseIntPipe } from "@nestjs/common";
 import BaseController from "../BaseController";
 import {
   AllowAuthorizedParty,
-  Groups,
   HasStudentDataAccess,
   IsBCPublicInstitution,
 } from "../../auth/decorators";
 import { AuthorizedParties } from "../../auth/authorized-parties.enum";
 import { ClientTypeBaseRoute } from "../../types";
-import { UserGroups } from "../../auth/user-groups.enum";
 import {
   AssessmentHistorySummaryAPIOutDTO,
-  AssessmentNOAAPIOutDTO,
   RequestAssessmentSummaryAPIOutDTO,
-  AwardDetailsAPIOutDTO,
-  RequestAssessmentTypeAPIOutDTO,
 } from "./models/assessment.dto";
-import {
-  ApiNotFoundResponse,
-  ApiTags,
-  ApiUnprocessableEntityResponse,
-} from "@nestjs/swagger";
+import { ApiTags } from "@nestjs/swagger";
 import { AssessmentControllerService } from "./assessment.controller.service";
-import {
-  EducationProgramOfferingService,
-  ApplicationExceptionService,
-} from "../../services";
-import { ApplicationExceptionStatus } from "@sims/sims-db";
 
 /**
  * Assessment controller for institutions.
  */
 @AllowAuthorizedParty(AuthorizedParties.institution)
 @Controller("assessment")
+@IsBCPublicInstitution()
 @ApiTags(`${ClientTypeBaseRoute.Institution}-assessment`)
 export class AssessmentInstitutionsController extends BaseController {
   constructor(
-    private readonly assessmentControllerService: AssessmentControllerService,
-  ) //   private readonly educationProgramOfferingService: EducationProgramOfferingService,
-  //   private readonly applicationExceptionService: ApplicationExceptionService,
-  {
+    private readonly assessmentControllerService: AssessmentControllerService, //   private readonly educationProgramOfferingService: EducationProgramOfferingService, //   private readonly applicationExceptionService: ApplicationExceptionService,
+  ) {
     super();
   }
 
@@ -50,9 +35,8 @@ export class AssessmentInstitutionsController extends BaseController {
    * @param applicationId application id.
    * @returns assessment requests or exceptions for the student application.
    */
-  // @IsBCPublicInstitution()
-  // @HasStudentDataAccess("studentId")
-  @Get("/student/:studentId/application/:applicationId/requests")
+  @HasStudentDataAccess("studentId")
+  @Get("student/:studentId/application/:applicationId/requests")
   async getRequestedAssessmentSummary(
     @Param("studentId", ParseIntPipe) studentId: number,
     @Param("applicationId", ParseIntPipe) applicationId: number,
@@ -63,23 +47,26 @@ export class AssessmentInstitutionsController extends BaseController {
     );
   }
 
-  // /**
-  //  * Method to get history of assessments for an application,
-  //  * i.e, this will have original assessment for the
-  //  * student application, and all approved student
-  //  * appeal and scholastic standings for the application
-  //  * which will have different assessment status.
-  //  * @param applicationId, application number.
-  //  * @returns summary of the assessment history for a student application.
-  //  */
-  // @Get("application/:applicationId/history")
-  // async getAssessmentHistorySummary(
-  //   @Param("applicationId", ParseIntPipe) applicationId: number,
-  // ): Promise<AssessmentHistorySummaryAPIOutDTO[]> {
-  //   return this.assessmentControllerService.getAssessmentHistorySummary(
-  //     applicationId,
-  //   );
-  // }
+  /**
+   * Method to get history of assessments for an application,
+   * i.e, this will have original assessment for the
+   * student application, and all approved student
+   * appeal and scholastic standings for the application
+   * which will have different assessment status.
+   * @param studentId student id.
+   * @param applicationId, application id.
+   * @returns summary of the assessment history for a student application.
+   */
+  @Get("student/:studentId/application/:applicationId/history")
+  async getAssessmentHistorySummary(
+    @Param("studentId", ParseIntPipe) studentId: number,
+    @Param("applicationId", ParseIntPipe) applicationId: number,
+  ): Promise<AssessmentHistorySummaryAPIOutDTO[]> {
+    return this.assessmentControllerService.getAssessmentHistorySummary(
+      applicationId,
+      studentId,
+    );
+  }
 
   // /**
   //  * Get the NOA values for a student application on a particular assessment.
diff --git a/sources/packages/backend/apps/api/src/route-controllers/education-program/education-program.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/education-program/education-program.institutions.controller.ts
index 00417de70f..d88c6f44c0 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/education-program/education-program.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/education-program/education-program.institutions.controller.ts
@@ -53,7 +53,7 @@ export class EducationProgramInstitutionsController extends BaseController {
    * @param paginationOptions pagination options.
    * @returns paginated programs summary.
    */
-  @HasLocationAccess("locationId")
+  // @HasLocationAccess("locationId")
   @Get("location/:locationId/summary")
   async getProgramsSummaryByLocationId(
     @Param("locationId", ParseIntPipe) locationId: number,
diff --git a/sources/packages/backend/apps/api/src/route-controllers/index.ts b/sources/packages/backend/apps/api/src/route-controllers/index.ts
index 081636dbc2..9fb09b963e 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/index.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/index.ts
@@ -60,3 +60,7 @@ export * from "./overaward/overaward.institutions.controller";
 export * from "./overaward/overaward.students.controller";
 export * from "./student/student.institutions.controller";
 export * from "./assessment/assessment.institutions.controller";
+export * from "./application-exception/application-exception.institutions.controller";
+export * from "./application-exception/application-exception.controller.service";
+export * from "./student-appeal/student-appeal.controller.service";
+export * from "./student-appeal/student-appeal.institutions.controller";
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
index e05271973e..3358d9860a 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
@@ -47,6 +47,7 @@ import {
   StudentAppealPendingPaginationOptionsAPIInDTO,
 } from "../models/pagination.dto";
 import { Role } from "../../auth/roles.enum";
+import { StudentAppealControllerService } from "./student-appeal.controller.service";
 
 @AllowAuthorizedParty(AuthorizedParties.aest)
 @Groups(UserGroups.AESTUser)
@@ -56,6 +57,7 @@ export class StudentAppealAESTController extends BaseController {
   constructor(
     private readonly studentAppealService: StudentAppealService,
     private readonly studentAssessmentService: StudentAssessmentService,
+    private readonly studentAppealControllerService: StudentAppealControllerService,
   ) {
     super();
   }
@@ -72,26 +74,9 @@ export class StudentAppealAESTController extends BaseController {
   async getStudentAppealWithRequests(
     @Param("appealId", ParseIntPipe) appealId: number,
   ): Promise<StudentAppealAPIOutDTO> {
-    const studentAppeal =
-      await this.studentAppealService.getAppealAndRequestsById(appealId);
-    if (!studentAppeal) {
-      throw new NotFoundException("Not able to find the student appeal.");
-    }
-
-    return {
-      id: studentAppeal.id,
-      submittedDate: studentAppeal.submittedDate,
-      status: studentAppeal.status,
-      appealRequests: studentAppeal.appealRequests.map((appealRequest) => ({
-        id: appealRequest.id,
-        appealStatus: appealRequest.appealStatus,
-        submittedData: appealRequest.submittedData,
-        submittedFormName: appealRequest.submittedFormName,
-        assessedDate: appealRequest.assessedDate,
-        noteDescription: appealRequest.note?.description,
-        assessedByUserName: getUserFullName(appealRequest.assessedBy),
-      })),
-    };
+    return this.studentAppealControllerService.getStudentAppealWithRequest(
+      appealId,
+    );
   }
 
   /**
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
new file mode 100644
index 0000000000..f8e8dd9e56
--- /dev/null
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
@@ -0,0 +1,46 @@
+import { NotFoundException } from "@nestjs/common";
+import { StudentAppealService } from "../../services";
+import BaseController from "../BaseController";
+import { StudentAppealAPIOutDTO } from "./models/student-appeal.dto";
+import { getUserFullName } from "../../utilities";
+
+export class StudentAppealControllerService extends BaseController {
+  constructor(private readonly studentAppealService: StudentAppealService) {
+    super();
+  }
+
+  /**
+   * Get the student appeal and its requests.
+   * @param appealId appeal id to be retrieved.
+   * @param studentId student id.
+   * @returns the student appeal and its requests.
+   */
+  async getStudentAppealWithRequest(
+    appealId: number,
+    studentId?: number,
+  ): Promise<StudentAppealAPIOutDTO> {
+    const studentAppeal =
+      await this.studentAppealService.getAppealAndRequestsById(
+        appealId,
+        studentId,
+      );
+    if (!studentAppeal) {
+      throw new NotFoundException("Not able to find the student appeal.");
+    }
+
+    return {
+      id: studentAppeal.id,
+      submittedDate: studentAppeal.submittedDate,
+      status: studentAppeal.status,
+      appealRequests: studentAppeal.appealRequests.map((appealRequest) => ({
+        id: appealRequest.id,
+        appealStatus: appealRequest.appealStatus,
+        submittedData: appealRequest.submittedData,
+        submittedFormName: appealRequest.submittedFormName,
+        assessedDate: appealRequest.assessedDate,
+        noteDescription: appealRequest.note?.description,
+        assessedByUserName: getUserFullName(appealRequest.assessedBy),
+      })),
+    };
+  }
+}
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
new file mode 100644
index 0000000000..77a31e6665
--- /dev/null
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
@@ -0,0 +1,52 @@
+import {
+  Controller,
+  Param,
+  Get,
+  ParseIntPipe,
+} from "@nestjs/common";
+import { AuthorizedParties } from "../../auth/authorized-parties.enum";
+import {
+  AllowAuthorizedParty,
+} from "../../auth/decorators";
+import {
+  ApiNotFoundResponse,
+  ApiTags,
+} from "@nestjs/swagger";
+import BaseController from "../BaseController";
+import {  ClientTypeBaseRoute } from "../../types";
+import { StudentAppealAPIOutDTO } from "./models/student-appeal.dto";
+import { StudentAppealControllerService } from "./student-appeal.controller.service";
+
+/**
+ * Student appeal controller for institutions.
+ */
+@AllowAuthorizedParty(AuthorizedParties.institution)
+@Controller("appeal")
+@ApiTags(`${ClientTypeBaseRoute.Institution}-appeal`)
+export class StudentAppealInstitutionsController extends BaseController {
+  constructor(
+    private readonly studentAppealControllerService: StudentAppealControllerService,
+  ) {
+    super();
+  }
+
+  /**
+   * Get the student appeal and its requests.
+   * @param appealId appeal id to be retrieved.
+   * @param studentId student id.
+   * @returns the student appeal and its requests.
+   */
+  @Get("student/:studentId/appeal/:appealId/requests")
+  @ApiNotFoundResponse({
+    description: "Not able to find the student appeal.",
+  })
+  async getStudentAppealWithRequest(
+    @Param("studentId", ParseIntPipe) studentId: number,
+    @Param("appealId", ParseIntPipe) appealId: number,
+  ): Promise<StudentAppealAPIOutDTO> {
+    return this.studentAppealControllerService.getStudentAppealWithRequest(
+      appealId,
+      studentId,
+    );
+  }
+}
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
index b4beb5bfc6..0ce07a948e 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
@@ -155,6 +155,7 @@ export class StudentAppealStudentsController extends BaseController {
   /**
    * Get the student appeal and its requests.
    * @param appealId appeal id to be retrieved.
+   * todo: ann controller service already created, dto is different work on it
    * @returns the student appeal and its requests.
    */
   @Get(":appealId/requests")
diff --git a/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts b/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts
index d1801192aa..2683afc67a 100644
--- a/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts
+++ b/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts
@@ -39,10 +39,14 @@ export class ApplicationExceptionService extends RecordDataModelService<Applicat
    * Get a student application exception detected after the student application was
    * submitted, for instance, when there are documents to be reviewed.
    * @param exceptionId exception to be retrieved.
+   * @param studentId student id.
    * @returns student application exception information.
    */
-  async getExceptionById(exceptionId: number): Promise<ApplicationException> {
-    return this.repo
+  async getExceptionDetails(
+    exceptionId: number,
+    studentId?: number,
+  ): Promise<ApplicationException> {
+    const exception = this.repo
       .createQueryBuilder("exception")
       .select([
         "exception.id",
@@ -57,8 +61,15 @@ export class ApplicationExceptionService extends RecordDataModelService<Applicat
       .innerJoin("exception.exceptionRequests", "exceptionRequest")
       .leftJoin("exception.exceptionNote", "exceptionNote")
       .leftJoin("exception.assessedBy", "assessedBy")
-      .where("exception.id = :exceptionId", { exceptionId })
-      .getOne();
+      .where("exception.id = :exceptionId", { exceptionId });
+
+    if (studentId) {
+      exception
+        .innerJoin("exception.application", "application")
+        .innerJoin("application.student", "student")
+        .andWhere("student.id = :studentId", { studentId });
+    }
+    return exception.getOne();
   }
 
   /**
diff --git a/sources/packages/backend/apps/api/src/services/student-assessment/student-assessment.service.ts b/sources/packages/backend/apps/api/src/services/student-assessment/student-assessment.service.ts
index b904df3562..6732ed17fd 100644
--- a/sources/packages/backend/apps/api/src/services/student-assessment/student-assessment.service.ts
+++ b/sources/packages/backend/apps/api/src/services/student-assessment/student-assessment.service.ts
@@ -234,7 +234,7 @@ export class StudentAssessmentService extends RecordDataModelService<StudentAsse
    * * WHEN 3:if assessmentWorkflowId is not null
    * * and assessmentData is not null, then status
    * * is Completed.
-   * @param applicationId applicationId.
+   * @param applicationId application id.
    * @param studentId applicant student.
    * @returns AssessmentHistory list
    */
diff --git a/sources/packages/backend/apps/api/src/services/student-scholastic-standings/student-scholastic-standings.service.ts b/sources/packages/backend/apps/api/src/services/student-scholastic-standings/student-scholastic-standings.service.ts
index 12df9f998e..775670299f 100644
--- a/sources/packages/backend/apps/api/src/services/student-scholastic-standings/student-scholastic-standings.service.ts
+++ b/sources/packages/backend/apps/api/src/services/student-scholastic-standings/student-scholastic-standings.service.ts
@@ -457,12 +457,14 @@ export class StudentScholasticStandingsService extends RecordDataModelService<St
   /**
    * Get unsuccessful scholastic standing of a student application.
    * @param applicationId application id.
+   * @param studentId student id.
    * @return student scholastic standing.
    */
   async getUnsuccessfulScholasticStandings(
     applicationId: number,
+    studentId?: number,
   ): Promise<StudentScholasticStanding> {
-    return this.repo
+    const unsuccessfulScholasticStandings = this.repo
       .createQueryBuilder("studentScholasticStanding")
       .select([
         "studentScholasticStanding.id",
@@ -470,8 +472,15 @@ export class StudentScholasticStandingsService extends RecordDataModelService<St
       ])
       .innerJoin("studentScholasticStanding.application", "application")
       .where("application.id = :applicationId", { applicationId })
-      .andWhere("studentScholasticStanding.unsuccessfulWeeks IS NOT NULL")
-      .getOne();
+      .andWhere("studentScholasticStanding.unsuccessfulWeeks IS NOT NULL");
+
+    if (studentId) {
+      unsuccessfulScholasticStandings
+        .innerJoin("application.student", "student")
+        .andWhere("student.id = :studentId", { studentId });
+    }
+
+    return unsuccessfulScholasticStandings.getOne();
   }
 
   /**
diff --git a/sources/packages/web/src/components/aest/students/assessment/History.vue b/sources/packages/web/src/components/aest/students/assessment/History.vue
index 4c36a7f791..deaa549b63 100644
--- a/sources/packages/web/src/components/aest/students/assessment/History.vue
+++ b/sources/packages/web/src/components/aest/students/assessment/History.vue
@@ -106,6 +106,10 @@ export default defineComponent({
       type: Array as PropType<AssessmentTriggerType[]>,
       required: true,
     },
+    studentId: {
+      type: Number,
+      required: false,
+    },
   },
   setup(props, context) {
     const { dateOnlyLongString } = useFormatters();
@@ -114,6 +118,7 @@ export default defineComponent({
       assessmentHistory.value =
         await StudentAssessmentsService.shared.getAssessmentHistory(
           props.applicationId,
+          props.studentId,
         );
     });
 
diff --git a/sources/packages/web/src/views/aest/ApplicationExceptionsApproval.vue b/sources/packages/web/src/components/common/students/applicationDetails/ApplicationExceptionsApproval.vue
similarity index 67%
rename from sources/packages/web/src/views/aest/ApplicationExceptionsApproval.vue
rename to sources/packages/web/src/components/common/students/applicationDetails/ApplicationExceptionsApproval.vue
index 9b79557354..ace39cb9d1 100644
--- a/sources/packages/web/src/views/aest/ApplicationExceptionsApproval.vue
+++ b/sources/packages/web/src/components/common/students/applicationDetails/ApplicationExceptionsApproval.vue
@@ -4,7 +4,7 @@
       <header-navigator
         title="Assessments"
         subTitle="View Submission"
-        :routeLocation="assessmentsSummaryRoute"
+        :routeLocation="backRouteLocation"
       />
     </template>
     <template #sub-header>
@@ -13,7 +13,8 @@
     <formio-container
       formName="studentExceptions"
       :formData="applicationExceptions"
-      @submitted="submitted"
+      :readOnly="readOnlyForm"
+      @submitted="$emit('submitted', $event)"
     >
       <template #actions="{ submit }" v-if="!readOnly">
         <check-permission-role :role="Role.StudentApproveDeclineExceptions">
@@ -32,16 +33,12 @@
   </full-page-container>
 </template>
 <script lang="ts">
-import { ref, onMounted, defineComponent } from "vue";
-import { AESTRoutesConst } from "@/constants/routes/RouteConstants";
-import { useRouter } from "vue-router";
+import { ref, onMounted, defineComponent, PropType } from "vue";
+import { RouteLocationRaw, useRouter } from "vue-router";
 import { ApplicationExceptionService } from "@/services/ApplicationExceptionService";
-import { ApplicationExceptionStatus, FormIOForm, Role } from "@/types";
-import {
-  ApplicationExceptionAPIOutDTO,
-  UpdateApplicationExceptionAPIInDTO,
-} from "@/services/http/dto";
-import { useAssessment, useFormatters, useSnackBar } from "@/composables";
+import { ApplicationExceptionStatus, Role } from "@/types";
+import { ApplicationExceptionAPIOutDTO } from "@/services/http/dto";
+import { useAssessment, useFormatters } from "@/composables";
 import HeaderTitleValue from "@/components/generic/HeaderTitleValue.vue";
 import CheckPermissionRole from "@/components/generic/CheckPermissionRole.vue";
 
@@ -77,35 +74,46 @@ type ApplicationExceptionFormModel = Omit<
 };
 
 export default defineComponent({
+  // todo: ann emit new
+  emits: ["submitted"],
   components: { HeaderTitleValue, CheckPermissionRole },
   props: {
     studentId: {
       type: Number,
-      required: true,
+      required: false,
     },
-    applicationId: {
+    exceptionId: {
       type: Number,
       required: true,
     },
-    exceptionId: {
-      type: Number,
+    backRouteLocation: {
+      type: Object as PropType<RouteLocationRaw>,
       required: true,
     },
+    processing: {
+      type: Boolean,
+      required: false,
+      default: false,
+    },
+    readOnlyForm: {
+      type: Boolean,
+      required: false,
+      default: false,
+    },
   },
   setup(props) {
     const router = useRouter();
-    const snackBar = useSnackBar();
     const { dateOnlyLongString } = useFormatters();
     const { mapRequestAssessmentChipStatus } = useAssessment();
     const applicationExceptions = ref({} as ApplicationExceptionFormModel);
     const submittedDate = ref("");
-    const processing = ref(false);
     const readOnly = ref(true);
 
     onMounted(async () => {
       const applicationException =
-        await ApplicationExceptionService.shared.getExceptionById(
+        await ApplicationExceptionService.shared.getExceptionDetails(
           props.exceptionId,
+          props.studentId,
         );
       applicationExceptions.value = {
         ...applicationException,
@@ -123,48 +131,17 @@ export default defineComponent({
       );
       readOnly.value =
         applicationException.exceptionStatus !==
-        ApplicationExceptionStatus.Pending;
+          ApplicationExceptionStatus.Pending || props.readOnlyForm;
     });
 
-    const assessmentsSummaryRoute = {
-      name: AESTRoutesConst.ASSESSMENTS_SUMMARY,
-      params: {
-        applicationId: props.applicationId,
-        studentId: props.studentId,
-      },
-    };
-
     const gotToAssessmentsSummary = () => {
-      router.push(assessmentsSummaryRoute);
-    };
-
-    const submitted = async (form: FormIOForm) => {
-      processing.value = true;
-      try {
-        const approveExceptionPayload =
-          form.data as UpdateApplicationExceptionAPIInDTO;
-        await ApplicationExceptionService.shared.approveException(
-          props.exceptionId,
-          approveExceptionPayload,
-        );
-        snackBar.success(
-          `Application exception status is now ${approveExceptionPayload.exceptionStatus}.`,
-        );
-        gotToAssessmentsSummary();
-      } catch {
-        snackBar.error("An unexpected error happened during the approval.");
-      } finally {
-        processing.value = false;
-      }
+      router.push(props.backRouteLocation);
     };
 
     return {
       gotToAssessmentsSummary,
-      assessmentsSummaryRoute,
       applicationExceptions,
-      submitted,
       submittedDate,
-      processing,
       readOnly,
       Role,
     };
diff --git a/sources/packages/web/src/views/aest/StudentAppealRequestsApproval.vue b/sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue
similarity index 68%
rename from sources/packages/web/src/views/aest/StudentAppealRequestsApproval.vue
rename to sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue
index 3e306db880..27a61a7689 100644
--- a/sources/packages/web/src/views/aest/StudentAppealRequestsApproval.vue
+++ b/sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue
@@ -1,10 +1,11 @@
 <template>
+  <!-- todo: why container?? -->
   <v-container>
     <div class="mb-4">
       <header-navigator
         title="Assessment"
         subTitle="View Request"
-        :routeLocation="assessmentsSummaryRoute"
+        :routeLocation="backRouteLocation"
       />
     </div>
     <full-page-container>
@@ -18,7 +19,7 @@
       <appeal-requests-approval-form
         :studentAppealRequests="studentAppealRequests"
         :readOnly="readOnly"
-        @submitted="submitted"
+        @submitted="$emit('submitted', $event)"
       >
         <template #approval-actions="{ submit }" v-if="!readOnly">
           <v-row justify="center" class="m-2">
@@ -49,24 +50,17 @@
   </v-container>
 </template>
 <script lang="ts">
-import { ref, onMounted, computed, defineComponent } from "vue";
-import { AESTRoutesConst } from "@/constants/routes/RouteConstants";
-import { useRouter } from "vue-router";
+import { ref, onMounted, computed, defineComponent, PropType } from "vue";
+import { RouteLocationRaw, useRouter } from "vue-router";
 import { StudentAppealService } from "@/services/StudentAppealService";
-import { useFormatters, useSnackBar } from "@/composables";
-import {
-  StudentAppealRequest,
-  StudentAppealApproval,
-  StudentAppealStatus,
-  ApiProcessError,
-  Role,
-} from "@/types";
+import { useFormatters } from "@/composables";
+import { StudentAppealRequest, StudentAppealStatus, Role } from "@/types";
 import AppealRequestsApprovalForm from "@/components/aest/AppealRequestsApprovalForm.vue";
 import StatusChipRequestedAssessment from "@/components/generic/StatusChipRequestedAssessment.vue";
-import { ASSESSMENT_ALREADY_IN_PROGRESS } from "@/services/http/dto/Assessment.dto";
 import CheckPermissionRole from "@/components/generic/CheckPermissionRole.vue";
 
 export default defineComponent({
+  emit: ["submitted"],
   components: {
     AppealRequestsApprovalForm,
     StatusChipRequestedAssessment,
@@ -77,30 +71,36 @@ export default defineComponent({
       type: Number,
       required: true,
     },
-    applicationId: {
+    appealId: {
       type: Number,
       required: true,
     },
-    appealId: {
-      type: Number,
+    backRouteLocation: {
+      type: Object as PropType<RouteLocationRaw>,
       required: true,
     },
+    readOnlyForm: {
+      type: Boolean,
+      required: false,
+      default: false,
+    },
   },
   setup(props) {
     const router = useRouter();
-
-    const snackBar = useSnackBar();
     const { dateOnlyLongString } = useFormatters();
     const studentAppealRequests = ref([] as StudentAppealRequest[]);
     const appealStatus = ref(StudentAppealStatus.Pending);
     const readOnly = computed(
-      () => appealStatus.value !== StudentAppealStatus.Pending,
+      () =>
+        appealStatus.value !== StudentAppealStatus.Pending ||
+        props.readOnlyForm,
     );
 
     onMounted(async () => {
       const appeal =
         await StudentAppealService.shared.getStudentAppealWithRequests(
           props.appealId,
+          props.studentId,
         );
       studentAppealRequests.value = appeal.appealRequests.map((request) => ({
         id: request.id,
@@ -118,43 +118,13 @@ export default defineComponent({
       appealStatus.value = appeal.status;
     });
 
-    const assessmentsSummaryRoute = {
-      name: AESTRoutesConst.ASSESSMENTS_SUMMARY,
-      params: {
-        applicationId: props.applicationId,
-        studentId: props.studentId,
-      },
-    };
-
     const gotToAssessmentsSummary = () => {
-      router.push(assessmentsSummaryRoute);
-    };
-
-    const submitted = async (approvals: StudentAppealApproval[]) => {
-      try {
-        await StudentAppealService.shared.approveStudentAppealRequests(
-          props.appealId,
-          approvals,
-        );
-        snackBar.success("The request was completed with success.");
-
-        gotToAssessmentsSummary();
-      } catch (error: unknown) {
-        if (error instanceof ApiProcessError) {
-          if (error.errorType === ASSESSMENT_ALREADY_IN_PROGRESS) {
-            snackBar.warn(error.message);
-            return;
-          }
-        }
-        snackBar.error("An unexpected error happened during the approval.");
-      }
+      router.push(props.backRouteLocation);
     };
 
     return {
       gotToAssessmentsSummary,
-      assessmentsSummaryRoute,
       studentAppealRequests,
-      submitted,
       appealStatus,
       readOnly,
       Role,
diff --git a/sources/packages/web/src/components/layouts/Institution/sidebar/InstitutionApplicationSideBar.vue b/sources/packages/web/src/components/layouts/Institution/sidebar/InstitutionApplicationSideBar.vue
index 0dbb5b4dd2..c744f89dec 100644
--- a/sources/packages/web/src/components/layouts/Institution/sidebar/InstitutionApplicationSideBar.vue
+++ b/sources/packages/web/src/components/layouts/Institution/sidebar/InstitutionApplicationSideBar.vue
@@ -42,6 +42,19 @@ export default defineComponent({
           },
         },
       },
+      {
+        title: "Assessments",
+        props: {
+          prependIcon: "mdi-checkbox-marked-outline",
+          to: {
+            name: InstitutionRoutesConst.ASSESSMENTS_SUMMARY,
+            params: {
+              applicationId: props.applicationId,
+              studentId: props.studentId,
+            },
+          },
+        },
+      },
     ]);
     return {
       items,
diff --git a/sources/packages/web/src/constants/routes/RouteConstants.ts b/sources/packages/web/src/constants/routes/RouteConstants.ts
index 9345a85db6..3492fc049d 100644
--- a/sources/packages/web/src/constants/routes/RouteConstants.ts
+++ b/sources/packages/web/src/constants/routes/RouteConstants.ts
@@ -78,6 +78,8 @@ export const InstitutionRoutesConst = {
   STUDENT_OVERAWARDS: Symbol(),
   STUDENT_NOTES: Symbol(),
   ASSESSMENTS_SUMMARY: Symbol(),
+  APPLICATION_EXCEPTION: Symbol(),
+  STUDENT_APPEAL_REQUEST: Symbol(),
 };
 
 /**
diff --git a/sources/packages/web/src/router/AESTRoutes.ts b/sources/packages/web/src/router/AESTRoutes.ts
index bf7a2b22f4..2758d1630e 100644
--- a/sources/packages/web/src/router/AESTRoutes.ts
+++ b/sources/packages/web/src/router/AESTRoutes.ts
@@ -41,9 +41,9 @@ import SupportingUser from "@/views/aest/SupportingUser.vue";
 import PendingDesignations from "@/views/aest/institution/PendingDesignation.vue";
 import DesignationAESTView from "@/views/aest/institution/DesignationAESTView.vue";
 import AssessmentsSummary from "@/views/aest/AssessmentsSummary.vue";
-import StudentAppealRequestsApproval from "@/views/aest/StudentAppealRequestsApproval.vue";
+import StudentAppealRequestsApproval from "@/views/aest/student/applicationDetails/StudentAppealRequestsApproval.vue";
 import NoticeOfAssessment from "@/views/aest/NoticeOfAssessment.vue";
-import ApplicationExceptionsApproval from "@/views/aest/ApplicationExceptionsApproval.vue";
+import ApplicationExceptionsApproval from "@/views/aest/student/applicationDetails/ApplicationExceptionsApproval.vue";
 import ViewScholasticStanding from "@/views/aest/student/ViewScholasticStanding.vue";
 import SINManagement from "@/views/aest/student/SINManagement.vue";
 import Overawards from "@/views/aest/student/Overawards.vue";
@@ -220,7 +220,7 @@ export const aestRoutes: Array<RouteRecordRaw> = [
             },
           },
           {
-            path: AppRoutes.StudentAppealRequestsApproval,
+            path: AppRoutes.StudentAppealRequest,
             name: AESTRoutesConst.STUDENT_APPEAL_REQUESTS_APPROVAL,
             props: true,
             component: StudentAppealRequestsApproval,
@@ -238,7 +238,7 @@ export const aestRoutes: Array<RouteRecordRaw> = [
             },
           },
           {
-            path: AppRoutes.ApplicationExceptionsApproval,
+            path: AppRoutes.ApplicationException,
             name: AESTRoutesConst.APPLICATION_EXCEPTIONS_APPROVAL,
             props: true,
             component: ApplicationExceptionsApproval,
diff --git a/sources/packages/web/src/router/InstitutionRoutes.ts b/sources/packages/web/src/router/InstitutionRoutes.ts
index 743e9b0ff4..00e0aa1532 100644
--- a/sources/packages/web/src/router/InstitutionRoutes.ts
+++ b/sources/packages/web/src/router/InstitutionRoutes.ts
@@ -48,6 +48,8 @@ import InstitutionStudentFileUploads from "@/views/institution/student/Instituti
 import InstitutionStudentOverawards from "@/views/institution/student/InstitutionStudentOverawards.vue";
 import InstitutionStudentNotes from "@/views/institution/student/InstitutionStudentNotes.vue";
 import InstitutionAssessmentsSummary from "@/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue";
+import ApplicationExceptions from "@/views/institution/student/applicationDetails/ApplicationExceptions.vue";
+import StudentAppealRequest from "@/views/institution/student/applicationDetails/StudentAppealRequest.vue";
 
 export const institutionRoutes: Array<RouteRecordRaw> = [
   {
@@ -174,7 +176,7 @@ export const institutionRoutes: Array<RouteRecordRaw> = [
         },
       },
       {
-        path: `${AppRoutes.LocationProgramInfoRequestSummary}`,
+        path: AppRoutes.LocationProgramInfoRequestSummary,
         name: InstitutionRoutesConst.PROGRAM_INFO_REQUEST_SUMMARY,
         components: {
           default: LocationProgramInfoRequestSummary,
@@ -194,7 +196,7 @@ export const institutionRoutes: Array<RouteRecordRaw> = [
         },
       },
       {
-        path: `${AppRoutes.LocationCOESummary}`,
+        path: AppRoutes.LocationCOESummary,
         name: InstitutionRoutesConst.COE_SUMMARY,
         components: {
           default: LocationCOESummary,
@@ -588,11 +590,23 @@ export const institutionRoutes: Array<RouteRecordRaw> = [
             component: InstitutionApplicationView,
           },
           {
-            path: AppRoutes.InstitutionAssessmentSummary,
+            path: AppRoutes.AssessmentSummary,
             name: InstitutionRoutesConst.ASSESSMENTS_SUMMARY,
             props: true,
             component: InstitutionAssessmentsSummary,
           },
+          {
+            path: AppRoutes.ApplicationException,
+            name: InstitutionRoutesConst.APPLICATION_EXCEPTION,
+            props: true,
+            component: ApplicationExceptions,
+          },
+          {
+            path: AppRoutes.StudentAppealRequest,
+            name: InstitutionRoutesConst.STUDENT_APPEAL_REQUEST,
+            props: true,
+            component: StudentAppealRequest,
+          },
         ],
       },
     ],
diff --git a/sources/packages/web/src/services/ApplicationExceptionService.ts b/sources/packages/web/src/services/ApplicationExceptionService.ts
index 41ab66bb08..d6b263b53c 100644
--- a/sources/packages/web/src/services/ApplicationExceptionService.ts
+++ b/sources/packages/web/src/services/ApplicationExceptionService.ts
@@ -18,12 +18,17 @@ export class ApplicationExceptionService {
    * Get a student application exception detected after the student application was
    * submitted, for instance, when there are documents to be reviewed.
    * @param exceptionId exception to be retrieved.
+   * @param studentId student id.
    * @returns student application exception information.
    */
-  async getExceptionById(
+  async getExceptionDetails(
     exceptionId: number,
+    studentId?: number,
   ): Promise<ApplicationExceptionAPIOutDTO> {
-    return ApiClient.ApplicationExceptionApi.getExceptionById(exceptionId);
+    return ApiClient.ApplicationExceptionApi.getExceptionDetails(
+      exceptionId,
+      studentId,
+    );
   }
 
   /**
diff --git a/sources/packages/web/src/services/StudentAppealService.ts b/sources/packages/web/src/services/StudentAppealService.ts
index a154b9abfb..19682d0e02 100644
--- a/sources/packages/web/src/services/StudentAppealService.ts
+++ b/sources/packages/web/src/services/StudentAppealService.ts
@@ -35,10 +35,17 @@ export class StudentAppealService {
     });
   }
 
+  /**
+   * Get student application appeal.
+   * @param appealId appeal id.
+   * @param studentId student id.
+   * @returns student application appeal.
+   */
   async getStudentAppealWithRequests(
     appealId: number,
+    studentId?: number,
   ): Promise<StudentAppealAPIOutDTO> {
-    return ApiClient.StudentAppealApi.getStudentAppealWithRequests(appealId);
+    return ApiClient.StudentAppealApi.getStudentAppealWithRequests(appealId, studentId);
   }
 
   async approveStudentAppealRequests(
diff --git a/sources/packages/web/src/services/StudentAssessmentsService.ts b/sources/packages/web/src/services/StudentAssessmentsService.ts
index 04e0fb3eed..debc86d99e 100644
--- a/sources/packages/web/src/services/StudentAssessmentsService.ts
+++ b/sources/packages/web/src/services/StudentAssessmentsService.ts
@@ -40,13 +40,18 @@ export class StudentAssessmentsService {
    * student application, and all approved student
    * appeal and scholastic standings for the application
    * which will have different assessment status.
-   * @param applicationId, application number.
+   * @param applicationId, application id.
+   * @param studentId, student id.
    * @returns summary of the assessment history for a student application.
    */
   async getAssessmentHistory(
     applicationId: number,
+    studentId?: number,
   ): Promise<AssessmentHistorySummaryAPIOutDTO[]> {
-    return ApiClient.StudentAssessmentApi.getAssessmentHistory(applicationId);
+    return ApiClient.StudentAssessmentApi.getAssessmentHistory(
+      applicationId,
+      studentId,
+    );
   }
 
   /**
diff --git a/sources/packages/web/src/services/http/ApplicationExceptionApi.ts b/sources/packages/web/src/services/http/ApplicationExceptionApi.ts
index 49fcfbf986..2d2bd7ddfb 100644
--- a/sources/packages/web/src/services/http/ApplicationExceptionApi.ts
+++ b/sources/packages/web/src/services/http/ApplicationExceptionApi.ts
@@ -18,13 +18,18 @@ export class ApplicationExceptionApi extends HttpBaseClient {
    * Get a student application exception detected after the student application was
    * submitted, for instance, when there are documents to be reviewed.
    * @param exceptionId exception to be retrieved.
+   * @param studentId student id.
    * @returns student application exception information.
    */
-  async getExceptionById(
+  async getExceptionDetails(
     exceptionId: number,
+    studentId?: number,
   ): Promise<ApplicationExceptionAPIOutDTO> {
+    const endpoint = studentId
+      ? `application-exception/student/${studentId}/exception/${exceptionId}`
+      : `application-exception/${exceptionId}`;
     return this.getCall<ApplicationExceptionAPIOutDTO>(
-      this.addClientRoot(`application-exception/${exceptionId}`),
+      this.addClientRoot(endpoint),
     );
   }
 
diff --git a/sources/packages/web/src/services/http/StudentAppealApi.ts b/sources/packages/web/src/services/http/StudentAppealApi.ts
index 1154e1d144..d10ea105da 100644
--- a/sources/packages/web/src/services/http/StudentAppealApi.ts
+++ b/sources/packages/web/src/services/http/StudentAppealApi.ts
@@ -24,11 +24,19 @@ export class StudentAppealApi extends HttpBaseClient {
     );
   }
 
+  /**
+   * Get student application appeal.
+   * @param appealId appeal id.
+   * @param studentId student id.
+   * @returns student application appeal.
+   */
   async getStudentAppealWithRequests(
     appealId: number,
+    studentId?: number,
   ): Promise<StudentAppealAPIOutDTO> {
+    const endpoint = studentId ? `appeal/student/${studentId}/appeal/${appealId}/requests` : `appeal/${appealId}/requests`;
     return this.getCall<StudentAppealAPIOutDTO>(
-      this.addClientRoot(`appeal/${appealId}/requests`),
+      this.addClientRoot(endpoint),
     );
   }
 
diff --git a/sources/packages/web/src/services/http/StudentAssessmentApi.ts b/sources/packages/web/src/services/http/StudentAssessmentApi.ts
index 68f93019ed..baed450632 100644
--- a/sources/packages/web/src/services/http/StudentAssessmentApi.ts
+++ b/sources/packages/web/src/services/http/StudentAssessmentApi.ts
@@ -36,14 +36,19 @@ export class StudentAssessmentApi extends HttpBaseClient {
    * student application, and all approved student
    * appeal and scholastic standings for the application
    * which will have different assessment status.
-   * @param applicationId, application number.
+   * @param applicationId, application id.
+   * @param studentId, student id.
    * @returns AssessmentHistorySummaryDTO list.
    */
   public async getAssessmentHistory(
     applicationId: number,
+    studentId?: number,
   ): Promise<AssessmentHistorySummaryAPIOutDTO[]> {
+    const endpoint = studentId
+      ? `assessment/student/${studentId}/application/${applicationId}/history`
+      : `assessment/application/${applicationId}/history`;
     return this.getCall<AssessmentHistorySummaryAPIOutDTO[]>(
-      this.addClientRoot(`assessment/application/${applicationId}/history`),
+      this.addClientRoot(endpoint),
     );
   }
 
diff --git a/sources/packages/web/src/types/AppRoutes.ts b/sources/packages/web/src/types/AppRoutes.ts
index 59bb9c1569..ef71775a29 100644
--- a/sources/packages/web/src/types/AppRoutes.ts
+++ b/sources/packages/web/src/types/AppRoutes.ts
@@ -47,7 +47,6 @@ export enum AppRoutes {
   InstitutionManageUsers = "manage-users",
   ActiveApplicationScholasticStandingView = "active-application/location/:locationId/scholastic-standing/:scholasticStandingId",
   OfferingsUpload = "offerings-upload",
-  InstitutionAssessmentSummary = "assessment-summary",
   // AEST
   AESTRoot = "/ministry",
   AESTDashboard = "dashboard",
@@ -61,12 +60,9 @@ export enum AppRoutes {
   AESTInstitutionProfileCreate = "institution-profile/create",
   PendingDesignations = "institution/designation/pending",
   DesignationAESTView = "institution/designation/:designationId/institution/:institutionId?",
-  AssessmentSummary = "assessment-summary",
   ScholasticStandingView = "scholastic-standing/:scholasticStandingId",
-  StudentAppealRequestsApproval = "request-changes/:appealId",
   NoticeOfAssessmentView = "notice-of-assessment/:assessmentId",
   Reports = "reports",
-  ApplicationExceptionsApproval = "application-exceptions/:exceptionId",
   Exceptions = "exceptions",
   OfferingChangeRequests = "institution/offering/change-requests",
   ViewOfferingChangeRequest = "institution/offering-change-request/:offeringId/program/:programId",
@@ -106,4 +102,8 @@ export enum AppRoutes {
   FileUploads = "files",
   Notes = "notes",
   ApplicationView = "view",
+  AssessmentSummary = "assessment-summary",
+  ApplicationException = "application-exceptions/:exceptionId",
+  StudentAppealRequest = "request-changes/:appealId",
+
 }
diff --git a/sources/packages/web/src/views/aest/student/applicationDetails/ApplicationExceptionsApproval.vue b/sources/packages/web/src/views/aest/student/applicationDetails/ApplicationExceptionsApproval.vue
new file mode 100644
index 0000000000..80ad1470e1
--- /dev/null
+++ b/sources/packages/web/src/views/aest/student/applicationDetails/ApplicationExceptionsApproval.vue
@@ -0,0 +1,77 @@
+<template>
+  <application-exceptions-approval
+    :exceptionId="exceptionId"
+    :backRouteLocation="assessmentsSummaryRoute"
+    :processing="processing"
+    @submitted="submitted"
+  />
+</template>
+<script lang="ts">
+import { ref, defineComponent } from "vue";
+import { AESTRoutesConst } from "@/constants/routes/RouteConstants";
+import { useRouter } from "vue-router";
+import { ApplicationExceptionService } from "@/services/ApplicationExceptionService";
+import { FormIOForm } from "@/types";
+import { UpdateApplicationExceptionAPIInDTO } from "@/services/http/dto";
+import { useSnackBar } from "@/composables";
+import ApplicationExceptionsApproval from "@/components/common/students/applicationDetails/ApplicationExceptionsApproval.vue";
+
+export default defineComponent({
+  components: {
+    ApplicationExceptionsApproval,
+  },
+  props: {
+    studentId: {
+      type: Number,
+      required: true,
+    },
+    applicationId: {
+      type: Number,
+      required: true,
+    },
+    exceptionId: {
+      type: Number,
+      required: true,
+    },
+  },
+  setup(props) {
+    const router = useRouter();
+    const snackBar = useSnackBar();
+    const processing = ref(false);
+
+    const assessmentsSummaryRoute = {
+      name: AESTRoutesConst.ASSESSMENTS_SUMMARY,
+      params: {
+        applicationId: props.applicationId,
+        studentId: props.studentId,
+      },
+    };
+    // todo: ann test
+    const submitted = async (form: FormIOForm) => {
+      processing.value = true;
+      try {
+        const approveExceptionPayload =
+          form.data as UpdateApplicationExceptionAPIInDTO;
+        await ApplicationExceptionService.shared.approveException(
+          props.exceptionId,
+          approveExceptionPayload,
+        );
+        snackBar.success(
+          `Application exception status is now ${approveExceptionPayload.exceptionStatus}.`,
+        );
+        router.push(assessmentsSummaryRoute);
+      } catch {
+        snackBar.error("An unexpected error happened during the approval.");
+      } finally {
+        processing.value = false;
+      }
+    };
+
+    return {
+      assessmentsSummaryRoute,
+      submitted,
+      processing,
+    };
+  },
+});
+</script>
diff --git a/sources/packages/web/src/views/aest/student/applicationDetails/StudentAppealRequestsApproval.vue b/sources/packages/web/src/views/aest/student/applicationDetails/StudentAppealRequestsApproval.vue
new file mode 100644
index 0000000000..6452a47f3e
--- /dev/null
+++ b/sources/packages/web/src/views/aest/student/applicationDetails/StudentAppealRequestsApproval.vue
@@ -0,0 +1,75 @@
+<template>
+  <student-appeal-requests-approval
+    :appealId="appealId"
+    :backRouteLocation="assessmentsSummaryRoute"
+    @submitted="submitted"
+  />
+</template>
+
+<script lang="ts">
+import { defineComponent } from "vue";
+import { AESTRoutesConst } from "@/constants/routes/RouteConstants";
+import { useRouter } from "vue-router";
+import { StudentAppealService } from "@/services/StudentAppealService";
+import { useSnackBar } from "@/composables";
+import { StudentAppealApproval, ApiProcessError } from "@/types";
+import { ASSESSMENT_ALREADY_IN_PROGRESS } from "@/services/http/dto/Assessment.dto";
+import StudentAppealRequestsApproval from "@/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue";
+
+export default defineComponent({
+  components: {
+    StudentAppealRequestsApproval,
+  },
+  props: {
+    studentId: {
+      type: Number,
+      required: true,
+    },
+    applicationId: {
+      type: Number,
+      required: true,
+    },
+    appealId: {
+      type: Number,
+      required: true,
+    },
+  },
+  setup(props) {
+    const router = useRouter();
+    const snackBar = useSnackBar();
+
+    const assessmentsSummaryRoute = {
+      name: AESTRoutesConst.ASSESSMENTS_SUMMARY,
+      params: {
+        applicationId: props.applicationId,
+        studentId: props.studentId,
+      },
+    };
+    // todo: ann check snackbar
+    const submitted = async (approvals: StudentAppealApproval[]) => {
+      try {
+        await StudentAppealService.shared.approveStudentAppealRequests(
+          props.appealId,
+          approvals,
+        );
+        snackBar.success("The request was completed with success.");
+
+        router.push(assessmentsSummaryRoute);
+      } catch (error: unknown) {
+        if (error instanceof ApiProcessError) {
+          if (error.errorType === ASSESSMENT_ALREADY_IN_PROGRESS) {
+            snackBar.warn(error.message);
+            return;
+          }
+        }
+        snackBar.error("An unexpected error happened during the approval.");
+      }
+    };
+
+    return {
+      assessmentsSummaryRoute,
+      submitted,
+    };
+  },
+});
+</script>
diff --git a/sources/packages/web/src/views/institution/student/applicationDetails/ApplicationExceptions.vue b/sources/packages/web/src/views/institution/student/applicationDetails/ApplicationExceptions.vue
new file mode 100644
index 0000000000..00b1cf96f0
--- /dev/null
+++ b/sources/packages/web/src/views/institution/student/applicationDetails/ApplicationExceptions.vue
@@ -0,0 +1,46 @@
+<template>
+  <application-exceptions-approval
+    :studentId="studentId"
+    :exceptionId="exceptionId"
+    :backRouteLocation="assessmentsSummaryRoute"
+    :readOnlyForm="true"
+  />
+</template>
+<script lang="ts">
+import { defineComponent } from "vue";
+import { InstitutionRoutesConst } from "@/constants/routes/RouteConstants";
+import ApplicationExceptionsApproval from "@/components/common/students/applicationDetails/ApplicationExceptionsApproval.vue";
+
+export default defineComponent({
+  components: {
+    ApplicationExceptionsApproval,
+  },
+  props: {
+    studentId: {
+      type: Number,
+      required: true,
+    },
+    applicationId: {
+      type: Number,
+      required: true,
+    },
+    exceptionId: {
+      type: Number,
+      required: true,
+    },
+  },
+  setup(props) {
+    const assessmentsSummaryRoute = {
+      name: InstitutionRoutesConst.ASSESSMENTS_SUMMARY,
+      params: {
+        applicationId: props.applicationId,
+        studentId: props.studentId,
+      },
+    };
+
+    return {
+      assessmentsSummaryRoute,
+    };
+  },
+});
+</script>
diff --git a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
index 8eec978324..474a67e42f 100644
--- a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
+++ b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
@@ -4,13 +4,13 @@
       <header-navigator
         title="Student applications"
         :routeLocation="{
-          name: AESTRoutesConst.STUDENT_APPLICATIONS,
+          name: InstitutionRoutesConst.STUDENT_APPLICATIONS,
           params: { studentId },
         }"
         subTitle="Assessment"
       />
     </template>
-    <request-assessment
+    <!-- <request-assessment
       class="mb-5"
       :applicationId="applicationId"
       :studentId="studentId"
@@ -27,19 +27,31 @@
       @viewOfferingRequest="goToOfferingRequest"
       @viewApplicationException="goToApplicationException"
       @viewScholasticStandingChange="goToScholasticStanding"
+    /> -->
+    <request-assessment
+      :applicationId="applicationId"
+      :studentId="studentId"
+      @viewStudentAppeal="goToStudentAppeal"
+      @viewApplicationException="goToApplicationException"
+    />
+    <history-assessment
+      :applicationId="applicationId"
+      :studentId="studentId"
+      :viewRequestTypes="assessmentRequestTypes"
+      @viewStudentAppeal="goToStudentAppeal"
+      @viewApplicationException="goToApplicationException"
     />
   </full-page-container>
 </template>
 
 <script lang="ts">
-import { AESTRoutesConst } from "@/constants/routes/RouteConstants";
+import { InstitutionRoutesConst } from "@/constants/routes/RouteConstants";
 import { useRouter } from "vue-router";
 import { defineComponent } from "vue";
 import { AssessmentTriggerType } from "@/types";
 import RequestAssessment from "@/components/aest/students/assessment/Request.vue";
 import HistoryAssessment from "@/components/aest/students/assessment/History.vue";
-// todo: remove the default value for student and institution id
-// todo: set required: true
+
 export default defineComponent({
   components: {
     RequestAssessment,
@@ -48,13 +60,11 @@ export default defineComponent({
   props: {
     studentId: {
       type: Number,
-      required: false,
-      default: 275,
+      required: true,
     },
     applicationId: {
       type: Number,
-      required: false,
-      default: 1734,
+      required: true,
     },
   },
   setup(props) {
@@ -70,7 +80,7 @@ export default defineComponent({
 
     const goToStudentAppeal = (appealId: number) => {
       router.push({
-        name: AESTRoutesConst.STUDENT_APPEAL_REQUESTS_APPROVAL,
+        name: InstitutionRoutesConst.STUDENT_APPEAL_REQUEST,
         params: {
           studentId: props.studentId,
           applicationId: props.applicationId,
@@ -81,7 +91,7 @@ export default defineComponent({
 
     const goToApplicationException = (exceptionId: number) => {
       router.push({
-        name: AESTRoutesConst.APPLICATION_EXCEPTIONS_APPROVAL,
+        name: InstitutionRoutesConst.APPLICATION_EXCEPTION,
         params: {
           studentId: props.studentId,
           applicationId: props.applicationId,
@@ -90,44 +100,44 @@ export default defineComponent({
       });
     };
 
-    const gotToViewAssessment = (assessmentId: number) => {
-      router.push({
-        name: AESTRoutesConst.ASSESSMENT_AWARD_VIEW,
-        params: {
-          studentId: props.studentId,
-          applicationId: props.applicationId,
-          assessmentId,
-        },
-      });
-    };
+    // const gotToViewAssessment = (assessmentId: number) => {
+    //   router.push({
+    //     name: InstitutionRoutesConst.ASSESSMENT_AWARD_VIEW,
+    //     params: {
+    //       studentId: props.studentId,
+    //       applicationId: props.applicationId,
+    //       assessmentId,
+    //     },
+    //   });
+    // };
 
-    const goToScholasticStanding = (scholasticStandingId: number) => {
-      router.push({
-        name: AESTRoutesConst.SCHOLASTIC_STANDING_VIEW,
-        params: {
-          studentId: props.studentId,
-          applicationId: props.applicationId,
-          scholasticStandingId,
-        },
-      });
-    };
-    const goToOfferingRequest = (offeringId: number, programId: number) => {
-      router.push({
-        name: AESTRoutesConst.OFFERING_CHANGE_REQUEST_VIEW,
-        params: {
-          offeringId,
-          programId,
-        },
-      });
-    };
+    // const goToScholasticStanding = (scholasticStandingId: number) => {
+    //   router.push({
+    //     name: InstitutionRoutesConst.SCHOLASTIC_STANDING_VIEW,
+    //     params: {
+    //       studentId: props.studentId,
+    //       applicationId: props.applicationId,
+    //       scholasticStandingId,
+    //     },
+    //   });
+    // };
+    // const goToOfferingRequest = (offeringId: number, programId: number) => {
+    //   router.push({
+    //     name: InstitutionRoutesConst.OFFERING_CHANGE_REQUEST_VIEW,
+    //     params: {
+    //       offeringId,
+    //       programId,
+    //     },
+    //   });
+    // };
 
     return {
-      AESTRoutesConst,
+      InstitutionRoutesConst,
       goToStudentAppeal,
-      gotToViewAssessment,
+      // gotToViewAssessment,
       goToApplicationException,
-      goToScholasticStanding,
-      goToOfferingRequest,
+      // goToScholasticStanding,
+      // goToOfferingRequest,
       assessmentRequestTypes,
     };
   },
diff --git a/sources/packages/web/src/views/institution/student/applicationDetails/StudentAppealRequest.vue b/sources/packages/web/src/views/institution/student/applicationDetails/StudentAppealRequest.vue
new file mode 100644
index 0000000000..dc2319f925
--- /dev/null
+++ b/sources/packages/web/src/views/institution/student/applicationDetails/StudentAppealRequest.vue
@@ -0,0 +1,44 @@
+<template>
+  <student-appeal-requests-approval
+    :studentId="studentId"
+    :appealId="appealId"
+    :backRouteLocation="assessmentsSummaryRoute"
+    :readOnlyForm="true"
+  />
+</template>
+
+<script lang="ts">
+import { defineComponent } from "vue";
+import { InstitutionRoutesConst } from "@/constants/routes/RouteConstants";
+import StudentAppealRequestsApproval from "@/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue";
+
+export default defineComponent({
+  components: { StudentAppealRequestsApproval },
+  props: {
+    studentId: {
+      type: Number,
+      required: true,
+    },
+    applicationId: {
+      type: Number,
+      required: true,
+    },
+    appealId: {
+      type: Number,
+      required: true,
+    },
+  },
+  setup(props) {
+    const assessmentsSummaryRoute = {
+      name: InstitutionRoutesConst.ASSESSMENTS_SUMMARY,
+      params: {
+        applicationId: props.applicationId,
+        studentId: props.studentId,
+      },
+    };
+    return {
+      assessmentsSummaryRoute,
+    };
+  },
+});
+</script>

From 9fcedb63c3c0e5a9fec29cdfd648ef86175b11ea Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Tue, 9 May 2023 17:57:40 -0400
Subject: [PATCH 03/12] appeals

---
 .../student-appeal.controller.service.ts       | 12 +++++-------
 .../student-appeal.institutions.controller.ts  | 18 ++++--------------
 sources/packages/web/package-lock.json         | 17 +++++++++++------
 sources/packages/web/package.json              |  1 +
 4 files changed, 21 insertions(+), 27 deletions(-)

diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
index f8e8dd9e56..3c6c3aa9cb 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
@@ -1,13 +1,11 @@
-import { NotFoundException } from "@nestjs/common";
-import { StudentAppealService } from "../../services";
-import BaseController from "../BaseController";
+import { Injectable, NotFoundException } from "@nestjs/common";
 import { StudentAppealAPIOutDTO } from "./models/student-appeal.dto";
 import { getUserFullName } from "../../utilities";
+import { StudentAppealService } from "../../services";
 
-export class StudentAppealControllerService extends BaseController {
-  constructor(private readonly studentAppealService: StudentAppealService) {
-    super();
-  }
+@Injectable()
+export class StudentAppealControllerService {
+  constructor(private readonly studentAppealService: StudentAppealService) {}
 
   /**
    * Get the student appeal and its requests.
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
index 77a31e6665..6e520ad666 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
@@ -1,19 +1,9 @@
-import {
-  Controller,
-  Param,
-  Get,
-  ParseIntPipe,
-} from "@nestjs/common";
+import { Controller, Param, Get, ParseIntPipe } from "@nestjs/common";
 import { AuthorizedParties } from "../../auth/authorized-parties.enum";
-import {
-  AllowAuthorizedParty,
-} from "../../auth/decorators";
-import {
-  ApiNotFoundResponse,
-  ApiTags,
-} from "@nestjs/swagger";
+import { AllowAuthorizedParty } from "../../auth/decorators";
+import { ApiNotFoundResponse, ApiTags } from "@nestjs/swagger";
 import BaseController from "../BaseController";
-import {  ClientTypeBaseRoute } from "../../types";
+import { ClientTypeBaseRoute } from "../../types";
 import { StudentAppealAPIOutDTO } from "./models/student-appeal.dto";
 import { StudentAppealControllerService } from "./student-appeal.controller.service";
 
diff --git a/sources/packages/web/package-lock.json b/sources/packages/web/package-lock.json
index 6f36629c71..afa3b2362b 100644
--- a/sources/packages/web/package-lock.json
+++ b/sources/packages/web/package-lock.json
@@ -13,6 +13,7 @@
         "@types/vuelidate": "^0.7.13",
         "@vee-validate/rules": "^4.0.4",
         "axios": "^0.21.2",
+        "caniuse-lite": "^1.0.30001486",
         "core-js": "^3.8.3",
         "dayjs": "^1.10.7",
         "formiojs": "^4.13.4",
@@ -5267,9 +5268,9 @@
       }
     },
     "node_modules/caniuse-lite": {
-      "version": "1.0.30001406",
-      "resolved": "https://registry.npmjs.org/caniuse-lite/-/caniuse-lite-1.0.30001406.tgz",
-      "integrity": "sha512-bWTlaXUy/rq0BBtYShc/jArYfBPjEV95euvZ8JVtO43oQExEN/WquoqpufFjNu4kSpi5cy5kMbNvzztWDfv1Jg==",
+      "version": "1.0.30001486",
+      "resolved": "https://registry.npmjs.org/caniuse-lite/-/caniuse-lite-1.0.30001486.tgz",
+      "integrity": "sha512-uv7/gXuHi10Whlj0pp5q/tsK/32J2QSqVRKQhs2j8VsDCjgyruAh/eEXHF822VqO9yT6iZKw3nRwZRSPBE9OQg==",
       "funding": [
         {
           "type": "opencollective",
@@ -5278,6 +5279,10 @@
         {
           "type": "tidelift",
           "url": "https://tidelift.com/funding/github/npm/caniuse-lite"
+        },
+        {
+          "type": "github",
+          "url": "https://github.com/sponsors/ai"
         }
       ]
     },
@@ -21596,9 +21601,9 @@
       }
     },
     "caniuse-lite": {
-      "version": "1.0.30001406",
-      "resolved": "https://registry.npmjs.org/caniuse-lite/-/caniuse-lite-1.0.30001406.tgz",
-      "integrity": "sha512-bWTlaXUy/rq0BBtYShc/jArYfBPjEV95euvZ8JVtO43oQExEN/WquoqpufFjNu4kSpi5cy5kMbNvzztWDfv1Jg=="
+      "version": "1.0.30001486",
+      "resolved": "https://registry.npmjs.org/caniuse-lite/-/caniuse-lite-1.0.30001486.tgz",
+      "integrity": "sha512-uv7/gXuHi10Whlj0pp5q/tsK/32J2QSqVRKQhs2j8VsDCjgyruAh/eEXHF822VqO9yT6iZKw3nRwZRSPBE9OQg=="
     },
     "case-sensitive-paths-webpack-plugin": {
       "version": "2.4.0",
diff --git a/sources/packages/web/package.json b/sources/packages/web/package.json
index 23e4b542cf..952a8b69ca 100644
--- a/sources/packages/web/package.json
+++ b/sources/packages/web/package.json
@@ -15,6 +15,7 @@
     "@types/vuelidate": "^0.7.13",
     "@vee-validate/rules": "^4.0.4",
     "axios": "^0.21.2",
+    "caniuse-lite": "^1.0.30001486",
     "core-js": "^3.8.3",
     "dayjs": "^1.10.7",
     "formiojs": "^4.13.4",

From cefada3be5d7cef5889fc1bc863811df1c2f874b Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Tue, 9 May 2023 21:09:59 -0400
Subject: [PATCH 04/12] refactor

---
 .../backend/apps/api/src/app.aest.module.ts   |   4 +-
 .../assessment.institutions.controller.ts     |  38 +------
 ...ucation-program.institutions.controller.ts |   2 +-
 .../models/student-appeal.dto.ts              |  10 ++
 .../student-appeal.aest.controller.ts         |   6 +-
 .../student-appeal.controller.service.ts      |  38 +++++--
 .../student-appeal.institutions.controller.ts |   9 +-
 .../student-appeal.students.controller.ts     |  26 +----
 .../ApplicationExceptionsApproval.vue         |  14 ++-
 .../StudentAppealRequestsApproval.vue         | 100 ++++++++++--------
 .../web/src/services/StudentAppealService.ts  |   5 +-
 .../web/src/services/http/StudentAppealApi.ts |  11 +-
 .../services/http/dto/StudentAppeal.dto.ts    |  10 ++
 sources/packages/web/src/types/AppRoutes.ts   |   1 -
 .../ApplicationExceptionsApproval.vue         |   9 +-
 .../StudentAppealRequestsApproval.vue         |   2 +-
 .../InstitutionAssessmentsSummary.vue         |  55 +---------
 17 files changed, 146 insertions(+), 194 deletions(-)

diff --git a/sources/packages/backend/apps/api/src/app.aest.module.ts b/sources/packages/backend/apps/api/src/app.aest.module.ts
index 2a60ab30c7..a3e594099b 100644
--- a/sources/packages/backend/apps/api/src/app.aest.module.ts
+++ b/sources/packages/backend/apps/api/src/app.aest.module.ts
@@ -109,6 +109,8 @@ import {
     OverawardAESTController,
   ],
   providers: [
+    ApplicationExceptionControllerService,
+    StudentAppealControllerService,
     WorkflowClientService,
     InstitutionRestrictionService,
     SupportingUserService,
@@ -162,8 +164,6 @@ import {
     NoteSharedService,
     OverawardControllerService,
     RestrictionSharedService,
-    ApplicationExceptionControllerService,
-    StudentAppealControllerService,
   ],
 })
 export class AppAESTModule {}
diff --git a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
index 3cde305224..f6bc6d7b1d 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
@@ -23,7 +23,7 @@ import { AssessmentControllerService } from "./assessment.controller.service";
 @ApiTags(`${ClientTypeBaseRoute.Institution}-assessment`)
 export class AssessmentInstitutionsController extends BaseController {
   constructor(
-    private readonly assessmentControllerService: AssessmentControllerService, //   private readonly educationProgramOfferingService: EducationProgramOfferingService, //   private readonly applicationExceptionService: ApplicationExceptionService,
+    private readonly assessmentControllerService: AssessmentControllerService,
   ) {
     super();
   }
@@ -67,40 +67,4 @@ export class AssessmentInstitutionsController extends BaseController {
       studentId,
     );
   }
-
-  // /**
-  //  * Get the NOA values for a student application on a particular assessment.
-  //  * @param assessmentId assessment id to get the NOA values.
-  //  * @returns NOA and application data.
-  //  */
-  // @Get(":assessmentId/noa")
-  // @ApiNotFoundResponse({
-  //   description: "Assessment id not found.",
-  // })
-  // @ApiUnprocessableEntityResponse({
-  //   description: "Notice of assessment data is not present.",
-  // })
-  // async getAssessmentNOA(
-  //   @Param("assessmentId", ParseIntPipe) assessmentId: number,
-  // ): Promise<AssessmentNOAAPIOutDTO> {
-  //   return this.assessmentControllerService.getAssessmentNOA(assessmentId);
-  // }
-
-  // /**
-  //  * Get estimated and actual(if present) award details of an assessment.
-  //  * @param assessmentId assessment to which awards details belong to.
-  //  * @returns estimated and actual award details.
-  //  */
-  // @Get(":assessmentId/award")
-  // @ApiNotFoundResponse({
-  //   description: "Assessment not found.",
-  // })
-  // async getAssessmentAwardDetails(
-  //   @Param("assessmentId", ParseIntPipe) assessmentId: number,
-  // ): Promise<AwardDetailsAPIOutDTO> {
-  //   return this.assessmentControllerService.getAssessmentAwardDetails(
-  //     assessmentId,
-  //     true,
-  //   );
-  // }
 }
diff --git a/sources/packages/backend/apps/api/src/route-controllers/education-program/education-program.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/education-program/education-program.institutions.controller.ts
index d88c6f44c0..00417de70f 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/education-program/education-program.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/education-program/education-program.institutions.controller.ts
@@ -53,7 +53,7 @@ export class EducationProgramInstitutionsController extends BaseController {
    * @param paginationOptions pagination options.
    * @returns paginated programs summary.
    */
-  // @HasLocationAccess("locationId")
+  @HasLocationAccess("locationId")
   @Get("location/:locationId/summary")
   async getProgramsSummaryByLocationId(
     @Param("locationId", ParseIntPipe) locationId: number,
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/models/student-appeal.dto.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/models/student-appeal.dto.ts
index 5d2c65e71a..136d40772d 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/models/student-appeal.dto.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/models/student-appeal.dto.ts
@@ -41,6 +41,9 @@ export class StudentAppealRequestAPIOutDTO {
   submittedData: unknown;
   submittedFormName: string;
   appealStatus: StudentAppealStatus;
+}
+
+export class DetailedStudentAppealRequestAPIOutDTO extends StudentAppealRequestAPIOutDTO {
   assessedDate?: Date;
   assessedByUserName?: string;
   noteDescription?: string;
@@ -53,6 +56,13 @@ export class StudentAppealAPIOutDTO {
   appealRequests: StudentAppealRequestAPIOutDTO[];
 }
 
+export class DetailedStudentAppealAPIOutDTO {
+  id: number;
+  submittedDate: Date;
+  status: StudentAppealStatus;
+  appealRequests: DetailedStudentAppealRequestAPIOutDTO[];
+}
+
 export class StudentAppealRequestApprovalAPIInDTO {
   @IsPositive()
   id: number;
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
index 3358d9860a..4da7e1c90c 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
@@ -30,7 +30,7 @@ import BaseController from "../BaseController";
 import { ApiProcessError, ClientTypeBaseRoute } from "../../types";
 import { UserGroups } from "../../auth/user-groups.enum";
 import {
-  StudentAppealAPIOutDTO,
+  DetailedStudentAppealAPIOutDTO,
   StudentAppealApprovalAPIInDTO,
   StudentAppealPendingSummaryAPIOutDTO,
 } from "./models/student-appeal.dto";
@@ -73,9 +73,11 @@ export class StudentAppealAESTController extends BaseController {
   })
   async getStudentAppealWithRequests(
     @Param("appealId", ParseIntPipe) appealId: number,
-  ): Promise<StudentAppealAPIOutDTO> {
+  ): Promise<DetailedStudentAppealAPIOutDTO> {
     return this.studentAppealControllerService.getStudentAppealWithRequest(
       appealId,
+      undefined,
+      {assessDetails: true}
     );
   }
 
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
index 3c6c3aa9cb..3bb0e7c639 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
@@ -1,5 +1,10 @@
 import { Injectable, NotFoundException } from "@nestjs/common";
-import { StudentAppealAPIOutDTO } from "./models/student-appeal.dto";
+import {
+  StudentAppealAPIOutDTO,
+  DetailedStudentAppealAPIOutDTO,
+  StudentAppealRequestAPIOutDTO,
+  DetailedStudentAppealRequestAPIOutDTO,
+} from "./models/student-appeal.dto";
 import { getUserFullName } from "../../utilities";
 import { StudentAppealService } from "../../services";
 
@@ -16,7 +21,10 @@ export class StudentAppealControllerService {
   async getStudentAppealWithRequest(
     appealId: number,
     studentId?: number,
-  ): Promise<StudentAppealAPIOutDTO> {
+    options?: {
+      assessDetails?: false;
+    },
+  ): Promise<StudentAppealAPIOutDTO | DetailedStudentAppealAPIOutDTO> {
     const studentAppeal =
       await this.studentAppealService.getAppealAndRequestsById(
         appealId,
@@ -30,15 +38,23 @@ export class StudentAppealControllerService {
       id: studentAppeal.id,
       submittedDate: studentAppeal.submittedDate,
       status: studentAppeal.status,
-      appealRequests: studentAppeal.appealRequests.map((appealRequest) => ({
-        id: appealRequest.id,
-        appealStatus: appealRequest.appealStatus,
-        submittedData: appealRequest.submittedData,
-        submittedFormName: appealRequest.submittedFormName,
-        assessedDate: appealRequest.assessedDate,
-        noteDescription: appealRequest.note?.description,
-        assessedByUserName: getUserFullName(appealRequest.assessedBy),
-      })),
+      appealRequests: studentAppeal.appealRequests.map((appealRequest) => {
+        const requests: DetailedStudentAppealRequestAPIOutDTO
+           = {
+          id: appealRequest.id,
+          appealStatus: appealRequest.appealStatus,
+          submittedData: appealRequest.submittedData,
+          submittedFormName: appealRequest.submittedFormName,
+        };
+        if (options?.assessDetails) {
+          requests.assessedDate = appealRequest.assessedDate;
+          requests.noteDescription = appealRequest.note?.description;
+          requests.assessedByUserName = getUserFullName(
+            appealRequest.assessedBy,
+          );
+        }
+        return requests;
+      }),
     };
   }
 }
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
index 6e520ad666..a3f19d4208 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
@@ -4,7 +4,7 @@ import { AllowAuthorizedParty } from "../../auth/decorators";
 import { ApiNotFoundResponse, ApiTags } from "@nestjs/swagger";
 import BaseController from "../BaseController";
 import { ClientTypeBaseRoute } from "../../types";
-import { StudentAppealAPIOutDTO } from "./models/student-appeal.dto";
+import { DetailedStudentAppealAPIOutDTO } from "./models/student-appeal.dto";
 import { StudentAppealControllerService } from "./student-appeal.controller.service";
 
 /**
@@ -21,10 +21,10 @@ export class StudentAppealInstitutionsController extends BaseController {
   }
 
   /**
-   * Get the student appeal and its requests.
+   * Get the student appeal details.
    * @param appealId appeal id to be retrieved.
    * @param studentId student id.
-   * @returns the student appeal and its requests.
+   * @returns the student appeal details.
    */
   @Get("student/:studentId/appeal/:appealId/requests")
   @ApiNotFoundResponse({
@@ -33,10 +33,11 @@ export class StudentAppealInstitutionsController extends BaseController {
   async getStudentAppealWithRequest(
     @Param("studentId", ParseIntPipe) studentId: number,
     @Param("appealId", ParseIntPipe) appealId: number,
-  ): Promise<StudentAppealAPIOutDTO> {
+  ): Promise<DetailedStudentAppealAPIOutDTO> {
     return this.studentAppealControllerService.getStudentAppealWithRequest(
       appealId,
       studentId,
+      {assessDetails: true}
     );
   }
 }
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
index 0ce07a948e..366ce67b61 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
@@ -54,6 +54,7 @@ export class StudentAppealStudentsController extends BaseController {
     private readonly studentAppealService: StudentAppealService,
     private readonly applicationService: ApplicationService,
     private readonly formService: FormService,
+    private readonly studentAppealControllerService: StudentAppealControllerService,
   ) {
     super();
   }
@@ -155,7 +156,6 @@ export class StudentAppealStudentsController extends BaseController {
   /**
    * Get the student appeal and its requests.
    * @param appealId appeal id to be retrieved.
-   * todo: ann controller service already created, dto is different work on it
    * @returns the student appeal and its requests.
    */
   @Get(":appealId/requests")
@@ -166,25 +166,9 @@ export class StudentAppealStudentsController extends BaseController {
     @Param("appealId", ParseIntPipe) appealId: number,
     @UserToken() userToken: StudentUserToken,
   ): Promise<StudentAppealAPIOutDTO> {
-    const studentAppeal =
-      await this.studentAppealService.getAppealAndRequestsById(
-        appealId,
-        userToken.studentId,
-      );
-    if (!studentAppeal) {
-      throw new NotFoundException("Not able to find the student appeal.");
-    }
-
-    return {
-      id: studentAppeal.id,
-      submittedDate: studentAppeal.submittedDate,
-      status: studentAppeal.status,
-      appealRequests: studentAppeal.appealRequests.map((appealRequest) => ({
-        id: appealRequest.id,
-        appealStatus: appealRequest.appealStatus,
-        submittedData: appealRequest.submittedData,
-        submittedFormName: appealRequest.submittedFormName,
-      })),
-    };
+    return this.studentAppealControllerService.getStudentAppealWithRequest(
+      appealId,
+      userToken.studentId,
+    );
   }
 }
diff --git a/sources/packages/web/src/components/common/students/applicationDetails/ApplicationExceptionsApproval.vue b/sources/packages/web/src/components/common/students/applicationDetails/ApplicationExceptionsApproval.vue
index ace39cb9d1..3e93346fca 100644
--- a/sources/packages/web/src/components/common/students/applicationDetails/ApplicationExceptionsApproval.vue
+++ b/sources/packages/web/src/components/common/students/applicationDetails/ApplicationExceptionsApproval.vue
@@ -36,8 +36,11 @@
 import { ref, onMounted, defineComponent, PropType } from "vue";
 import { RouteLocationRaw, useRouter } from "vue-router";
 import { ApplicationExceptionService } from "@/services/ApplicationExceptionService";
-import { ApplicationExceptionStatus, Role } from "@/types";
-import { ApplicationExceptionAPIOutDTO } from "@/services/http/dto";
+import { ApplicationExceptionStatus, FormIOForm, Role } from "@/types";
+import {
+  ApplicationExceptionAPIOutDTO,
+  UpdateApplicationExceptionAPIInDTO,
+} from "@/services/http/dto";
 import { useAssessment, useFormatters } from "@/composables";
 import HeaderTitleValue from "@/components/generic/HeaderTitleValue.vue";
 import CheckPermissionRole from "@/components/generic/CheckPermissionRole.vue";
@@ -74,8 +77,11 @@ type ApplicationExceptionFormModel = Omit<
 };
 
 export default defineComponent({
-  // todo: ann emit new
-  emits: ["submitted"],
+  emits: {
+    submitted: (form: FormIOForm<UpdateApplicationExceptionAPIInDTO>) => {
+      return !!form;
+    },
+  },
   components: { HeaderTitleValue, CheckPermissionRole },
   props: {
     studentId: {
diff --git a/sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue b/sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue
index 27a61a7689..f452c1fedb 100644
--- a/sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue
+++ b/sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue
@@ -1,66 +1,72 @@
 <template>
-  <!-- todo: why container?? -->
-  <v-container>
-    <div class="mb-4">
+  <full-page-container>
+    <template #header>
       <header-navigator
         title="Assessment"
         subTitle="View Request"
         :routeLocation="backRouteLocation"
       />
-    </div>
-    <full-page-container>
-      <body-header title="Student change">
-        <template #status-chip>
-          <status-chip-requested-assessment
-            :status="appealStatus"
-          ></status-chip-requested-assessment>
-        </template>
-      </body-header>
-      <appeal-requests-approval-form
-        :studentAppealRequests="studentAppealRequests"
-        :readOnly="readOnly"
-        @submitted="$emit('submitted', $event)"
-      >
-        <template #approval-actions="{ submit }" v-if="!readOnly">
-          <v-row justify="center" class="m-2">
-            <v-btn
-              color="primary"
-              variant="outlined"
-              class="mr-2"
-              data-cy="cancelApprovalRequestButton"
-              @click="gotToAssessmentsSummary"
-              >Cancel</v-btn
-            >
-            <check-permission-role :role="Role.StudentApproveDeclineAppeals">
-              <template #="{ notAllowed }">
-                <v-btn
-                  color="primary"
-                  class="ml-2"
-                  data-cy="completeStudentRequest"
-                  @click="submit"
-                  :disabled="notAllowed"
-                  >Complete student request
-                </v-btn>
-              </template>
-            </check-permission-role>
-          </v-row>
-        </template>
-      </appeal-requests-approval-form>
-    </full-page-container>
-  </v-container>
+    </template>
+    <body-header title="Student change">
+      <template #status-chip>
+        <status-chip-requested-assessment
+          :status="appealStatus"
+        ></status-chip-requested-assessment>
+      </template>
+    </body-header>
+    <appeal-requests-approval-form
+      :studentAppealRequests="studentAppealRequests"
+      :readOnly="readOnly"
+      @submitted="$emit('submitted', $event)"
+    >
+      <template #approval-actions="{ submit }" v-if="!readOnly">
+        <v-row justify="center" class="m-2">
+          <v-btn
+            color="primary"
+            variant="outlined"
+            class="mr-2"
+            data-cy="cancelApprovalRequestButton"
+            @click="gotToAssessmentsSummary"
+            >Cancel</v-btn
+          >
+          <check-permission-role :role="Role.StudentApproveDeclineAppeals">
+            <template #="{ notAllowed }">
+              <v-btn
+                color="primary"
+                class="ml-2"
+                data-cy="completeStudentRequest"
+                @click="submit"
+                :disabled="notAllowed"
+                >Complete student request
+              </v-btn>
+            </template>
+          </check-permission-role>
+        </v-row>
+      </template>
+    </appeal-requests-approval-form>
+  </full-page-container>
 </template>
 <script lang="ts">
 import { ref, onMounted, computed, defineComponent, PropType } from "vue";
 import { RouteLocationRaw, useRouter } from "vue-router";
 import { StudentAppealService } from "@/services/StudentAppealService";
 import { useFormatters } from "@/composables";
-import { StudentAppealRequest, StudentAppealStatus, Role } from "@/types";
+import {
+  StudentAppealRequest,
+  StudentAppealStatus,
+  Role,
+  StudentAppealApproval,
+} from "@/types";
 import AppealRequestsApprovalForm from "@/components/aest/AppealRequestsApprovalForm.vue";
 import StatusChipRequestedAssessment from "@/components/generic/StatusChipRequestedAssessment.vue";
 import CheckPermissionRole from "@/components/generic/CheckPermissionRole.vue";
 
 export default defineComponent({
-  emit: ["submitted"],
+  emits: {
+    submitted: (approvals: StudentAppealApproval[]) => {
+      return !!approvals.length;
+    },
+  },
   components: {
     AppealRequestsApprovalForm,
     StatusChipRequestedAssessment,
@@ -69,7 +75,7 @@ export default defineComponent({
   props: {
     studentId: {
       type: Number,
-      required: true,
+      required: false,
     },
     appealId: {
       type: Number,
diff --git a/sources/packages/web/src/services/StudentAppealService.ts b/sources/packages/web/src/services/StudentAppealService.ts
index 19682d0e02..9a37e8b5a5 100644
--- a/sources/packages/web/src/services/StudentAppealService.ts
+++ b/sources/packages/web/src/services/StudentAppealService.ts
@@ -45,7 +45,10 @@ export class StudentAppealService {
     appealId: number,
     studentId?: number,
   ): Promise<StudentAppealAPIOutDTO> {
-    return ApiClient.StudentAppealApi.getStudentAppealWithRequests(appealId, studentId);
+    return ApiClient.StudentAppealApi.getStudentAppealWithRequests(
+      appealId,
+      studentId,
+    );
   }
 
   async approveStudentAppealRequests(
diff --git a/sources/packages/web/src/services/http/StudentAppealApi.ts b/sources/packages/web/src/services/http/StudentAppealApi.ts
index d10ea105da..d92b6cc8a4 100644
--- a/sources/packages/web/src/services/http/StudentAppealApi.ts
+++ b/sources/packages/web/src/services/http/StudentAppealApi.ts
@@ -8,6 +8,7 @@ import {
   StudentAppealApprovalAPIInDTO,
   StudentAppealPendingSummaryAPIOutDTO,
   StudentAppealRequestApprovalAPIInDTO,
+  DetailedStudentAppealAPIOutDTO,
 } from "./dto/StudentAppeal.dto";
 
 /**
@@ -33,11 +34,11 @@ export class StudentAppealApi extends HttpBaseClient {
   async getStudentAppealWithRequests(
     appealId: number,
     studentId?: number,
-  ): Promise<StudentAppealAPIOutDTO> {
-    const endpoint = studentId ? `appeal/student/${studentId}/appeal/${appealId}/requests` : `appeal/${appealId}/requests`;
-    return this.getCall<StudentAppealAPIOutDTO>(
-      this.addClientRoot(endpoint),
-    );
+  ): Promise<StudentAppealAPIOutDTO | DetailedStudentAppealAPIOutDTO> {
+    const endpoint = studentId
+      ? `appeal/student/${studentId}/appeal/${appealId}/requests`
+      : `appeal/${appealId}/requests`;
+    return this.getCall<StudentAppealAPIOutDTO|DetailedStudentAppealAPIOutDTO>(this.addClientRoot(endpoint));
   }
 
   async approveStudentAppealRequests(
diff --git a/sources/packages/web/src/services/http/dto/StudentAppeal.dto.ts b/sources/packages/web/src/services/http/dto/StudentAppeal.dto.ts
index 6fc4e193ba..c9067c1ec7 100644
--- a/sources/packages/web/src/services/http/dto/StudentAppeal.dto.ts
+++ b/sources/packages/web/src/services/http/dto/StudentAppeal.dto.ts
@@ -14,6 +14,9 @@ export interface StudentAppealRequestAPIOutDTO {
   submittedData: unknown;
   submittedFormName: string;
   appealStatus: StudentAppealStatus;
+}
+
+export interface DetailedStudentAppealRequestAPIOutDTO extends StudentAppealRequestAPIOutDTO {
   assessedDate?: Date;
   assessedByUserName?: string;
   noteDescription?: string;
@@ -26,6 +29,13 @@ export interface StudentAppealAPIOutDTO {
   appealRequests: StudentAppealRequestAPIOutDTO[];
 }
 
+export interface DetailedStudentAppealAPIOutDTO {
+  id: number;
+  submittedDate: Date;
+  status: StudentAppealStatus;
+  appealRequests: DetailedStudentAppealRequestAPIOutDTO[];
+}
+
 export interface StudentAppealRequestApprovalAPIInDTO {
   id: number;
   appealStatus: StudentAppealStatus;
diff --git a/sources/packages/web/src/types/AppRoutes.ts b/sources/packages/web/src/types/AppRoutes.ts
index ef71775a29..d184d53975 100644
--- a/sources/packages/web/src/types/AppRoutes.ts
+++ b/sources/packages/web/src/types/AppRoutes.ts
@@ -105,5 +105,4 @@ export enum AppRoutes {
   AssessmentSummary = "assessment-summary",
   ApplicationException = "application-exceptions/:exceptionId",
   StudentAppealRequest = "request-changes/:appealId",
-
 }
diff --git a/sources/packages/web/src/views/aest/student/applicationDetails/ApplicationExceptionsApproval.vue b/sources/packages/web/src/views/aest/student/applicationDetails/ApplicationExceptionsApproval.vue
index 80ad1470e1..baa6c45e59 100644
--- a/sources/packages/web/src/views/aest/student/applicationDetails/ApplicationExceptionsApproval.vue
+++ b/sources/packages/web/src/views/aest/student/applicationDetails/ApplicationExceptionsApproval.vue
@@ -46,12 +46,13 @@ export default defineComponent({
         studentId: props.studentId,
       },
     };
-    // todo: ann test
-    const submitted = async (form: FormIOForm) => {
+
+    const submitted = async (
+      form: FormIOForm<UpdateApplicationExceptionAPIInDTO>,
+    ) => {
       processing.value = true;
       try {
-        const approveExceptionPayload =
-          form.data as UpdateApplicationExceptionAPIInDTO;
+        const approveExceptionPayload = form.data;
         await ApplicationExceptionService.shared.approveException(
           props.exceptionId,
           approveExceptionPayload,
diff --git a/sources/packages/web/src/views/aest/student/applicationDetails/StudentAppealRequestsApproval.vue b/sources/packages/web/src/views/aest/student/applicationDetails/StudentAppealRequestsApproval.vue
index 6452a47f3e..023de44b5b 100644
--- a/sources/packages/web/src/views/aest/student/applicationDetails/StudentAppealRequestsApproval.vue
+++ b/sources/packages/web/src/views/aest/student/applicationDetails/StudentAppealRequestsApproval.vue
@@ -45,7 +45,7 @@ export default defineComponent({
         studentId: props.studentId,
       },
     };
-    // todo: ann check snackbar
+
     const submitted = async (approvals: StudentAppealApproval[]) => {
       try {
         await StudentAppealService.shared.approveStudentAppealRequests(
diff --git a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
index 474a67e42f..c3e5126d25 100644
--- a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
+++ b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
@@ -10,30 +10,14 @@
         subTitle="Assessment"
       />
     </template>
-    <!-- <request-assessment
-      class="mb-5"
-      :applicationId="applicationId"
-      :studentId="studentId"
-      @viewStudentAppeal="goToStudentAppeal"
-      @viewApplicationException="goToApplicationException"
-      @viewOfferingRequest="goToOfferingRequest"
-    />
-    <history-assessment
-      :applicationId="applicationId"
-      :studentId="studentId"
-      :viewRequestTypes="assessmentRequestTypes"
-      @viewStudentAppeal="goToStudentAppeal"
-      @viewAssessment="gotToViewAssessment"
-      @viewOfferingRequest="goToOfferingRequest"
-      @viewApplicationException="goToApplicationException"
-      @viewScholasticStandingChange="goToScholasticStanding"
-    /> -->
     <request-assessment
+      class="mb-5"
       :applicationId="applicationId"
       :studentId="studentId"
       @viewStudentAppeal="goToStudentAppeal"
       @viewApplicationException="goToApplicationException"
     />
+
     <history-assessment
       :applicationId="applicationId"
       :studentId="studentId"
@@ -68,7 +52,6 @@ export default defineComponent({
     },
   },
   setup(props) {
-    // todo: check with jason if we need all form and view assessment for all types.
     const router = useRouter();
     // The assessment trigger types for which the request form must be visible by default.
     const assessmentRequestTypes = [
@@ -100,44 +83,10 @@ export default defineComponent({
       });
     };
 
-    // const gotToViewAssessment = (assessmentId: number) => {
-    //   router.push({
-    //     name: InstitutionRoutesConst.ASSESSMENT_AWARD_VIEW,
-    //     params: {
-    //       studentId: props.studentId,
-    //       applicationId: props.applicationId,
-    //       assessmentId,
-    //     },
-    //   });
-    // };
-
-    // const goToScholasticStanding = (scholasticStandingId: number) => {
-    //   router.push({
-    //     name: InstitutionRoutesConst.SCHOLASTIC_STANDING_VIEW,
-    //     params: {
-    //       studentId: props.studentId,
-    //       applicationId: props.applicationId,
-    //       scholasticStandingId,
-    //     },
-    //   });
-    // };
-    // const goToOfferingRequest = (offeringId: number, programId: number) => {
-    //   router.push({
-    //     name: InstitutionRoutesConst.OFFERING_CHANGE_REQUEST_VIEW,
-    //     params: {
-    //       offeringId,
-    //       programId,
-    //     },
-    //   });
-    // };
-
     return {
       InstitutionRoutesConst,
       goToStudentAppeal,
-      // gotToViewAssessment,
       goToApplicationException,
-      // goToScholasticStanding,
-      // goToOfferingRequest,
       assessmentRequestTypes,
     };
   },

From 7343f9351371d5af701c292a42185e354408441c Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Wed, 10 May 2023 17:17:45 -0400
Subject: [PATCH 05/12] refactor

---
 .../apps/api/src/app.students.module.ts        |  2 ++
 ...cation-exception.institutions.controller.ts |  8 +++++++-
 .../assessment/assessment.aest.controller.ts   |  8 --------
 .../student-appeal.aest.controller.ts          |  2 +-
 .../student-appeal.controller.service.ts       | 18 +++++++++---------
 .../student-appeal.institutions.controller.ts  | 10 ++++++++--
 .../student-appeal.students.controller.ts      |  1 +
 .../web/src/services/StudentAppealService.ts   |  4 ++--
 .../web/src/services/http/StudentAppealApi.ts  |  4 +++-
 9 files changed, 33 insertions(+), 24 deletions(-)

diff --git a/sources/packages/backend/apps/api/src/app.students.module.ts b/sources/packages/backend/apps/api/src/app.students.module.ts
index 373da3d73f..977da25824 100644
--- a/sources/packages/backend/apps/api/src/app.students.module.ts
+++ b/sources/packages/backend/apps/api/src/app.students.module.ts
@@ -40,6 +40,7 @@ import {
   ApplicationControllerService,
   OverawardStudentsController,
   OverawardControllerService,
+  StudentAppealControllerService,
 } from "./route-controllers";
 import { AuthModule } from "./auth/auth.module";
 import { ConfigModule } from "@sims/utilities/config";
@@ -112,6 +113,7 @@ import { ATBCIntegrationModule } from "@sims/integrations/atbc-integration";
     DisbursementOverawardService,
     NoteSharedService,
     OverawardControllerService,
+    StudentAppealControllerService,
   ],
 })
 export class AppStudentsModule {}
diff --git a/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
index e9e7803db2..6fcf7c69bc 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
@@ -1,5 +1,9 @@
 import { Controller, Get, Param, ParseIntPipe } from "@nestjs/common";
-import { AllowAuthorizedParty } from "../../auth/decorators";
+import { 
+  AllowAuthorizedParty,
+  IsBCPublicInstitution,
+  HasStudentDataAccess, 
+} from "../../auth/decorators";
 import { AuthorizedParties } from "../../auth/authorized-parties.enum";
 import { ClientTypeBaseRoute } from "../../types";
 import { ApiNotFoundResponse, ApiTags } from "@nestjs/swagger";
@@ -12,6 +16,7 @@ import { ApplicationExceptionControllerService } from "./application-exception.c
  */
 @AllowAuthorizedParty(AuthorizedParties.institution)
 @Controller("application-exception")
+@IsBCPublicInstitution()
 @ApiTags(`${ClientTypeBaseRoute.Institution}-application-exception`)
 export class ApplicationExceptionInstitutionsController extends BaseController {
   constructor(
@@ -26,6 +31,7 @@ export class ApplicationExceptionInstitutionsController extends BaseController {
    * @param exceptionId exception to be retrieved.
    * @returns student application exception information.
    */
+  @HasStudentDataAccess("studentId")
   @Get("student/:studentId/exception/:exceptionId")
   @ApiNotFoundResponse({
     description: "Student application exception not found.",
diff --git a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.aest.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.aest.controller.ts
index e85abaf01c..d35bc5ef62 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.aest.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.aest.controller.ts
@@ -9,7 +9,6 @@ import {
   AssessmentNOAAPIOutDTO,
   RequestAssessmentSummaryAPIOutDTO,
   AwardDetailsAPIOutDTO,
-  RequestAssessmentTypeAPIOutDTO,
 } from "./models/assessment.dto";
 import {
   ApiNotFoundResponse,
@@ -17,11 +16,6 @@ import {
   ApiUnprocessableEntityResponse,
 } from "@nestjs/swagger";
 import { AssessmentControllerService } from "./assessment.controller.service";
-import {
-  EducationProgramOfferingService,
-  ApplicationExceptionService,
-} from "../../services";
-import { ApplicationExceptionStatus } from "@sims/sims-db";
 
 @AllowAuthorizedParty(AuthorizedParties.aest)
 @Groups(UserGroups.AESTUser)
@@ -30,8 +24,6 @@ import { ApplicationExceptionStatus } from "@sims/sims-db";
 export class AssessmentAESTController extends BaseController {
   constructor(
     private readonly assessmentControllerService: AssessmentControllerService,
-    private readonly educationProgramOfferingService: EducationProgramOfferingService,
-    private readonly applicationExceptionService: ApplicationExceptionService,
   ) {
     super();
   }
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
index 4da7e1c90c..332fdf5550 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
@@ -77,7 +77,7 @@ export class StudentAppealAESTController extends BaseController {
     return this.studentAppealControllerService.getStudentAppealWithRequest(
       appealId,
       undefined,
-      {assessDetails: true}
+      { assessDetails: true },
     );
   }
 
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
index 3bb0e7c639..0127773177 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
@@ -22,7 +22,7 @@ export class StudentAppealControllerService {
     appealId: number,
     studentId?: number,
     options?: {
-      assessDetails?: false;
+      assessDetails?: boolean;
     },
   ): Promise<StudentAppealAPIOutDTO | DetailedStudentAppealAPIOutDTO> {
     const studentAppeal =
@@ -39,19 +39,19 @@ export class StudentAppealControllerService {
       submittedDate: studentAppeal.submittedDate,
       status: studentAppeal.status,
       appealRequests: studentAppeal.appealRequests.map((appealRequest) => {
-        const requests: DetailedStudentAppealRequestAPIOutDTO
-           = {
+        let requests = {
           id: appealRequest.id,
           appealStatus: appealRequest.appealStatus,
           submittedData: appealRequest.submittedData,
           submittedFormName: appealRequest.submittedFormName,
-        };
+        } as StudentAppealRequestAPIOutDTO;
         if (options?.assessDetails) {
-          requests.assessedDate = appealRequest.assessedDate;
-          requests.noteDescription = appealRequest.note?.description;
-          requests.assessedByUserName = getUserFullName(
-            appealRequest.assessedBy,
-          );
+          requests = {
+            ...requests,
+            assessedDate: appealRequest.assessedDate,
+            noteDescription: appealRequest.note?.description,
+            assessedByUserName: getUserFullName(appealRequest.assessedBy),
+          } as DetailedStudentAppealRequestAPIOutDTO;
         }
         return requests;
       }),
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
index a3f19d4208..5e8cb217d6 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
@@ -1,6 +1,10 @@
 import { Controller, Param, Get, ParseIntPipe } from "@nestjs/common";
 import { AuthorizedParties } from "../../auth/authorized-parties.enum";
-import { AllowAuthorizedParty } from "../../auth/decorators";
+import { 
+  AllowAuthorizedParty,
+  IsBCPublicInstitution,
+  HasStudentDataAccess, 
+} from "../../auth/decorators";
 import { ApiNotFoundResponse, ApiTags } from "@nestjs/swagger";
 import BaseController from "../BaseController";
 import { ClientTypeBaseRoute } from "../../types";
@@ -11,6 +15,7 @@ import { StudentAppealControllerService } from "./student-appeal.controller.serv
  * Student appeal controller for institutions.
  */
 @AllowAuthorizedParty(AuthorizedParties.institution)
+@IsBCPublicInstitution()
 @Controller("appeal")
 @ApiTags(`${ClientTypeBaseRoute.Institution}-appeal`)
 export class StudentAppealInstitutionsController extends BaseController {
@@ -26,6 +31,7 @@ export class StudentAppealInstitutionsController extends BaseController {
    * @param studentId student id.
    * @returns the student appeal details.
    */
+  @HasStudentDataAccess("studentId")
   @Get("student/:studentId/appeal/:appealId/requests")
   @ApiNotFoundResponse({
     description: "Not able to find the student appeal.",
@@ -37,7 +43,7 @@ export class StudentAppealInstitutionsController extends BaseController {
     return this.studentAppealControllerService.getStudentAppealWithRequest(
       appealId,
       studentId,
-      {assessDetails: true}
+      { assessDetails: true },
     );
   }
 }
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
index 366ce67b61..a8c9b33e0b 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
@@ -44,6 +44,7 @@ import {
   INVALID_APPLICATION_NUMBER,
 } from "../../constants";
 import { StudentAppealRequestModel } from "../../services/student-appeal/student-appeal.model";
+import { StudentAppealControllerService } from "./student-appeal.controller.service";
 
 @AllowAuthorizedParty(AuthorizedParties.student)
 @RequiresStudentAccount()
diff --git a/sources/packages/web/src/services/StudentAppealService.ts b/sources/packages/web/src/services/StudentAppealService.ts
index 9a37e8b5a5..458b3f2515 100644
--- a/sources/packages/web/src/services/StudentAppealService.ts
+++ b/sources/packages/web/src/services/StudentAppealService.ts
@@ -1,7 +1,7 @@
 import ApiClient from "@/services/http/ApiClient";
 import { PaginationOptions, StudentAppealRequest } from "@/types";
 import {
-  StudentAppealAPIOutDTO,
+  DetailedStudentAppealAPIOutDTO,
   StudentAppealPendingSummaryAPIOutDTO,
   StudentAppealRequestAPIInDTO,
   StudentAppealRequestApprovalAPIInDTO,
@@ -44,7 +44,7 @@ export class StudentAppealService {
   async getStudentAppealWithRequests(
     appealId: number,
     studentId?: number,
-  ): Promise<StudentAppealAPIOutDTO> {
+  ): Promise<DetailedStudentAppealAPIOutDTO> {
     return ApiClient.StudentAppealApi.getStudentAppealWithRequests(
       appealId,
       studentId,
diff --git a/sources/packages/web/src/services/http/StudentAppealApi.ts b/sources/packages/web/src/services/http/StudentAppealApi.ts
index d92b6cc8a4..60e37397eb 100644
--- a/sources/packages/web/src/services/http/StudentAppealApi.ts
+++ b/sources/packages/web/src/services/http/StudentAppealApi.ts
@@ -38,7 +38,9 @@ export class StudentAppealApi extends HttpBaseClient {
     const endpoint = studentId
       ? `appeal/student/${studentId}/appeal/${appealId}/requests`
       : `appeal/${appealId}/requests`;
-    return this.getCall<StudentAppealAPIOutDTO|DetailedStudentAppealAPIOutDTO>(this.addClientRoot(endpoint));
+    return this.getCall<
+      StudentAppealAPIOutDTO | DetailedStudentAppealAPIOutDTO
+    >(this.addClientRoot(endpoint));
   }
 
   async approveStudentAppealRequests(

From 9294c23ed87600aecfb4b1c104a740198d77228e Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Wed, 10 May 2023 17:34:34 -0400
Subject: [PATCH 06/12] refactor

---
 .../assessment/assessment.institutions.controller.ts             | 1 +
 sources/packages/web/package-lock.json                           | 1 -
 sources/packages/web/package.json                                | 1 -
 3 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
index f6bc6d7b1d..d658938d43 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
@@ -57,6 +57,7 @@ export class AssessmentInstitutionsController extends BaseController {
    * @param applicationId, application id.
    * @returns summary of the assessment history for a student application.
    */
+  @HasStudentDataAccess("studentId")
   @Get("student/:studentId/application/:applicationId/history")
   async getAssessmentHistorySummary(
     @Param("studentId", ParseIntPipe) studentId: number,
diff --git a/sources/packages/web/package-lock.json b/sources/packages/web/package-lock.json
index afa3b2362b..1e93e34aaf 100644
--- a/sources/packages/web/package-lock.json
+++ b/sources/packages/web/package-lock.json
@@ -13,7 +13,6 @@
         "@types/vuelidate": "^0.7.13",
         "@vee-validate/rules": "^4.0.4",
         "axios": "^0.21.2",
-        "caniuse-lite": "^1.0.30001486",
         "core-js": "^3.8.3",
         "dayjs": "^1.10.7",
         "formiojs": "^4.13.4",
diff --git a/sources/packages/web/package.json b/sources/packages/web/package.json
index 952a8b69ca..23e4b542cf 100644
--- a/sources/packages/web/package.json
+++ b/sources/packages/web/package.json
@@ -15,7 +15,6 @@
     "@types/vuelidate": "^0.7.13",
     "@vee-validate/rules": "^4.0.4",
     "axios": "^0.21.2",
-    "caniuse-lite": "^1.0.30001486",
     "core-js": "^3.8.3",
     "dayjs": "^1.10.7",
     "formiojs": "^4.13.4",

From 5efa7bc53b18d8057c3f903e95c724ee207682d4 Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Wed, 10 May 2023 19:05:26 -0400
Subject: [PATCH 07/12] moved assessment summary to application details folder

---
 sources/packages/web/src/router/AESTRoutes.ts                   | 2 +-
 .../{ => student/applicationDetails}/AssessmentsSummary.vue     | 0
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename sources/packages/web/src/views/aest/{ => student/applicationDetails}/AssessmentsSummary.vue (100%)

diff --git a/sources/packages/web/src/router/AESTRoutes.ts b/sources/packages/web/src/router/AESTRoutes.ts
index 2758d1630e..19fa072029 100644
--- a/sources/packages/web/src/router/AESTRoutes.ts
+++ b/sources/packages/web/src/router/AESTRoutes.ts
@@ -40,7 +40,7 @@ import AESTApplicationSideBar from "@/components/layouts/aest/AESTApplicationSid
 import SupportingUser from "@/views/aest/SupportingUser.vue";
 import PendingDesignations from "@/views/aest/institution/PendingDesignation.vue";
 import DesignationAESTView from "@/views/aest/institution/DesignationAESTView.vue";
-import AssessmentsSummary from "@/views/aest/AssessmentsSummary.vue";
+import AssessmentsSummary from "@/views/aest/student/applicationDetails/AssessmentsSummary.vue";
 import StudentAppealRequestsApproval from "@/views/aest/student/applicationDetails/StudentAppealRequestsApproval.vue";
 import NoticeOfAssessment from "@/views/aest/NoticeOfAssessment.vue";
 import ApplicationExceptionsApproval from "@/views/aest/student/applicationDetails/ApplicationExceptionsApproval.vue";
diff --git a/sources/packages/web/src/views/aest/AssessmentsSummary.vue b/sources/packages/web/src/views/aest/student/applicationDetails/AssessmentsSummary.vue
similarity index 100%
rename from sources/packages/web/src/views/aest/AssessmentsSummary.vue
rename to sources/packages/web/src/views/aest/student/applicationDetails/AssessmentsSummary.vue

From 42bbc9a63c63f4cda53248afe4fbe63bdf6bf668 Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Fri, 12 May 2023 16:10:03 -0400
Subject: [PATCH 08/12] feedbacj

---
 ...ation-exception.institutions.controller.ts |  2 +-
 .../assessment.institutions.controller.ts     |  7 ++--
 .../apps/api/src/route-controllers/index.ts   | 33 ++++++++++---------
 .../models/student-appeal.dto.ts              | 11 ++-----
 .../student-appeal.aest.controller.ts         |  8 ++---
 .../student-appeal.controller.service.ts      | 28 ++++++++--------
 .../student-appeal.institutions.controller.ts | 22 +++++++------
 .../student-appeal.students.controller.ts     |  7 ++--
 .../application-exception.service.ts          |  2 +-
 .../InstitutionAssessmentsSummary.vue         |  1 -
 10 files changed, 59 insertions(+), 62 deletions(-)

diff --git a/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
index 6fcf7c69bc..8e4bd910a4 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
@@ -17,6 +17,7 @@ import { ApplicationExceptionControllerService } from "./application-exception.c
 @AllowAuthorizedParty(AuthorizedParties.institution)
 @Controller("application-exception")
 @IsBCPublicInstitution()
+@HasStudentDataAccess("studentId")
 @ApiTags(`${ClientTypeBaseRoute.Institution}-application-exception`)
 export class ApplicationExceptionInstitutionsController extends BaseController {
   constructor(
@@ -31,7 +32,6 @@ export class ApplicationExceptionInstitutionsController extends BaseController {
    * @param exceptionId exception to be retrieved.
    * @returns student application exception information.
    */
-  @HasStudentDataAccess("studentId")
   @Get("student/:studentId/exception/:exceptionId")
   @ApiNotFoundResponse({
     description: "Student application exception not found.",
diff --git a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
index d658938d43..27fb312cf3 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/assessment/assessment.institutions.controller.ts
@@ -12,7 +12,7 @@ import {
   RequestAssessmentSummaryAPIOutDTO,
 } from "./models/assessment.dto";
 import { ApiTags } from "@nestjs/swagger";
-import { AssessmentControllerService } from "./assessment.controller.service";
+import { AssessmentControllerService } from "..";
 
 /**
  * Assessment controller for institutions.
@@ -20,6 +20,7 @@ import { AssessmentControllerService } from "./assessment.controller.service";
 @AllowAuthorizedParty(AuthorizedParties.institution)
 @Controller("assessment")
 @IsBCPublicInstitution()
+@HasStudentDataAccess("studentId")
 @ApiTags(`${ClientTypeBaseRoute.Institution}-assessment`)
 export class AssessmentInstitutionsController extends BaseController {
   constructor(
@@ -35,7 +36,6 @@ export class AssessmentInstitutionsController extends BaseController {
    * @param applicationId application id.
    * @returns assessment requests or exceptions for the student application.
    */
-  @HasStudentDataAccess("studentId")
   @Get("student/:studentId/application/:applicationId/requests")
   async getRequestedAssessmentSummary(
     @Param("studentId", ParseIntPipe) studentId: number,
@@ -48,7 +48,7 @@ export class AssessmentInstitutionsController extends BaseController {
   }
 
   /**
-   * Method to get history of assessments for an application,
+   * Get history of assessments for an application,
    * i.e, this will have original assessment for the
    * student application, and all approved student
    * appeal and scholastic standings for the application
@@ -57,7 +57,6 @@ export class AssessmentInstitutionsController extends BaseController {
    * @param applicationId, application id.
    * @returns summary of the assessment history for a student application.
    */
-  @HasStudentDataAccess("studentId")
   @Get("student/:studentId/application/:applicationId/history")
   async getAssessmentHistorySummary(
     @Param("studentId", ParseIntPipe) studentId: number,
diff --git a/sources/packages/backend/apps/api/src/route-controllers/index.ts b/sources/packages/backend/apps/api/src/route-controllers/index.ts
index 9fb09b963e..d1e81e7721 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/index.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/index.ts
@@ -1,7 +1,6 @@
 export * from "./program-year/program-year.students.controller";
 export * from "./config/config.controller";
 export * from "./dynamic-form/dynamic-form.controller";
-export * from "./education-program/education-program.controller.service";
 export * from "./education-program/education-program.aest.controller";
 export * from "./education-program/education-program.institutions.controller";
 export * from "./education-program/education-program.students.controller";
@@ -13,22 +12,17 @@ export * from "./restriction/restriction.aest.controller";
 export * from "./restriction/restriction.students.controller";
 export * from "./designation-agreement/designation-agreement.institutions.controller";
 export * from "./supporting-user/supporting-user.aest.controller";
-export * from "./designation-agreement/designation-agreement.controller.service";
 export * from "./designation-agreement/designation-agreement.aest.controller";
 export * from "./application/application.aest.controller";
 export * from "./application/application.students.controller";
 export * from "./application/application.institutions.controller";
-export * from "./assessment/assessment.controller.service";
 export * from "./institution/institution.aest.controller";
 export * from "./institution/institution.institutions.controller";
-export * from "./institution/institution.controller.service";
-export * from "./student/student.controller.service";
 export * from "./student/student.aest.controller";
 export * from "./student/student.students.controller";
 export * from "./student-appeal/student-appeal.students.controller";
 export * from "./student-appeal/student-appeal.aest.controller";
 export * from "./assessment/assessment.aest.controller";
-export * from "./institution-locations/institution-location.controller.service";
 export * from "./institution-locations/institution-location.institutions.controller";
 export * from "./institution-locations/institution-location.students.controller";
 export * from "./institution-locations/institution-location.aest.controller";
@@ -39,28 +33,37 @@ export * from "./education-program/education-program.students.controller";
 export * from "./application-exception/application-exception.aest.controller";
 export * from "./student-scholastic-standings/student-scholastic-standings.institutions.controller";
 export * from "./student-scholastic-standings/student-scholastic-standings.aest.controller";
-export * from "./student-scholastic-standings/student-scholastic-standings.controller.service";
 export * from "./institution-user/institution-user.aest.controller";
 export * from "./institution-user/institution-user.institutions.controller";
-export * from "./institution-user/institution-user.controller.service";
 export * from "./student-account-applications/student-account-application.aest.controller";
 export * from "./student-account-applications/student-account-application.students.controller";
 export * from "./education-program-offering/education-program-offering.institutions.controller";
-export * from "./education-program-offering/education-program-offering.controller.service";
 export * from "./education-program-offering/education-program-offering.students.controller";
-export * from "./user/user.controller.service";
 export * from "./user/user.aest.controller";
 export * from "./user/user.institutions.controller";
-export * from "./application/application.controller.service";
-export * from "./confirmation-of-enrollment/confirmation-of-enrollment.controller.service";
 export * from "./confirmation-of-enrollment/confirmation-of-enrollment.aest.controller";
-export * from "./overaward/overaward.controller.service";
 export * from "./overaward/overaward.aest.controller";
 export * from "./overaward/overaward.institutions.controller";
 export * from "./overaward/overaward.students.controller";
 export * from "./student/student.institutions.controller";
 export * from "./assessment/assessment.institutions.controller";
 export * from "./application-exception/application-exception.institutions.controller";
-export * from "./application-exception/application-exception.controller.service";
-export * from "./student-appeal/student-appeal.controller.service";
 export * from "./student-appeal/student-appeal.institutions.controller";
+
+
+// Controller services.
+export * from "./student-scholastic-standings/student-scholastic-standings.controller.service";
+export * from "./institution-locations/institution-location.controller.service";
+export * from "./institution/institution.controller.service";
+export * from "./student/student.controller.service";
+export * from "./education-program-offering/education-program-offering.controller.service";
+export * from "./institution-user/institution-user.controller.service";
+export * from "./assessment/assessment.controller.service";
+export * from "./designation-agreement/designation-agreement.controller.service";
+export * from "./education-program/education-program.controller.service";
+export * from "./user/user.controller.service";
+export * from "./application/application.controller.service";
+export * from "./confirmation-of-enrollment/confirmation-of-enrollment.controller.service";
+export * from "./overaward/overaward.controller.service";
+export * from "./application-exception/application-exception.controller.service";
+export * from "./student-appeal/student-appeal.controller.service";
\ No newline at end of file
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/models/student-appeal.dto.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/models/student-appeal.dto.ts
index 136d40772d..9b6956b85e 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/models/student-appeal.dto.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/models/student-appeal.dto.ts
@@ -49,18 +49,11 @@ export class DetailedStudentAppealRequestAPIOutDTO extends StudentAppealRequestA
   noteDescription?: string;
 }
 
-export class StudentAppealAPIOutDTO {
+export class StudentAppealAPIOutDTO<T> {
   id: number;
   submittedDate: Date;
   status: StudentAppealStatus;
-  appealRequests: StudentAppealRequestAPIOutDTO[];
-}
-
-export class DetailedStudentAppealAPIOutDTO {
-  id: number;
-  submittedDate: Date;
-  status: StudentAppealStatus;
-  appealRequests: DetailedStudentAppealRequestAPIOutDTO[];
+  appealRequests: T[];
 }
 
 export class StudentAppealRequestApprovalAPIInDTO {
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
index 332fdf5550..f5a185efa7 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.aest.controller.ts
@@ -30,7 +30,8 @@ import BaseController from "../BaseController";
 import { ApiProcessError, ClientTypeBaseRoute } from "../../types";
 import { UserGroups } from "../../auth/user-groups.enum";
 import {
-  DetailedStudentAppealAPIOutDTO,
+  DetailedStudentAppealRequestAPIOutDTO,
+  StudentAppealAPIOutDTO,
   StudentAppealApprovalAPIInDTO,
   StudentAppealPendingSummaryAPIOutDTO,
 } from "./models/student-appeal.dto";
@@ -73,10 +74,9 @@ export class StudentAppealAESTController extends BaseController {
   })
   async getStudentAppealWithRequests(
     @Param("appealId", ParseIntPipe) appealId: number,
-  ): Promise<DetailedStudentAppealAPIOutDTO> {
-    return this.studentAppealControllerService.getStudentAppealWithRequest(
+  ): Promise<StudentAppealAPIOutDTO<DetailedStudentAppealRequestAPIOutDTO>> {
+    return this.studentAppealControllerService.getStudentAppealWithRequest<DetailedStudentAppealRequestAPIOutDTO>(
       appealId,
-      undefined,
       { assessDetails: true },
     );
   }
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
index 0127773177..15556df91f 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
@@ -1,9 +1,6 @@
 import { Injectable, NotFoundException } from "@nestjs/common";
 import {
   StudentAppealAPIOutDTO,
-  DetailedStudentAppealAPIOutDTO,
-  StudentAppealRequestAPIOutDTO,
-  DetailedStudentAppealRequestAPIOutDTO,
 } from "./models/student-appeal.dto";
 import { getUserFullName } from "../../utilities";
 import { StudentAppealService } from "../../services";
@@ -15,20 +12,22 @@ export class StudentAppealControllerService {
   /**
    * Get the student appeal and its requests.
    * @param appealId appeal id to be retrieved.
-   * @param studentId student id.
+   * @param options options
+   * - `studentId` student id.
+   * - `assessDetails`, if true, will return access details.
    * @returns the student appeal and its requests.
    */
-  async getStudentAppealWithRequest(
+  async getStudentAppealWithRequest<T>(
     appealId: number,
-    studentId?: number,
     options?: {
+      studentId?: number,
       assessDetails?: boolean;
     },
-  ): Promise<StudentAppealAPIOutDTO | DetailedStudentAppealAPIOutDTO> {
+  ): Promise<StudentAppealAPIOutDTO<T>> {
     const studentAppeal =
       await this.studentAppealService.getAppealAndRequestsById(
         appealId,
-        studentId,
+        options?.studentId,
       );
     if (!studentAppeal) {
       throw new NotFoundException("Not able to find the student appeal.");
@@ -39,21 +38,22 @@ export class StudentAppealControllerService {
       submittedDate: studentAppeal.submittedDate,
       status: studentAppeal.status,
       appealRequests: studentAppeal.appealRequests.map((appealRequest) => {
-        let requests = {
+        let request = {
           id: appealRequest.id,
           appealStatus: appealRequest.appealStatus,
           submittedData: appealRequest.submittedData,
           submittedFormName: appealRequest.submittedFormName,
-        } as StudentAppealRequestAPIOutDTO;
+        } as T;
+
         if (options?.assessDetails) {
-          requests = {
-            ...requests,
+          request = {
+            ...request,
             assessedDate: appealRequest.assessedDate,
             noteDescription: appealRequest.note?.description,
             assessedByUserName: getUserFullName(appealRequest.assessedBy),
-          } as DetailedStudentAppealRequestAPIOutDTO;
+          } as T;
         }
-        return requests;
+        return request as T;
       }),
     };
   }
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
index 5e8cb217d6..d0eaa84b55 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.institutions.controller.ts
@@ -1,21 +1,25 @@
 import { Controller, Param, Get, ParseIntPipe } from "@nestjs/common";
 import { AuthorizedParties } from "../../auth/authorized-parties.enum";
-import { 
+import {
   AllowAuthorizedParty,
   IsBCPublicInstitution,
-  HasStudentDataAccess, 
+  HasStudentDataAccess,
 } from "../../auth/decorators";
 import { ApiNotFoundResponse, ApiTags } from "@nestjs/swagger";
 import BaseController from "../BaseController";
 import { ClientTypeBaseRoute } from "../../types";
-import { DetailedStudentAppealAPIOutDTO } from "./models/student-appeal.dto";
-import { StudentAppealControllerService } from "./student-appeal.controller.service";
+import {
+  DetailedStudentAppealRequestAPIOutDTO,
+  StudentAppealAPIOutDTO,
+} from "./models/student-appeal.dto";
+import { StudentAppealControllerService } from "..";
 
 /**
  * Student appeal controller for institutions.
  */
 @AllowAuthorizedParty(AuthorizedParties.institution)
 @IsBCPublicInstitution()
+@HasStudentDataAccess("studentId")
 @Controller("appeal")
 @ApiTags(`${ClientTypeBaseRoute.Institution}-appeal`)
 export class StudentAppealInstitutionsController extends BaseController {
@@ -27,11 +31,10 @@ export class StudentAppealInstitutionsController extends BaseController {
 
   /**
    * Get the student appeal details.
-   * @param appealId appeal id to be retrieved.
    * @param studentId student id.
+   * @param appealId appeal id to be retrieved.
    * @returns the student appeal details.
    */
-  @HasStudentDataAccess("studentId")
   @Get("student/:studentId/appeal/:appealId/requests")
   @ApiNotFoundResponse({
     description: "Not able to find the student appeal.",
@@ -39,11 +42,10 @@ export class StudentAppealInstitutionsController extends BaseController {
   async getStudentAppealWithRequest(
     @Param("studentId", ParseIntPipe) studentId: number,
     @Param("appealId", ParseIntPipe) appealId: number,
-  ): Promise<DetailedStudentAppealAPIOutDTO> {
-    return this.studentAppealControllerService.getStudentAppealWithRequest(
+  ): Promise<StudentAppealAPIOutDTO<DetailedStudentAppealRequestAPIOutDTO>> {
+    return this.studentAppealControllerService.getStudentAppealWithRequest<DetailedStudentAppealRequestAPIOutDTO>(
       appealId,
-      studentId,
-      { assessDetails: true },
+      { assessDetails: true, studentId: studentId },
     );
   }
 }
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
index a8c9b33e0b..2395d5ad5d 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
@@ -18,6 +18,7 @@ import {
 import {
   StudentAppealAPIInDTO,
   StudentAppealAPIOutDTO,
+  StudentAppealRequestAPIOutDTO,
 } from "./models/student-appeal.dto";
 import { PrimaryIdentifierAPIOutDTO } from "../models/primary.identifier.dto";
 import { AuthorizedParties } from "../../auth/authorized-parties.enum";
@@ -166,10 +167,10 @@ export class StudentAppealStudentsController extends BaseController {
   async getStudentAppealWithRequests(
     @Param("appealId", ParseIntPipe) appealId: number,
     @UserToken() userToken: StudentUserToken,
-  ): Promise<StudentAppealAPIOutDTO> {
-    return this.studentAppealControllerService.getStudentAppealWithRequest(
+  ): Promise<StudentAppealAPIOutDTO<StudentAppealRequestAPIOutDTO>> {
+    return this.studentAppealControllerService.getStudentAppealWithRequest<StudentAppealRequestAPIOutDTO>(
       appealId,
-      userToken.studentId,
+      { studentId : userToken.studentId },
     );
   }
 }
diff --git a/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts b/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts
index 2683afc67a..6d2512cb5d 100644
--- a/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts
+++ b/sources/packages/backend/apps/api/src/services/application-exception/application-exception.service.ts
@@ -76,8 +76,8 @@ export class ApplicationExceptionService extends RecordDataModelService<Applicat
    * Get a student application exception detected after the student application was
    * submitted, for instance, when there are documents to be reviewed.
    * @param applicationId application associated with the exception.
-   * @param status statuses to be filtered.
    * @param studentId student id.
+   * @param status statuses to be filtered.
    * @returns student application exception information.
    */
   async getExceptionsByApplicationId(
diff --git a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
index c3e5126d25..9645cfc743 100644
--- a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
+++ b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
@@ -17,7 +17,6 @@
       @viewStudentAppeal="goToStudentAppeal"
       @viewApplicationException="goToApplicationException"
     />
-
     <history-assessment
       :applicationId="applicationId"
       :studentId="studentId"

From aaf3d72d73a007f33ff09fbb508bfc57d810f382 Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Fri, 12 May 2023 19:01:57 -0400
Subject: [PATCH 09/12] refactor

---
 .../apps/api/src/route-controllers/index.ts   | 33 +++++++++----------
 .../student-appeal.students.controller.ts     |  2 +-
 .../StudentAppealRequestsApproval.vue         |  4 +--
 .../web/src/services/StudentAppealService.ts  | 12 +++----
 .../web/src/services/http/StudentAppealApi.ts | 17 +++++-----
 .../services/http/dto/StudentAppeal.dto.ts    | 14 +++-----
 .../web/src/services/http/dto/index.ts        |  1 +
 .../InstitutionAssessmentsSummary.vue         | 15 ++++++---
 .../views/student/StudentAppealRequests.vue   |  3 +-
 9 files changed, 49 insertions(+), 52 deletions(-)

diff --git a/sources/packages/backend/apps/api/src/route-controllers/index.ts b/sources/packages/backend/apps/api/src/route-controllers/index.ts
index d1e81e7721..cad9b21b1a 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/index.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/index.ts
@@ -1,6 +1,7 @@
 export * from "./program-year/program-year.students.controller";
 export * from "./config/config.controller";
 export * from "./dynamic-form/dynamic-form.controller";
+export * from "./education-program/education-program.controller.service";
 export * from "./education-program/education-program.aest.controller";
 export * from "./education-program/education-program.institutions.controller";
 export * from "./education-program/education-program.students.controller";
@@ -12,17 +13,22 @@ export * from "./restriction/restriction.aest.controller";
 export * from "./restriction/restriction.students.controller";
 export * from "./designation-agreement/designation-agreement.institutions.controller";
 export * from "./supporting-user/supporting-user.aest.controller";
+export * from "./designation-agreement/designation-agreement.controller.service";
 export * from "./designation-agreement/designation-agreement.aest.controller";
 export * from "./application/application.aest.controller";
 export * from "./application/application.students.controller";
 export * from "./application/application.institutions.controller";
+export * from "./assessment/assessment.controller.service";
 export * from "./institution/institution.aest.controller";
 export * from "./institution/institution.institutions.controller";
+export * from "./institution/institution.controller.service";
+export * from "./student/student.controller.service";
 export * from "./student/student.aest.controller";
 export * from "./student/student.students.controller";
 export * from "./student-appeal/student-appeal.students.controller";
 export * from "./student-appeal/student-appeal.aest.controller";
 export * from "./assessment/assessment.aest.controller";
+export * from "./institution-locations/institution-location.controller.service";
 export * from "./institution-locations/institution-location.institutions.controller";
 export * from "./institution-locations/institution-location.students.controller";
 export * from "./institution-locations/institution-location.aest.controller";
@@ -33,37 +39,28 @@ export * from "./education-program/education-program.students.controller";
 export * from "./application-exception/application-exception.aest.controller";
 export * from "./student-scholastic-standings/student-scholastic-standings.institutions.controller";
 export * from "./student-scholastic-standings/student-scholastic-standings.aest.controller";
+export * from "./student-scholastic-standings/student-scholastic-standings.controller.service";
 export * from "./institution-user/institution-user.aest.controller";
 export * from "./institution-user/institution-user.institutions.controller";
+export * from "./institution-user/institution-user.controller.service";
 export * from "./student-account-applications/student-account-application.aest.controller";
 export * from "./student-account-applications/student-account-application.students.controller";
 export * from "./education-program-offering/education-program-offering.institutions.controller";
+export * from "./education-program-offering/education-program-offering.controller.service";
 export * from "./education-program-offering/education-program-offering.students.controller";
+export * from "./user/user.controller.service";
 export * from "./user/user.aest.controller";
 export * from "./user/user.institutions.controller";
+export * from "./application/application.controller.service";
+export * from "./confirmation-of-enrollment/confirmation-of-enrollment.controller.service";
 export * from "./confirmation-of-enrollment/confirmation-of-enrollment.aest.controller";
+export * from "./overaward/overaward.controller.service";
 export * from "./overaward/overaward.aest.controller";
 export * from "./overaward/overaward.institutions.controller";
 export * from "./overaward/overaward.students.controller";
 export * from "./student/student.institutions.controller";
 export * from "./assessment/assessment.institutions.controller";
 export * from "./application-exception/application-exception.institutions.controller";
-export * from "./student-appeal/student-appeal.institutions.controller";
-
-
-// Controller services.
-export * from "./student-scholastic-standings/student-scholastic-standings.controller.service";
-export * from "./institution-locations/institution-location.controller.service";
-export * from "./institution/institution.controller.service";
-export * from "./student/student.controller.service";
-export * from "./education-program-offering/education-program-offering.controller.service";
-export * from "./institution-user/institution-user.controller.service";
-export * from "./assessment/assessment.controller.service";
-export * from "./designation-agreement/designation-agreement.controller.service";
-export * from "./education-program/education-program.controller.service";
-export * from "./user/user.controller.service";
-export * from "./application/application.controller.service";
-export * from "./confirmation-of-enrollment/confirmation-of-enrollment.controller.service";
-export * from "./overaward/overaward.controller.service";
 export * from "./application-exception/application-exception.controller.service";
-export * from "./student-appeal/student-appeal.controller.service";
\ No newline at end of file
+export * from "./student-appeal/student-appeal.controller.service";
+export * from "./student-appeal/student-appeal.institutions.controller";
\ No newline at end of file
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
index 2395d5ad5d..b10ae68274 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.students.controller.ts
@@ -170,7 +170,7 @@ export class StudentAppealStudentsController extends BaseController {
   ): Promise<StudentAppealAPIOutDTO<StudentAppealRequestAPIOutDTO>> {
     return this.studentAppealControllerService.getStudentAppealWithRequest<StudentAppealRequestAPIOutDTO>(
       appealId,
-      { studentId : userToken.studentId },
+      { studentId: userToken.studentId },
     );
   }
 }
diff --git a/sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue b/sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue
index f452c1fedb..74ea1470be 100644
--- a/sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue
+++ b/sources/packages/web/src/components/common/students/applicationDetails/StudentAppealRequestsApproval.vue
@@ -60,7 +60,7 @@ import {
 import AppealRequestsApprovalForm from "@/components/aest/AppealRequestsApprovalForm.vue";
 import StatusChipRequestedAssessment from "@/components/generic/StatusChipRequestedAssessment.vue";
 import CheckPermissionRole from "@/components/generic/CheckPermissionRole.vue";
-
+import { DetailedStudentAppealRequestAPIOutDTO } from "@/services/http/dto/StudentAppeal.dto";
 export default defineComponent({
   emits: {
     submitted: (approvals: StudentAppealApproval[]) => {
@@ -104,7 +104,7 @@ export default defineComponent({
 
     onMounted(async () => {
       const appeal =
-        await StudentAppealService.shared.getStudentAppealWithRequests(
+        await StudentAppealService.shared.getStudentAppealWithRequests<DetailedStudentAppealRequestAPIOutDTO>(
           props.appealId,
           props.studentId,
         );
diff --git a/sources/packages/web/src/services/StudentAppealService.ts b/sources/packages/web/src/services/StudentAppealService.ts
index 458b3f2515..94c535335a 100644
--- a/sources/packages/web/src/services/StudentAppealService.ts
+++ b/sources/packages/web/src/services/StudentAppealService.ts
@@ -1,12 +1,12 @@
 import ApiClient from "@/services/http/ApiClient";
 import { PaginationOptions, StudentAppealRequest } from "@/types";
 import {
-  DetailedStudentAppealAPIOutDTO,
+  PaginatedResultsAPIOutDTO,
+  StudentAppealAPIOutDTO,
   StudentAppealPendingSummaryAPIOutDTO,
   StudentAppealRequestAPIInDTO,
   StudentAppealRequestApprovalAPIInDTO,
-} from "@/services/http/dto/StudentAppeal.dto";
-import { PaginatedResultsAPIOutDTO } from "./http/dto";
+} from "./http/dto";
 
 /**
  * Client service layer for Student Appeal.
@@ -41,11 +41,11 @@ export class StudentAppealService {
    * @param studentId student id.
    * @returns student application appeal.
    */
-  async getStudentAppealWithRequests(
+  async getStudentAppealWithRequests<T>(
     appealId: number,
     studentId?: number,
-  ): Promise<DetailedStudentAppealAPIOutDTO> {
-    return ApiClient.StudentAppealApi.getStudentAppealWithRequests(
+  ): Promise<StudentAppealAPIOutDTO<T>> {
+    return ApiClient.StudentAppealApi.getStudentAppealWithRequests<T>(
       appealId,
       studentId,
     );
diff --git a/sources/packages/web/src/services/http/StudentAppealApi.ts b/sources/packages/web/src/services/http/StudentAppealApi.ts
index 60e37397eb..1df8e4de0f 100644
--- a/sources/packages/web/src/services/http/StudentAppealApi.ts
+++ b/sources/packages/web/src/services/http/StudentAppealApi.ts
@@ -1,15 +1,14 @@
 import { getPaginationQueryString } from "@/helpers";
 import HttpBaseClient from "@/services/http/common/HttpBaseClient";
 import { PaginationOptions } from "@/types";
-import { PaginatedResultsAPIOutDTO } from "./dto";
 import {
+  PaginatedResultsAPIOutDTO,
   StudentAppealAPIInDTO,
   StudentAppealAPIOutDTO,
   StudentAppealApprovalAPIInDTO,
   StudentAppealPendingSummaryAPIOutDTO,
   StudentAppealRequestApprovalAPIInDTO,
-  DetailedStudentAppealAPIOutDTO,
-} from "./dto/StudentAppeal.dto";
+} from "./dto";
 
 /**
  * Http API client for Student Appeal.
@@ -26,21 +25,21 @@ export class StudentAppealApi extends HttpBaseClient {
   }
 
   /**
-   * Get student application appeal.
+   * Get student application appeal for ministry.
    * @param appealId appeal id.
    * @param studentId student id.
    * @returns student application appeal.
    */
-  async getStudentAppealWithRequests(
+  async getStudentAppealWithRequests<T>(
     appealId: number,
     studentId?: number,
-  ): Promise<StudentAppealAPIOutDTO | DetailedStudentAppealAPIOutDTO> {
+  ): Promise<StudentAppealAPIOutDTO<T>> {
     const endpoint = studentId
       ? `appeal/student/${studentId}/appeal/${appealId}/requests`
       : `appeal/${appealId}/requests`;
-    return this.getCall<
-      StudentAppealAPIOutDTO | DetailedStudentAppealAPIOutDTO
-    >(this.addClientRoot(endpoint));
+    return this.getCall<StudentAppealAPIOutDTO<T>>(
+      this.addClientRoot(endpoint),
+    );
   }
 
   async approveStudentAppealRequests(
diff --git a/sources/packages/web/src/services/http/dto/StudentAppeal.dto.ts b/sources/packages/web/src/services/http/dto/StudentAppeal.dto.ts
index c9067c1ec7..310a1fdce9 100644
--- a/sources/packages/web/src/services/http/dto/StudentAppeal.dto.ts
+++ b/sources/packages/web/src/services/http/dto/StudentAppeal.dto.ts
@@ -16,24 +16,18 @@ export interface StudentAppealRequestAPIOutDTO {
   appealStatus: StudentAppealStatus;
 }
 
-export interface DetailedStudentAppealRequestAPIOutDTO extends StudentAppealRequestAPIOutDTO {
+export interface DetailedStudentAppealRequestAPIOutDTO
+  extends StudentAppealRequestAPIOutDTO {
   assessedDate?: Date;
   assessedByUserName?: string;
   noteDescription?: string;
 }
 
-export interface StudentAppealAPIOutDTO {
+export interface StudentAppealAPIOutDTO<T> {
   id: number;
   submittedDate: Date;
   status: StudentAppealStatus;
-  appealRequests: StudentAppealRequestAPIOutDTO[];
-}
-
-export interface DetailedStudentAppealAPIOutDTO {
-  id: number;
-  submittedDate: Date;
-  status: StudentAppealStatus;
-  appealRequests: DetailedStudentAppealRequestAPIOutDTO[];
+  appealRequests: T[];
 }
 
 export interface StudentAppealRequestApprovalAPIInDTO {
diff --git a/sources/packages/web/src/services/http/dto/index.ts b/sources/packages/web/src/services/http/dto/index.ts
index 029fdfe21b..46babcd7b9 100644
--- a/sources/packages/web/src/services/http/dto/index.ts
+++ b/sources/packages/web/src/services/http/dto/index.ts
@@ -23,3 +23,4 @@ export * from "@/services/http/dto/Config.dto";
 export * from "@/services/http/dto/DesignationAgreement.dto";
 export * from "@/services/http/dto/SupportingUser.dto";
 export * from "@/services/http/dto/Overaward.dto";
+export * from "@/services/http/dto/StudentAppeal.dto";
diff --git a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
index 9645cfc743..f77d0808bd 100644
--- a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
+++ b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
@@ -3,10 +3,7 @@
     <template #header>
       <header-navigator
         title="Student applications"
-        :routeLocation="{
-          name: InstitutionRoutesConst.STUDENT_APPLICATIONS,
-          params: { studentId },
-        }"
+        :routeLocation="backRoute"
         subTitle="Assessment"
       />
     </template>
@@ -30,7 +27,7 @@
 <script lang="ts">
 import { InstitutionRoutesConst } from "@/constants/routes/RouteConstants";
 import { useRouter } from "vue-router";
-import { defineComponent } from "vue";
+import { defineComponent, computed } from "vue";
 import { AssessmentTriggerType } from "@/types";
 import RequestAssessment from "@/components/aest/students/assessment/Request.vue";
 import HistoryAssessment from "@/components/aest/students/assessment/History.vue";
@@ -82,11 +79,19 @@ export default defineComponent({
       });
     };
 
+    const backRoute = computed({
+      name: InstitutionRoutesConst.STUDENT_APPLICATIONS,
+      params: {
+        studentId: props.studentId,
+      },
+    });
+
     return {
       InstitutionRoutesConst,
       goToStudentAppeal,
       goToApplicationException,
       assessmentRequestTypes,
+      backRoute,
     };
   },
 });
diff --git a/sources/packages/web/src/views/student/StudentAppealRequests.vue b/sources/packages/web/src/views/student/StudentAppealRequests.vue
index 2c5c6ae694..8fe6d5415b 100644
--- a/sources/packages/web/src/views/student/StudentAppealRequests.vue
+++ b/sources/packages/web/src/views/student/StudentAppealRequests.vue
@@ -34,6 +34,7 @@ import { StudentAppealService } from "@/services/StudentAppealService";
 import { StudentAppealRequest, StudentAppealStatus } from "@/types";
 import AppealRequestsApprovalForm from "@/components/aest/AppealRequestsApprovalForm.vue";
 import StatusChipRequestedAssessment from "@/components/generic/StatusChipRequestedAssessment.vue";
+import { StudentAppealRequestAPIOutDTO } from "@/services/http/dto/StudentAppeal.dto";
 
 export default defineComponent({
   components: {
@@ -56,7 +57,7 @@ export default defineComponent({
 
     onMounted(async () => {
       const appeal =
-        await StudentAppealService.shared.getStudentAppealWithRequests(
+        await StudentAppealService.shared.getStudentAppealWithRequests<StudentAppealRequestAPIOutDTO>(
           props.appealId,
         );
       studentAppealRequests.value =

From d6697ea658ae608f4547952c40f1b9dcea7c5283 Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Fri, 12 May 2023 19:05:20 -0400
Subject: [PATCH 10/12] refactor

---
 .../student-appeal/student-appeal.controller.service.ts         | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
index 15556df91f..38b346c94b 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
@@ -53,7 +53,7 @@ export class StudentAppealControllerService {
             assessedByUserName: getUserFullName(appealRequest.assessedBy),
           } as T;
         }
-        return request as T;
+        return request;
       }),
     };
   }

From 7b7f6555a05eb37ef2ea9fa73a97372b6fe996bf Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Fri, 12 May 2023 19:19:26 -0400
Subject: [PATCH 11/12] refactot

---
 .../applicationDetails/InstitutionAssessmentsSummary.vue      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
index f77d0808bd..363814edf3 100644
--- a/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
+++ b/sources/packages/web/src/views/institution/student/applicationDetails/InstitutionAssessmentsSummary.vue
@@ -79,12 +79,12 @@ export default defineComponent({
       });
     };
 
-    const backRoute = computed({
+    const backRoute = computed(() => ({
       name: InstitutionRoutesConst.STUDENT_APPLICATIONS,
       params: {
         studentId: props.studentId,
       },
-    });
+    }));
 
     return {
       InstitutionRoutesConst,

From 36c4814810dbef731d95ea66184d96ec8c41d9d4 Mon Sep 17 00:00:00 2001
From: Ann <ann.jacob@aot-technologies.com>
Date: Mon, 15 May 2023 13:09:47 -0400
Subject: [PATCH 12/12] pretiier fix

---
 .../application-exception.institutions.controller.ts        | 4 ++--
 .../student-appeal/student-appeal.controller.service.ts     | 6 ++----
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
index 8e4bd910a4..8c0589b3fc 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/application-exception/application-exception.institutions.controller.ts
@@ -1,8 +1,8 @@
 import { Controller, Get, Param, ParseIntPipe } from "@nestjs/common";
-import { 
+import {
   AllowAuthorizedParty,
   IsBCPublicInstitution,
-  HasStudentDataAccess, 
+  HasStudentDataAccess,
 } from "../../auth/decorators";
 import { AuthorizedParties } from "../../auth/authorized-parties.enum";
 import { ClientTypeBaseRoute } from "../../types";
diff --git a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
index 38b346c94b..996c4add96 100644
--- a/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
+++ b/sources/packages/backend/apps/api/src/route-controllers/student-appeal/student-appeal.controller.service.ts
@@ -1,7 +1,5 @@
 import { Injectable, NotFoundException } from "@nestjs/common";
-import {
-  StudentAppealAPIOutDTO,
-} from "./models/student-appeal.dto";
+import { StudentAppealAPIOutDTO } from "./models/student-appeal.dto";
 import { getUserFullName } from "../../utilities";
 import { StudentAppealService } from "../../services";
 
@@ -20,7 +18,7 @@ export class StudentAppealControllerService {
   async getStudentAppealWithRequest<T>(
     appealId: number,
     options?: {
-      studentId?: number,
+      studentId?: number;
       assessDetails?: boolean;
     },
   ): Promise<StudentAppealAPIOutDTO<T>> {