From b44938bda34ccb1ae54317e5cb9dfce8016e9e0f Mon Sep 17 00:00:00 2001
From: averheecke-scottlogic
 <125259220+averheecke-scottlogic@users.noreply.github.com>
Date: Sun, 24 Dec 2023 05:08:56 +0000
Subject: [PATCH] [#12656] Student Home Page: Indicate default sort  (#12660)

* sortBy to CREATION DATE

* snapshot update

* sort to creation time upon load

* linting to specs

* trailing space fix

* replaces sort with sort function

---------

Co-authored-by: Dominic Lim <46486515+domlimm@users.noreply.github.com>
---
 .../student-home-page.component.spec.ts.snap   | 18 ++++++++++--------
 .../student-home-page.component.ts             |  4 ++--
 2 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/src/web/app/pages-student/student-home-page/__snapshots__/student-home-page.component.spec.ts.snap b/src/web/app/pages-student/student-home-page/__snapshots__/student-home-page.component.spec.ts.snap
index fd0ab2a5d62..9113fe2508a 100644
--- a/src/web/app/pages-student/student-home-page/__snapshots__/student-home-page.component.spec.ts.snap
+++ b/src/web/app/pages-student/student-home-page/__snapshots__/student-home-page.component.spec.ts.snap
@@ -13,7 +13,7 @@ exports[`StudentHomePageComponent should snap when courses are still loading 1`]
   isCoursesLoading={[Function Boolean]}
   route={[Function ActivatedRoute]}
   sessionSubmissionStatusPipe={[Function SubmissionStatusPipe]}
-  sortBy="0"
+  sortBy={[Function Number]}
   statusMessageService={[Function StatusMessageService]}
   studentFeedbackSessionStatusAwaiting={[Function String]}
   studentFeedbackSessionStatusClosed={[Function String]}
@@ -58,7 +58,7 @@ exports[`StudentHomePageComponent should snap when there is course loading faile
   isCoursesLoading="false"
   route={[Function ActivatedRoute]}
   sessionSubmissionStatusPipe={[Function SubmissionStatusPipe]}
-  sortBy="0"
+  sortBy={[Function Number]}
   statusMessageService={[Function StatusMessageService]}
   studentFeedbackSessionStatusAwaiting={[Function String]}
   studentFeedbackSessionStatusClosed={[Function String]}
@@ -106,7 +106,7 @@ exports[`StudentHomePageComponent should snap with all feedback sessions over 2
   isCoursesLoading="false"
   route={[Function ActivatedRoute]}
   sessionSubmissionStatusPipe={[Function SubmissionStatusPipe]}
-  sortBy="0"
+  sortBy={[Function Number]}
   statusMessageService={[Function StatusMessageService]}
   studentFeedbackSessionStatusAwaiting={[Function String]}
   studentFeedbackSessionStatusClosed={[Function String]}
@@ -152,6 +152,7 @@ exports[`StudentHomePageComponent should snap with all feedback sessions over 2
               </button>
               <button
                 class="btn btn-light"
+                disabled=""
                 id="sort-course-date-btn"
               >
                  Creation Date 
@@ -488,7 +489,7 @@ exports[`StudentHomePageComponent should snap with default fields 1`] = `
   isCoursesLoading={[Function Boolean]}
   route={[Function ActivatedRoute]}
   sessionSubmissionStatusPipe={[Function SubmissionStatusPipe]}
-  sortBy="0"
+  sortBy={[Function Number]}
   statusMessageService={[Function StatusMessageService]}
   studentFeedbackSessionStatusAwaiting={[Function String]}
   studentFeedbackSessionStatusClosed={[Function String]}
@@ -533,7 +534,7 @@ exports[`StudentHomePageComponent should snap with feedback sessions 1`] = `
   isCoursesLoading="false"
   route={[Function ActivatedRoute]}
   sessionSubmissionStatusPipe={[Function SubmissionStatusPipe]}
-  sortBy="0"
+  sortBy={[Function Number]}
   statusMessageService={[Function StatusMessageService]}
   studentFeedbackSessionStatusAwaiting={[Function String]}
   studentFeedbackSessionStatusClosed={[Function String]}
@@ -598,7 +599,7 @@ exports[`StudentHomePageComponent should snap with no course 1`] = `
   isCoursesLoading="false"
   route={[Function ActivatedRoute]}
   sessionSubmissionStatusPipe={[Function SubmissionStatusPipe]}
-  sortBy="0"
+  sortBy={[Function Number]}
   statusMessageService={[Function StatusMessageService]}
   studentFeedbackSessionStatusAwaiting={[Function String]}
   studentFeedbackSessionStatusClosed={[Function String]}
@@ -641,7 +642,7 @@ exports[`StudentHomePageComponent should snap with no feedback session over 2 co
   isCoursesLoading="false"
   route={[Function ActivatedRoute]}
   sessionSubmissionStatusPipe={[Function SubmissionStatusPipe]}
-  sortBy="0"
+  sortBy={[Function Number]}
   statusMessageService={[Function StatusMessageService]}
   studentFeedbackSessionStatusAwaiting={[Function String]}
   studentFeedbackSessionStatusClosed={[Function String]}
@@ -687,6 +688,7 @@ exports[`StudentHomePageComponent should snap with no feedback session over 2 co
               </button>
               <button
                 class="btn btn-light"
+                disabled=""
                 id="sort-course-date-btn"
               >
                  Creation Date 
@@ -777,7 +779,7 @@ exports[`StudentHomePageComponent should snap with no feedback sessions 1`] = `
   isCoursesLoading="false"
   route={[Function ActivatedRoute]}
   sessionSubmissionStatusPipe={[Function SubmissionStatusPipe]}
-  sortBy="0"
+  sortBy={[Function Number]}
   statusMessageService={[Function StatusMessageService]}
   studentFeedbackSessionStatusAwaiting={[Function String]}
   studentFeedbackSessionStatusClosed={[Function String]}
diff --git a/src/web/app/pages-student/student-home-page/student-home-page.component.ts b/src/web/app/pages-student/student-home-page/student-home-page.component.ts
index 08e013d8d26..5f268335e6d 100644
--- a/src/web/app/pages-student/student-home-page/student-home-page.component.ts
+++ b/src/web/app/pages-student/student-home-page/student-home-page.component.ts
@@ -72,7 +72,7 @@ export class StudentHomePageComponent implements OnInit {
   isCoursesLoading: boolean = false;
   hasCoursesLoadingFailed: boolean = false;
 
-  sortBy: SortBy = SortBy.NONE;
+  sortBy: SortBy = SortBy.COURSE_CREATION_DATE;
 
   sessionSubmissionStatusPipe = new SubmissionStatusPipe();
   formatDateDetailPipe = new FormatDateDetailPipe(this.timezoneService);
@@ -114,7 +114,7 @@ export class StudentHomePageComponent implements OnInit {
             });
           });
 
-          this.courses.sort((a: StudentCourse, b: StudentCourse) => (a.course.courseId > b.course.courseId ? 1 : -1));
+          this.sortCoursesBy(SortBy.COURSE_CREATION_DATE);
           this.courses.slice(0, 3).forEach((course: StudentCourse) => {
             course.isTabExpanded = true;
             this.loadFeedbackSessionsForCourse(course.course.courseId);