minor changes ✏️

Mugilan-Codes · Nov 27, 2021 · 2cc0ec7 · 2cc0ec7 · SaiTejaKaragalla · Feb 6, 2022
1 parent f9692aa
commit 2cc0ec7
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 13 deletions.
diff --git a/client/src/components/layout/Spinner.js b/client/src/components/layout/Spinner.js
@@ -1,6 +1,6 @@
 import spinner from './spinner.gif';
 
-export default () => (
+const Spinner = () => (
   <>
     <img
       src={spinner}
@@ -9,3 +9,5 @@ export default () => (
     />
   </>
 );
+
+export default Spinner;
diff --git a/client/src/components/profile-form/AddEducation.js b/client/src/components/profile-form/AddEducation.js
@@ -20,13 +20,14 @@ const AddEducation = ({ addEducation }) => {
   const { school, degree, fieldofstudy, from, to, current, description } =
     formData;
 
-  const handleOnChange = (e) =>
-    setFormData({ ...formData, [e.target.name]: e.target.value });
+  const handleOnChange = (e) => {
+    setFormData({
+      ...formData,
+      [e.target.name]: e.target.value,
+    });
+  };
 
   const handleCheckboxOnChange = (e) => {
-    console.log(e.target.name);
-    console.log(e.target.checked);
-    console.log(e.target.value);
     setFormData({ ...formData, current: !current });
   };
 
@@ -47,7 +48,7 @@ const AddEducation = ({ addEducation }) => {
       <small>* = required field</small>
 
       <form className='form' onSubmit={handleOnSubmit}>
-        <div class='form-group'>
+        <div className='form-group'>
           <input
             type='text'
             placeholder='* School or Bootcamp'
@@ -58,7 +59,7 @@ const AddEducation = ({ addEducation }) => {
           />
         </div>
 
-        <div class='form-group'>
+        <div className='form-group'>
           <input
             type='text'
             placeholder='* Degree or Certificate'
@@ -69,7 +70,7 @@ const AddEducation = ({ addEducation }) => {
           />
         </div>
 
-        <div class='form-group'>
+        <div className='form-group'>
           <input
             type='text'
             placeholder='Field Of Study'

diff --git a/client/src/components/profile-form/AddExperience.js b/client/src/components/profile-form/AddExperience.js
@@ -20,12 +20,12 @@ const AddExperience = ({ addExperience }) => {
   const { company, title, location, from, to, current, description } = formData;
 
   const handleOnChange = (e) =>
-    setFormData({ ...formData, [e.target.name]: e.target.value });
+    setFormData({
+      ...formData,
+      [e.target.name]: e.target.value,
+    });
 
   const handleCheckboxOnChange = (e) => {
-    console.log(e.target.name);
-    console.log(e.target.checked);
-    console.log(e.target.value);
     setFormData({ ...formData, current: !current });
   };
 

diff --git a/client/src/components/profile-form/ProfileForm.js b/client/src/components/profile-form/ProfileForm.js
@@ -5,6 +5,7 @@ import { connect } from 'react-redux';
 
 import { createProfile, getCurrentProfile } from '../../actions/profile';
 
+// TODO: Optimize code
 /*
   NOTE: declare initialState outside of component
   so that it doesn't trigger a useEffect