Setting report object properties to snake case items for API

department-of-veterans-affairs · Mar 4, 2024 · 96d67cc · 96d67cc
1 parent 2d797ab
commit 96d67cc
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 16 deletions.
diff --git a/src/applications/representative-search/api/RepresentativeFinderApi.js b/src/applications/representative-search/api/RepresentativeFinderApi.js
@@ -1,3 +1,5 @@
+/* eslint-disable camelcase */
+
 import {
   fetchAndUpdateSessionExpiration as fetch,
   apiRequest,
@@ -64,17 +66,19 @@ class RepresentativeFinderApi {
 
   static reportResult(newReport) {
     const reportRequestBody = {
-      representativeId: newReport.representativeId,
+      representative_id: newReport.representative_id,
       flags: [],
     };
 
     const startTime = new Date().getTime();
 
-    for (const [flagType, flaggedValue] of Object.entries(newReport.reports)) {
-      if (flaggedValue !== null) {
+    for (const [flag_type, flagged_value] of Object.entries(
+      newReport.reports,
+    )) {
+      if (flagged_value !== null) {
         reportRequestBody.flags.push({
-          flagType,
-          flaggedValue,
+          flag_type,
+          flagged_value,
         });
       }
     }

diff --git a/src/applications/representative-search/components/results/ReportModal.jsx b/src/applications/representative-search/components/results/ReportModal.jsx
@@ -1,10 +1,11 @@
+/* eslint-disable camelcase */
+
 import React, { useState } from 'react';
 import PropTypes from 'prop-types';
 import {
   VaModal,
   VaCheckboxGroup,
 } from '@department-of-veterans-affairs/component-library/dist/react-bindings';
-import { snakeCase } from 'lodash';
 
 const ReportModal = ({
   representativeName,
@@ -19,7 +20,7 @@ const ReportModal = ({
   testReportObject,
 }) => {
   const [reportObject, setReportObject] = useState({
-    phone: null,
+    phone_number: null,
     email: null,
     address: null,
     other: null,
@@ -64,7 +65,7 @@ const ReportModal = ({
         newState.email = checked ? email : null;
         break;
       case '3':
-        newState.phone = checked ? phone : null;
+        newState.phone_number = checked ? phone : null;
         break;
       case '4':
         setOtherIsBlankError(false);
@@ -78,17 +79,15 @@ const ReportModal = ({
   };
 
   const onSubmitModal = () => {
-    const formattedReportObject = { representativeId, reports: {} };
+    const formattedReportObject = {
+      representative_id: representativeId,
+      reports: {},
+    };
 
     // push non-null items to reports object
     Object.keys(reportObject).forEach(prop => {
       if (reportObject[prop] !== null) {
-        if (prop === 'phone') {
-          formattedReportObject.reports[snakeCase('phoneNumber')] =
-            reportObject.phone;
-        } else {
-          formattedReportObject.reports[prop] = reportObject[prop];
-        }
+        formattedReportObject.reports[prop] = reportObject[prop];
       }
     });
 
@@ -103,7 +102,7 @@ const ReportModal = ({
 
     submitRepresentativeReport(formattedReportObject);
     setReportObject({
-      phone: null,
+      phone_number: null,
       email: null,
       address: null,
       other: null,