fix: saving RB

refactored saving Jobs the same way

Author: maxceem

src/routes/JobForm/index.jsx

@@ -59,22 +59,22 @@ const JobForm = ({ teamId, jobId }) => {
     }
   };
 
-  const getRequestData = (values) => {
-    return {
-      projectId: values.projectId,
-      externalId: values.externalId,
-      description: values.description,
-      title: values.title,
-      startDate: values.startDate,
-      duration: values.duration,
-      numPositions: values.numPositions,
-      resourceType: values.resourceType,
-      rateType: values.rateType,
-      workload: values.workload,
-      skills: values.skills,
-      status: values.status,
-    };
-  };
+  // as we are using `PUT` method (not `PATCH`) we have send ALL the fields
+  // fields which we don't send would become `null` otherwise
+  const getRequestData = (values) => _.pick(values, [
+    'projectId',
+    'externalId',
+    'description',
+    'title',
+    'startDate',
+    'duration',
+    'numPositions',
+    'resourceType',
+    'rateType',
+    'workload',
+    'skills',
+    'status',
+  ]);
 
   useEffect(() => {
     if (skills && job && !options) {

src/routes/ResourceBookingForm/index.jsx

@@ -67,18 +67,19 @@ const ResourceBookingDetails = ({ teamId, resourceBookingId }) => {
     );
   };
 
-  const getRequestData = (values) => {
-    return {
-      projectId: values.projectId,
-      startDate: values.startDate,
-      endDate: values.endDate,
-      memberRate: values.memberRate,
-      customerRate: values.customerRate,
-      status: values.status,
-      userId: values.userId,
-      rateType: values.rateType,
-    };
-  };
+  // as we are using `PUT` method (not `PATCH`) we have send ALL the fields
+  // fields which we don't send would become `null` otherwise
+  const getRequestData = (values) => _.pick(values, [
+    'projectId',
+    'userId',
+    'jobId',
+    'status',
+    'startDate',
+    'endDate',
+    'memberRate',
+    'customerRate',
+    'rateType',
+  ]);
 
   return (
     <Page title="Edit Member Details">