fix: UNI-278 add else branch in getInitialDisplayCourses to reset sea…

…rch bar state (#284)
devsoc-unsw · May 9, 2024 · ca55f1d · ca55f1d
1 parent 7f46ee3
commit ca55f1d
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/frontend/src/components/CoursesList/CoursesList.tsx b/frontend/src/components/CoursesList/CoursesList.tsx
@@ -32,7 +32,7 @@ export default function CoursesList({
         // default courses
         try {
           const { courses } = (await get(
-            `/courses?offset=${index}`
+            `/courses?offset=${index}`,
           )) as Courses;
           fetchedCourses = courses;
         } catch (err) {
@@ -71,7 +71,7 @@ export default function CoursesList({
     const getSearchResults = async () => {
       try {
         const { courses } = (await get(
-          `/course/search/${searchTerm}`
+          `/course/search/${searchTerm}`,
         )) as Courses;
         searchCoursesRef.current = courses;
       } catch (err) {
@@ -84,6 +84,9 @@ export default function CoursesList({
     const getInitialDisplayCourses = () => {
       if (searchTerm !== "") {
         getSearchResults();
+      } else {
+        setDisplayCourses(initialCourses.slice(0, paginationOffset));
+        setInitialLoading(false);
       }
     };
     const loadOnScroll = () => {
@@ -119,7 +122,7 @@ export default function CoursesList({
                 terms={c.terms}
               />
             </a>
-          )
+          ),
         )}
         {!initialLoading ? (
           <p className="text-center opacity-50">No more courses</p>