fix CI build errors

VikeLabs · Jun 5, 2022 · ddada51 · ddada51
1 parent 87ad861
commit ddada51
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/src/lib/hooks/useSavedCourses.ts b/src/lib/hooks/useSavedCourses.ts
@@ -104,13 +104,14 @@ export const useSavedCourses = (): SavedCourses => {
   const updateCourseColor = useCallback(
     (pid: string, term: string, color: string) => {
       data.map((item) => {
-        if (item.term == term && item.pid == pid) {
+        if (item.term === term && item.pid === pid) {
           item.color = color;
         }
+        return data;
       });
       setData(data);
     },
-    [data]
+    [data, setData]
   );
   /**
    * Checks the equality of two courses

diff --git a/src/pages/scheduler/components/CourseCard.tsx b/src/pages/scheduler/components/CourseCard.tsx
@@ -1,6 +1,6 @@
 import { useCallback } from 'react';
 
-import { ChevronDownIcon, ChevronUpIcon, CloseIcon, CopyIcon, EditIcon } from '@chakra-ui/icons';
+import { ChevronDownIcon, ChevronUpIcon, CloseIcon, EditIcon } from '@chakra-ui/icons';
 import {
   Box,
   Text,
@@ -96,7 +96,7 @@ export function CourseCard({
     (color: string) => {
       handleColorChange({ term, code, subject, pid, color });
     },
-    [code, handleDelete, pid, subject, term, color]
+    [code, handleColorChange, pid, subject, term]
   );
 
   const onShowSections = useCallback(() => {
@@ -150,11 +150,7 @@ export function CourseCard({
                     <MenuList>
                       {colorList.map((colorOption) => {
                         return (
-                          <MenuItem
-                            icon={<CopyIcon />}
-                            key={colorOption.color}
-                            onClick={() => onColorChange(colorOption.color)}
-                          >
+                          <MenuItem key={colorOption.color} onClick={() => onColorChange(colorOption.color)}>
                             {colorOption.name}
                           </MenuItem>
                         );