Revert "revert changes"

This reverts commit 5e172dc.

.github/workflows/eslint.yml

@@ -19,4 +19,4 @@ jobs:
           npm install
           npm install -g eslint
       - name: Run ESLint
-        run: eslint . --ext .js,.jsx,.ts,.tsx --max-warnings 0
+        run: eslint . --ext .js,.jsx,.ts,.tsx --max-warnings 0

lib/components/popups/course-search/prereqs/PrereqDisplay.tsx

@@ -63,15 +63,15 @@ const PrereqDisplay: FC = () => {
     setHasPreReqs(false);
 
     // First get all valid preReqs (isNegative = true)
-    let preReqs = filterNNegatives(courseToShow);
+    let preReqs = filterNNegatives(version);
     setNNegativePreReqs(preReqs);
     // If there exists preReqs, we need to process and display them.
     if (version !== 'None' && preReqs.length > 0) {
       setHasPreReqs(true);
       display(preReqs);
     }
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [courseToShow]);
+  }, [version, courseCache]);
 
   /**
    * Processes then displays prereqs.

lib/components/popups/course-search/prereqs/PrereqDropdown.tsx

@@ -36,7 +36,9 @@ const PrereqDropdown: FC<{
   const getChildPrereqs = () => {
     // eslint-disable-next-line array-callback-return
     return element.map((el: any, index: number): JSX.Element => {
-      return processPrereqs(el, index);
+      if (typeof el === 'number') {
+        return processPrereqs(el, index);
+      } else return <></>;
     });
   };
 

lib/index.css

@@ -4,11 +4,7 @@
 
 body {
   margin: 0;
-  font-family:
-    Futura,
-    Trebuchet MS,
-    Arial,
-    sans-serif;
+  font-family: Futura, Trebuchet MS, Arial, sans-serif;
   -webkit-font-smoothing: antialiased;
   -moz-osx-font-smoothing: grayscale;
 }

lib/resources/assets.tsx

@@ -276,11 +276,9 @@ export const course_tags = [
  * @param inspected - the course
  * @returns array with valid prereqs
  */
-export const filterNNegatives = (
-  inspected: UserCourse | Course | 'None' | null,
-): any[] => {
+export const filterNNegatives = (inspected: Course | 'None'): any[] => {
   let preReqs: any[] = [];
-  if (inspected !== 'None' && inspected) {
+  if (inspected !== 'None' && inspected !== undefined) {
     preReqs = inspected.preReq.filter((section: any) => {
       return section.IsNegative === 'N';
     });
@@ -1033,8 +1031,7 @@ export const checkAllPrereqs = (
   return new Promise((resolve) => {
     getCourse(number, courseCache, currCourses, -1).then((course) => {
       if (course.resp !== null) {
-        const version = getVersion(course.resp, semester + ' ' + year.year);
-        let filtered = filterNNegatives(version);
+        let filtered = filterNNegatives(course.resp);
         if (filtered.length === 0) {
           return resolve(true);
         } else {
@@ -1064,24 +1061,6 @@ export const checkAllPrereqs = (
   });
 };
 
-/**
- * @param course - course object
- * @param term - term of the course
- * @returns the version of course with matching term
- */
-export const getVersion = (course: any, term: string) => {
-  for (let v of course.versions) {
-    if (v.term === term) {
-      return v;
-    }
-  }
-  if (course.versions.length > 0) {
-    return course.versions[0];
-  } else {
-    return course;
-  }
-};
-
 /**
  * @param major - major name
  * @returns the major object from the name

lib/resources/redux_sample/Counter.module.css

@@ -64,9 +64,7 @@
   left: 0;
   top: 0;
   opacity: 0;
-  transition:
-    width 1s linear,
-    opacity 0.5s ease 1s;
+  transition: width 1s linear, opacity 0.5s ease 1s;
 }
 
 .asyncButton:active:after {