Merged in feature/IO-3722-disable-contact-fortellis (pull request #3286)

IO-3722 Remove delivery date for bypass vehicles.

Approved-by: Dave Richer

client/src/components/shop-info/shop-info.responsibilitycenters.component.jsx

@@ -157,36 +157,36 @@ export function ShopInfoResponsibilityCenterComponent({ bodyshop, form }) {
                 </Col>
                 {HasFeatureAccess({ featureName: "export", bodyshop }) &&
                   ClosingPeriod.treatment === "on" && (
-                    <Col xs={24} sm={12} xl={8}>
+                  <Col xs={24} sm={12} xl={8}>
-                      <Form.Item
+                    <Form.Item
-                        key="ClosingPeriod"
+                      key="ClosingPeriod"
-                        name={["accountingconfig", "ClosingPeriod"]}
+                      name={["accountingconfig", "ClosingPeriod"]}
-                        label={t("bodyshop.fields.closingperiod")}
+                      label={t("bodyshop.fields.closingperiod")}
-                      >
+                    >
-                        <DatePicker.RangePicker format="MM/DD/YYYY" presets={DatePickerRanges} />
+                      <DatePicker.RangePicker format="MM/DD/YYYY" presets={DatePickerRanges} />
-                      </Form.Item>
+                    </Form.Item>
-                    </Col>
+                  </Col>
-                  )}
+                )}
                 {HasFeatureAccess({ featureName: "export", bodyshop }) &&
                   ADPPayroll.treatment === "on" && (
-                    <Col xs={24} sm={12} xl={8}>
+                  <Col xs={24} sm={12} xl={8}>
-                      <Form.Item
+                    <Form.Item
-                        key="companyCode"
+                      key="companyCode"
-                        name={["accountingconfig", "companyCode"]}
+                      name={["accountingconfig", "companyCode"]}
-                        label={t("bodyshop.fields.companycode")}
+                      label={t("bodyshop.fields.companycode")}
-                      >
+                    >
-                        <Input />
+                      <Input />
-                      </Form.Item>
+                    </Form.Item>
-                    </Col>
+                  </Col>
-                  )}
+                )}
                 {HasFeatureAccess({ featureName: "export", bodyshop }) &&
                   ADPPayroll.treatment === "on" && (
-                    <Col xs={24} sm={12} xl={8}>
+                  <Col xs={24} sm={12} xl={8}>
                       <Form.Item key="batchID" name={["accountingconfig", "batchID"]} label={t("bodyshop.fields.batchid")}>
-                        <Input />
+                      <Input />
-                      </Form.Item>
+                    </Form.Item>
-                    </Col>
+                  </Col>
-                  )}
+                )}
                 {HasFeatureAccess({ featureName: "export", bodyshop }) && !hasDMSKey && (
                   <>
                     <Col xs={24} sm={12} xl={8}>
@@ -512,6 +512,15 @@ export function ShopInfoResponsibilityCenterComponent({ bodyshop, form }) {
                 >
                   <InputNumber min={0} max={100} suffix="%" />
                 </Form.Item>
+                {bodyshop.cdk_dealerid && (
+                  <Form.Item
+                    label={t("bodyshop.fields.dms.disablecontact")}
+                    valuePropName="checked"
+                    name={["cdk_configuration", "disablecontact"]}
+                  >
+                    <Switch />
+                  </Form.Item>
+                )}
                 {bodyshop.pbs_serialnumber && (
                   <Form.Item
                     label={t("bodyshop.fields.dms.disablecontactvehiclecreation")}

client/src/components/vehicles-list/vehicles-list.component.jsx

@@ -44,9 +44,7 @@ export default function VehiclesListComponent({ loading, vehicles, total, refetc
       key: "description",
       render: (text, record) => {
         return (
-          <span>{`${record.v_model_yr || ""} ${record.v_make_desc || ""} ${
+          <span>{`${record.v_model_yr || ""} ${record.v_color || ""} ${record.v_make_desc || ""} ${record.v_model_desc || ""} `}</span>
-            record.v_model_desc || ""
-          } ${record.v_color || ""}`}</span>
         );
       }
     },
@@ -111,7 +109,13 @@ export default function VehiclesListComponent({ loading, vehicles, total, refetc
     >
       <ResponsiveTable
         loading={loading}
-        pagination={{ placement: "top", pageSize: currentPageSize, current: currentPage, showSizeChanger: true, total: total }}
+        pagination={{
+          placement: "top",
+          pageSize: currentPageSize,
+          current: currentPage,
+          showSizeChanger: true,
+          total: total
+        }}
         columns={columns}
         mobileColumnKeys={["v_vin", "description", "plate_no"]}
         rowKey="id"

client/src/redux/user/user.sagas.js

@@ -9,14 +9,13 @@ import {
 } from "@firebase/auth";
 import { arrayUnion, doc, getDoc, setDoc, updateDoc } from "@firebase/firestore";
 import { getToken } from "@firebase/messaging";
-import * as Sentry from "@sentry/react";
+// import * as Sentry from "@sentry/react";
 import { notification } from "antd";
 import axios from "axios";
 import i18next from "i18next";
 //import LogRocket from "logrocket";
 import { all, call, delay, put, select, takeLatest } from "redux-saga/effects";
 import {
-  //analytics,
   auth,
   firestore,
   getCurrentUser,

client/src/translations/en_us/common.json

@@ -370,6 +370,7 @@
           "cashierid": "Cashier ID",
           "default_journal": "Default Journal",
           "disablebillwip": "Disable bill WIP for A/P Posting",
+          "disablecontact": "Disable Contact Updates/Creation",
           "disablecontactvehiclecreation": "Disable Contact & Vehicle Updates/Creation",
           "dms_acctnumber": "DMS Account #",
           "dms_control_override": "Static Control # Override",

client/src/translations/es/common.json

@@ -370,6 +370,7 @@
           "cashierid": "",
           "default_journal": "",
           "disablebillwip": "",
+          "disablecontact": "",
           "disablecontactvehiclecreation": "",
           "dms_acctnumber": "",
           "dms_control_override": "",

client/src/translations/fr/common.json

@@ -370,6 +370,7 @@
           "cashierid": "",
           "default_journal": "",
           "disablebillwip": "",
+          "disablecontact": "",
           "disablecontactvehiclecreation": "",
           "dms_acctnumber": "",
           "dms_control_override": "",

server/fortellis/fortellis.js

@@ -15,7 +15,7 @@ const _ = require("lodash");
 const moment = require("moment-timezone");
 
 const replaceSpecialRegex = /[^a-zA-Z0-9 ]+/g;
-
+const bypassCustomerId = "bypass";
 // Helper function to handle FortellisApiError logging
 function handleFortellisApiError(socket, error, functionName, additionalDetails = {}) {
   if (error instanceof FortellisApiError) {
@@ -95,7 +95,8 @@ async function FortellisJobExport({ socket, redisHelpers, txEnvelope, jobid }) {
       defaultFortellisTTL
     );
 
-    let DMSVehCustomer;
+    let DMSVehCustomerFromVehicle;
+    //let DMSVehCustomer;
     if (!DMSVid.newId) {
       CreateFortellisLogEvent(socket, "DEBUG", `{2.1} Querying the Vehicle using the DMSVid: ${DMSVid.vehiclesVehId}`);
       const DMSVeh = await QueryDmsVehicleById({ socket, redisHelpers, JobData, DMSVid });
@@ -106,46 +107,66 @@ async function FortellisJobExport({ socket, redisHelpers, txEnvelope, jobid }) {
         DMSVeh,
         defaultFortellisTTL
       );
+      DMSVehCustomerFromVehicle = DMSVeh?.owners && DMSVeh.owners.find((o) => o.id.assigningPartyId === "CURRENT");
 
-      const DMSVehCustomerFromVehicle =
+      // //Add in contact bypass for Fortellis.
-        DMSVeh?.owners && DMSVeh.owners.find((o) => o.id.assigningPartyId === "CURRENT");
+      // if (!JobData.bodyshop.cdk_configuration.disablecontact) {
+      //   const DMSVehCustomerFromVehicle =
+      //     DMSVeh?.owners && DMSVeh.owners.find((o) => o.id.assigningPartyId === "CURRENT");
 
-      if (DMSVehCustomerFromVehicle?.id && DMSVehCustomerFromVehicle.id.value) {
+      //   if (DMSVehCustomerFromVehicle?.id && DMSVehCustomerFromVehicle.id.value) {
-        CreateFortellisLogEvent(
+      //     CreateFortellisLogEvent(
-          socket,
+      //       socket,
-          "DEBUG",
+      //       "DEBUG",
-          `{2.2} Querying the Customer using the ID from DMSVeh: ${DMSVehCustomerFromVehicle.id.value}`
+      //       `{2.2} Querying the Customer using the ID from DMSVeh: ${DMSVehCustomerFromVehicle.id.value}`
-        );
+      //     );
-        DMSVehCustomer = await QueryDmsCustomerById({
+      //     DMSVehCustomer = await QueryDmsCustomerById({
-          socket,
+      //       socket,
-          redisHelpers,
+      //       redisHelpers,
-          JobData,
+      //       JobData,
-          CustomerId: DMSVehCustomerFromVehicle.id.value
+      //       CustomerId: DMSVehCustomerFromVehicle.id.value
-        });
+      //     });
-        await setSessionTransactionData(
+      //     await setSessionTransactionData(
-          socket.id,
+      //       socket.id,
-          getTransactionType(jobid),
+      //       getTransactionType(jobid),
-          FortellisCacheEnums.DMSVehCustomer,
+      //       FortellisCacheEnums.DMSVehCustomer,
-          DMSVehCustomer,
+      //       DMSVehCustomer,
-          defaultFortellisTTL
+      //       defaultFortellisTTL
-        );
+      //     );
-      }
+      //   }
+      // }
     }
     CreateFortellisLogEvent(socket, "DEBUG", `{2.3} Querying the Customer using the name.`);
+    if (JobData.bodyshop.cdk_configuration.disablecontact) {
+      //Just go straight to posting.
+      await FortellisSelectedCustomer({ socket, redisHelpers, selectedCustomerId: bypassCustomerId, jobid });
+    } else {
+      const DMSCustList = await QueryDmsCustomerByName({ socket, redisHelpers, JobData });
+      await setSessionTransactionData(
+        socket.id,
+        getTransactionType(jobid),
+        FortellisCacheEnums.DMSCustList,
+        DMSCustList,
+        defaultFortellisTTL
+      );
 
-    const DMSCustList = await QueryDmsCustomerByName({ socket, redisHelpers, JobData });
+      socket.emit("fortellis-select-customer",
-    await setSessionTransactionData(
+        //Removed to save one one API call while disputing with fortellis. 
-      socket.id,
+        //   [
-      getTransactionType(jobid),
+        //  // ...(DMSVehCustomer ? [{ ...DMSVehCustomer, vinOwner: true }] : []),
-      FortellisCacheEnums.DMSCustList,
+        //   ...DMSCustList
-      DMSCustList,
+        // ]
-      defaultFortellisTTL
+        DMSVehCustomerFromVehicle ?
-    );
+          DMSCustList.map(c => {
-
+            //if customer id is the same as the current assigned owner on the vehicle id, set it as vinowner true. )
-    socket.emit("fortellis-select-customer", [
+            if (DMSVehCustomerFromVehicle?.id?.value === c.customerId) {
-      ...(DMSVehCustomer ? [{ ...DMSVehCustomer, vinOwner: true }] : []),
+              return { ...c, vinOwner: true }
-      ...DMSCustList
+            } else {
-    ]);
+              return c
+            }
+          }) : DMSCustList
+      );
+    }
   } catch (error) {
     CreateFortellisLogEvent(socket, "ERROR", `Error in FortellisJobExport - ${error} `, {
       error: error.message,
@@ -218,36 +239,40 @@ async function FortellisSelectedCustomer({ socket, redisHelpers, selectedCustome
       });
       return;
     }
-
+    //Bypass only the customer creation. We still need to create the vehicle and update it to post the service history later on.
     let DMSCust;
-    if (selectedCustomerId) {
+    if (!JobData.bodyshop.cdk_configuration.disablecontact) {
-      CreateFortellisLogEvent(socket, "DEBUG", `{3.1} Querying the Customer using Customer ID: ${selectedCustomerId}`);
+      if (selectedCustomerId) {
+        CreateFortellisLogEvent(socket, "DEBUG", `{3.1} Querying the Customer using Customer ID: ${selectedCustomerId}`);
 
-      //Get cust list from Redis. Return the item
+        //Get cust list from Redis. Return the item
-      const DMSCustList =
+        const DMSCustList =
-        (await getSessionTransactionData(socket.id, getTransactionType(jobid), FortellisCacheEnums.DMSCustList)) || [];
+          (await getSessionTransactionData(socket.id, getTransactionType(jobid), FortellisCacheEnums.DMSCustList)) || [];
-      const existingCustomerInDMSCustList = DMSCustList.find((c) => c.customerId === selectedCustomerId);
+        const existingCustomerInDMSCustList = DMSCustList.find((c) => c.customerId === selectedCustomerId);
-      DMSCust = existingCustomerInDMSCustList || {
+        DMSCust = existingCustomerInDMSCustList || {
-        customerId: selectedCustomerId //This is the fall back in case it is the generic customer.
+          customerId: selectedCustomerId //This is the fall back in case it is the generic customer.
-      };
+        };
-      await setSessionTransactionData(
+        await setSessionTransactionData(
-        socket.id,
+          socket.id,
-        getTransactionType(jobid),
+          getTransactionType(jobid),
-        FortellisCacheEnums.DMSCust,
+          FortellisCacheEnums.DMSCust,
-        DMSCust,
+          DMSCust,
-        defaultFortellisTTL
+          defaultFortellisTTL
-      );
+        );
-    } else {
+      } else {
-      CreateFortellisLogEvent(socket, "DEBUG", `{3.2} Creating new customer.`);
+        CreateFortellisLogEvent(socket, "DEBUG", `{3.2} Creating new customer.`);
-      const DMSCustomerInsertResponse = await InsertDmsCustomer({ socket, redisHelpers, JobData });
+        const DMSCustomerInsertResponse = await InsertDmsCustomer({ socket, redisHelpers, JobData });
-      DMSCust = { customerId: DMSCustomerInsertResponse.data };
+        DMSCust = { customerId: DMSCustomerInsertResponse.data };
-      await setSessionTransactionData(
+        await setSessionTransactionData(
-        socket.id,
+          socket.id,
-        getTransactionType(jobid),
+          getTransactionType(jobid),
-        FortellisCacheEnums.DMSCust,
+          FortellisCacheEnums.DMSCust,
-        DMSCust,
+          DMSCust,
-        defaultFortellisTTL
+          defaultFortellisTTL
-      );
+        );
+      }
+    }else{
+      DMSCust = { customerId: bypassCustomerId };
     }
 
     let DMSVeh;
@@ -258,8 +283,12 @@ async function FortellisSelectedCustomer({ socket, redisHelpers, selectedCustome
       DMSVeh = await getSessionTransactionData(socket.id, getTransactionType(jobid), FortellisCacheEnums.DMSVeh);
       CreateFortellisLogEvent(socket, "DEBUG", `{4.3} Updating Existing Vehicle to associate to owner.`);
 
+      //If it's a bypass scenario, skip this all.
       //Check to see if the vehicle needs to be updated - i.e. the owner is not the selected customer.
-      if (!DMSVeh?.owners.find((o) => o.id.value === DMSCust.customerId && o.id.assigningPartyId === "CURRENT")) {
+      if (
+        selectedCustomerId !== bypassCustomerId &&
+        !DMSVeh?.owners.find((o) => o.id.value === DMSCust.customerId && o.id.assigningPartyId === "CURRENT")
+      ) {
         DMSVeh = await UpdateDmsVehicle({
           socket,
           redisHelpers,
@@ -782,12 +811,14 @@ async function InsertDmsVehicle({ socket, redisHelpers, JobData, txEnvelope, DMS
           // "chassis": "",
           // "color": "",
           // "dealerBodyStyle": "",
-          deliveryDate:
+          ...(DMSCust?.customerId !== bypassCustomerId && {
-            txEnvelope.dms_unsold === true
+            deliveryDate:
-              ? ""
+              txEnvelope.dms_unsold === true
-              : moment()
+                ? ""
+                : moment()
                   // .tz(JobData.bodyshop.timezone)
                   .format("YYYY-MM-DD"),
+          }),
           // "deliveryMileage": 4,
           // "doorsQuantity": 4,
           // "engineNumber": "",
@@ -902,14 +933,17 @@ async function InsertDmsVehicle({ socket, redisHelpers, JobData, txEnvelope, DMS
           // "warrantyExpDate": "2015-01-12",
           // "wheelbase": ""
         },
-        owners: [
+        // Owners is not required. Exclude it if we are bypassing.
-          {
+        ...(DMSCust?.customerId !== bypassCustomerId && {
-            id: {
+          owners: [
-              assigningPartyId: "CURRENT",
+            {
-              value: DMSCust.customerId
+              id: {
+                assigningPartyId: "CURRENT",
+                value: DMSCust.customerId
+              }
             }
-          }
+          ]
-        ]
+        })
         //"inventoryAccount": "237"
       }
     });
@@ -1009,12 +1043,14 @@ async function UpdateDmsVehicle({ socket, redisHelpers, JobData, DMSVeh, DMSCust
                 modelAbrev: txEnvelope.dms_model
               }
             : {}),
-          deliveryDate:
+          ...(DMSCust?.customerId !== bypassCustomerId && {
-            txEnvelope.dms_unsold === true
+            deliveryDate:
-              ? ""
+              txEnvelope.dms_unsold === true
-              : moment(DMSVehToSend.vehicle.deliveryDate)
+                ? ""
+                : moment(DMSVehToSend.vehicle.deliveryDate)
                   //.tz(JobData.bodyshop.timezone)
                   .toISOString()
+          })
         },
         owners: ids
       }