refactoring, comments

HASKI-RAK · Sep 3, 2024 · 6fa9e17 · 6fa9e17
1 parent 60b11d3
commit 6fa9e17
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 8 deletions.
diff --git a/src/components/CreateCourse/Modal/CreateCourseModal.tsx b/src/components/CreateCourse/Modal/CreateCourseModal.tsx
@@ -23,14 +23,15 @@ const CreateCourseModal = memo(
     setSuccessRemoteCourseCreated,
     handleCloseCreateCourseModal
   }: CreateCourseModalProps) => {
+    //Hooks
     const { t } = useTranslation()
-
     const [selectedRemoteCourse, setSelectedRemoteCourse] = useState<RemoteCourse>()
     const [activeStep, setActiveStep] = useState<number>(0)
     const [courseStartDateValue, setCourseStartDateValue] = useState<Dayjs | null>(dayjs(new Date()))
     const [user, setUser] = useState<User>()
     const [isSending, setIsSending] = useState<boolean>(false)
 
+    //Stores
     const getUser = usePersistedStore((state) => state.getUser)
     const { addSnackbar } = useContext(SnackbarContext)
 

diff --git a/src/components/CreateCourse/Table/CreateCourseDetailsTable.tsx b/src/components/CreateCourse/Table/CreateCourseDetailsTable.tsx
@@ -21,6 +21,7 @@ const CreateCourseDetailsTable = ({
   datePickerValue,
   setDatePickerValue
 }: CreateCourseDetailsTableProps) => {
+  //Hooks
   const { t } = useTranslation()
 
   return (

diff --git a/src/components/CreateCourse/Table/CreateCourseTable.tsx b/src/components/CreateCourse/Table/CreateCourseTable.tsx
@@ -22,13 +22,14 @@ type CreateCourseTableProps = {
 }
 
 const CreateCourseTable = memo(({ onCourseSelect, selectedCourseName }: CreateCourseTableProps) => {
+  //Hooks
   const { t } = useTranslation()
   const { addSnackbar } = useContext(SnackbarContext)
-
   const [remoteLmsCourses, setRemoteLmsCourses] = useState<RemoteCourse[]>([])
   const [alreadyCreatedCourses, setAlreadyCreatedCourses] = useState<RemoteCourse[]>([])
   const [availableCourses, setAvailableCourses] = useState<RemoteCourse[]>([])
 
+  //Stores
   const getCourses = useStore((state) => state.getCourses)
   const getUser = usePersistedStore((state) => state.getUser)
 
@@ -79,11 +80,9 @@ const CreateCourseTable = memo(({ onCourseSelect, selectedCourseName }: CreateCo
       })
   }, [])
 
-  const handleChange = (_event: React.MouseEvent<HTMLElement>, nextView: string) => {
+  const handleSelectedCourseChange = (_event: React.MouseEvent<HTMLElement>, nextView: string) => {
     const selectedCourse = remoteLmsCourses.find((course) => course.fullname === nextView)
-    if (selectedCourse) {
-      onCourseSelect(selectedCourse)
-    }
+    selectedCourse && onCourseSelect(selectedCourse)
   }
 
   //Early return
@@ -114,7 +113,7 @@ const CreateCourseTable = memo(({ onCourseSelect, selectedCourseName }: CreateCo
             orientation="vertical"
             value={selectedCourseName}
             exclusive
-            onChange={handleChange}
+            onChange={handleSelectedCourseChange}
             sx={{
               '& .MuiToggleButtonGroup-grouped:not(:first-of-type)': {
                 borderColor: (theme) => theme.palette.common.black
@@ -158,7 +157,7 @@ const CreateCourseTable = memo(({ onCourseSelect, selectedCourseName }: CreateCo
           <ToggleButtonGroup
             orientation="vertical"
             value={selectedCourseName}
-            onChange={handleChange}
+            onChange={handleSelectedCourseChange}
             sx={{
               '& .MuiToggleButtonGroup-grouped:not(:first-of-type)': {
                 //is the exact same color as disabled togglebutton