fix: update the cance/reset logic in course management

codetapacademy · Jul 19, 2019 · 6a555c5 · 6a555c5
1 parent ff1f7f8
commit 6a555c5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/component/course-edit/course-edit.component.jsx b/src/component/course-edit/course-edit.component.jsx
@@ -1,7 +1,7 @@
 import React from 'react'
 import { StyledActionButtonWrapper, StyledCourseEdit } from './course-edit.style';
 
-const CourseEdit = ({ title, handleTitle, addCourse, courseIdToEdit, description, handleDescription }) => {
+const CourseEdit = ({ title, handleTitle, addCourse, courseIdToEdit, description, handleDescription, handleCancel }) => {
   return (
     <StyledCourseEdit>
       <input
@@ -20,7 +20,7 @@ const CourseEdit = ({ title, handleTitle, addCourse, courseIdToEdit, description
         <button onClick={addCourse}>
           {courseIdToEdit ? 'Save course' : 'Add course'}
         </button>
-        {courseIdToEdit && <button onClick={handleTitle}>
+        {courseIdToEdit && <button onClick={handleCancel}>
           Cancel
         </button>}
       </StyledActionButtonWrapper>

diff --git a/src/component/course-panel/course-panel.component.jsx b/src/component/course-panel/course-panel.component.jsx
@@ -105,6 +105,12 @@ const CoursePanel = () => {
     }
     setTitle(value)
   }
+
+  const handleCancel = () => {
+    setCourseIdToEdit(null)
+    setTitle('')
+    setDescription('')
+  }
 
   const handleDescription = e => {
     const { value } = e.target
@@ -122,6 +128,7 @@ const CoursePanel = () => {
       <CourseEdit
         addCourse={addCourse}
         handleTitle={handleTitle}
+        handleCancel={handleCancel}
         courseIdToEdit={courseIdToEdit}
         description={courseEditValue.description}
         handleDescription={handleDescription}