better name for function

Author: gitmaz

src/components/Applicant/index.js

@@ -3,7 +3,7 @@ import React, { useState, useEffect } from 'react';
 /*
   This component is responsible of rendering individual applicant dialogs
  */
-const Applicant = ({ onAddApplicant, onCancelAdd, applicantToEdit, onCancelEdit, isFirstApplicant }) => {
+const Applicant = ({ onAddOrUpdateApplicant, onCancelAdd, applicantToEdit, onCancelEdit, isFirstApplicant }) => {
     const [firstName, setFirstName] = useState('');
     const [lastName, setLastName] = useState('');
     const [mobileNumber, setMobileNumber] = useState('');
@@ -43,13 +43,13 @@ const Applicant = ({ onAddApplicant, onCancelAdd, applicantToEdit, onCancelEdit,
         }
     }, [isPrimary, isEditingPrimary]);
 
-    const handleAddApplicant = () => {
+    const handleAddOrUpdateApplicant  = () => {
 
         if (!validateForm()) {
             return;
         }
 
-       if(onAddApplicant({ firstName, lastName, mobileNumber, email, isPrimary }).status === "failure"){
+       if(onAddOrUpdateApplicant({ firstName, lastName, mobileNumber, email, isPrimary }).status === "failure"){
            setEmailWarning('Emails are not unique amongst applicants.');
        }else{
            setEmailWarning('');
@@ -163,7 +163,7 @@ const Applicant = ({ onAddApplicant, onCancelAdd, applicantToEdit, onCancelEdit,
             </label>
             <br />
             {emailWarning && <div className="Warning-message">{emailWarning}</div>}
-            <button onClick={handleAddApplicant}>Ok</button>
+            <button onClick={handleAddOrUpdateApplicant }>Ok</button>
             <button className="cancel" onClick={handleCancel}>
                 Cancel
             </button>

src/components/Applicants/index.js

@@ -45,7 +45,12 @@ const Applicants = () => {
         }
     }, [applicants]);
 
-    const handleAddApplicant = ({ firstName, lastName, mobileNumber, email, isPrimary }) => {
+    const handleEditApplicant = (applicant) => {
+        setAddingApplicant(false);
+        setEditingApplicant(applicant);
+    };
+    
+    const handleAddOrUpdateApplicant  = ({ firstName, lastName, mobileNumber, email, isPrimary }) => {
         if (hasDuplicates(email, editingApplicant?.id)) {
             return { status: 'failure', reason: 'has duplicates' };
         }
@@ -84,10 +89,7 @@ const Applicants = () => {
         return { status: 'success' };
     };
 
-    const handleEditApplicant = (applicant) => {
-        setAddingApplicant(false);
-        setEditingApplicant(applicant);
-    };
+    
 
     const handleDeleteApplicant = (applicant) => {
         setApplicants((prevApplicants) => prevApplicants.filter((a) => a.id !== applicant.id));
@@ -174,7 +176,7 @@ const Applicants = () => {
                             </div>
                             {(editingApplicant && editingApplicant.id === applicant.id) && (
                                 <Applicant
-                                    onAddApplicant={handleAddApplicant}
+                                    onAddOrUpdateApplicant={handleAddOrUpdateApplicant }
                                     onCancelAdd={handleCancelAdd}
                                     applicantToEdit={editingApplicant}
                                     onCancelEdit={handleCancelEdit}
@@ -185,7 +187,7 @@ const Applicants = () => {
                     {addingApplicant && (
                         <li key="new-applicant" className="Applicant-list-item">
                             <Applicant
-                                onAddApplicant={handleAddApplicant}
+                                onAddOrUpdateApplicant={handleAddOrUpdateApplicant }
                                 onCancelAdd={handleCancelAdd}
                                 isFirstApplicant={applicants.length === 0}
                             />