Update autohouse error logging.

release/2022-06-24

Approved-by: Patrick Fic

bodyshop_translations.babel

@@ -1,4 +1,4 @@
-<babeledit_project be_version="2.7.1" version="1.2">
+<babeledit_project version="1.2" be_version="2.7.1">
     <!--
 
     BabelEdit project file
@@ -40046,6 +40046,27 @@
                                             </translation>
                                         </translations>
                                     </concept_node>
+                                    <concept_node>
+                                        <name>jobs_reconcile</name>
+                                        <definition_loaded>false</definition_loaded>
+                                        <description></description>
+                                        <comment></comment>
+                                        <default_text></default_text>
+                                        <translations>
+                                            <translation>
+                                                <language>en-US</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>es-MX</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                            <translation>
+                                                <language>fr-CA</language>
+                                                <approved>false</approved>
+                                            </translation>
+                                        </translations>
+                                    </concept_node>
                                     <concept_node>
                                         <name>lag_time</name>
                                         <definition_loaded>false</definition_loaded>

client/src/components/bill-cm-returns-table/bill-cm-returns-table.component.jsx

@@ -5,7 +5,6 @@ import ReadOnlyFormItemComponent from "../form-items-formatted/read-only-form-it
 import "./bill-cm-returns-table.styles.scss";
 export default function BillCmdReturnsTableComponent({
   form,
-  loadOutstandingReturns,
   returnLoading,
   returnData,
 }) {

client/src/components/bill-form/bill-form.component.jsx

@@ -58,6 +58,7 @@ export function BillFormComponent({
     {},
     bodyshop.imexshopid
   );
+
   const handleVendorSelect = (props, opt) => {
     setDiscount(opt.discount);
 
@@ -140,13 +141,14 @@ export function BillFormComponent({
             onBlur={() => {
               if (form.getFieldValue("jobid") !== null) {
                 loadLines({ variables: { id: form.getFieldValue("jobid") } });
-                if (form.getFieldValue("vendorid") !== null)
+                if (form.getFieldValue("vendorid") !== null) {
                   loadOutstandingReturns({
                     variables: {
                       jobId: form.getFieldValue("jobid"),
                       vendorId: form.getFieldValue("vendorid"),
                     },
                   });
+                }
               }
             }}
           />
@@ -273,12 +275,13 @@ export function BillFormComponent({
                   getFieldValue("jobid") &&
                   getFieldValue("vendorid")
                 ) {
-                  loadOutstandingReturns({
-                    variables: {
-                      jobId: form.getFieldValue("jobid"),
-                      vendorId: form.getFieldValue("vendorid"),
-                    },
-                  });
+                  //Removed as this would cause an additional reload when validating the form on submit and clear the values.
+                  // loadOutstandingReturns({
+                  //   variables: {
+                  //     jobId: form.getFieldValue("jobid"),
+                  //     vendorId: form.getFieldValue("vendorid"),
+                  //   },
+                  // });
                 }
 
                 if (

client/src/components/bill-form/bill-form.container.jsx

@@ -63,7 +63,6 @@ export function BillFormContainer({
       {!billEdit && (
         <BillCmdReturnsTableComponent
           form={form}
-          loadOutstandingReturns={loadOutstandingReturns}
           returnLoading={returnLoading}
           returnData={returnData}
         />

client/src/translations/en_us/common.json

@@ -2380,6 +2380,7 @@
 				"job_costing_ro_date_summary": "Job Costing by RO - Summary",
 				"job_costing_ro_estimator": "Job Costing by Estimator",
 				"job_costing_ro_ins_co": "Job Costing by RO Source",
+				"jobs_reconcile": "Parts/Sublet/Labor Reconciliation",
 				"lag_time": "Lag Time",
 				"open_orders": "Open Orders by Date",
 				"open_orders_csr": "Open Orders by CSR",

client/src/translations/es/common.json

@@ -2380,6 +2380,7 @@
 				"job_costing_ro_date_summary": "",
 				"job_costing_ro_estimator": "",
 				"job_costing_ro_ins_co": "",
+				"jobs_reconcile": "",
 				"lag_time": "",
 				"open_orders": "",
 				"open_orders_csr": "",

client/src/translations/fr/common.json

@@ -2380,6 +2380,7 @@
 				"job_costing_ro_date_summary": "",
 				"job_costing_ro_estimator": "",
 				"job_costing_ro_ins_co": "",
+				"jobs_reconcile": "",
 				"lag_time": "",
 				"open_orders": "",
 				"open_orders_csr": "",

client/src/utils/TemplateConstants.js

@@ -1534,6 +1534,18 @@ export const TemplateList = (type, context) => {
             },
             group: "purchases",
           },
+          jobs_reconcile: {
+            title: i18n.t("reportcenter.templates.jobs_reconcile"),
+            subject: i18n.t("reportcenter.templates.jobs_reconcile"),
+            key: "jobs_reconcile",
+            //idtype: "vendor",
+            disabled: false,
+            rangeFilter: {
+              object: i18n.t("reportcenter.labels.objects.jobs"),
+              field: i18n.t("jobs.fields.date_invoiced"),
+            },
+            group: "jobs",
+          },
         }
       : {}),
     ...(!type || type === "courtesycarcontract"

server/accounting/qbo/qbo-payments.js

@@ -179,7 +179,6 @@ exports.default = async (req, res) => {
         ret.push({ paymentid: payment.id, success: true });
       } catch (error) {
         logger.log("qbo-payment-create-error", "ERROR", req.user.email, {
-          
           error:
             (error && error.authResponse && error.authResponse.body) ||
             (error && error.message),
@@ -217,7 +216,10 @@ exports.default = async (req, res) => {
     res.status(200).json(ret);
   } catch (error) {
     console.log(error);
-    logger.log("qbo-payment-create-error", "ERROR", req.user.email,  { error: error.message, stack: error.stack });
+    logger.log("qbo-payment-create-error", "ERROR", req.user.email, {
+      error: error.message,
+      stack: error.stack,
+    });
     res.status(400).json(error);
   }
 };
@@ -240,7 +242,7 @@ async function InsertPayment(
 
   if (invoices && invoices.length !== 1) {
     throw new Error(
-      `More than 1 invoice with DocNumber ${payment.ro_number} found.`
+      `More than 1 invoice with DocNumber ${payment.job.ro_number} found.`
     );
   }
 

server/data/autohouse.js

@@ -39,7 +39,7 @@ exports.default = async (req, res) => {
   const { bodyshops } = await client.request(queries.GET_AUTOHOUSE_SHOPS);
 
   const specificShopIds = req.body.bodyshopIds; // ['uuid]
-  const { start, end } = req.body; //YYYY-MM-DD
+  const { start, end, skipUpload } = req.body; //YYYY-MM-DD
   const allxmlsToUpload = [];
   const allErrors = [];
   try {
@@ -107,7 +107,7 @@ exports.default = async (req, res) => {
       } catch (error) {
         //Error at the shop level.
         logger.log("autohouse-error-shop", "ERROR", "api", bodyshop.id, {
-          error,
+          ...error,
         });
 
         allErrors.push({
@@ -125,17 +125,29 @@ exports.default = async (req, res) => {
       }
     }
 
-    // for (const xmlObj of allxmlsToUpload) {
-    //   fs.writeFileSync(`./logs/${xmlObj.filename}`, xmlObj.xml);
-    // }
+    if (skipUpload) {
+      for (const xmlObj of allxmlsToUpload) {
+        fs.writeFileSync(`./logs/${xmlObj.filename}`, xmlObj.xml);
+      }
 
-    // res.json(allxmlsToUpload);
-    // return;
+      res.json(allxmlsToUpload);
+      sendServerEmail({
+        subject: `Autohouse Report ${moment().format("MM-DD-YY")}`,
+        text: `Errors: ${allErrors.map((e) => JSON.stringify(e, null, 2))}
+      Uploaded: ${JSON.stringify(
+        allxmlsToUpload.map((x) => ({ filename: x.filename, count: x.count })),
+        null,
+        2
+      )}
+      `,
+      });
+      return;
+    }
 
     let sftp = new Client();
     sftp.on("error", (errors) =>
       logger.log("autohouse-sftp-error", "ERROR", "api", null, {
-        errors,
+        ...errors,
       })
     );
     try {
@@ -160,7 +172,7 @@ exports.default = async (req, res) => {
       //***TODO  Change filing naming when creating the cron job. IM_ShopInternalName_DDMMYYYY_HHMMSS.xml
     } catch (error) {
       logger.log("autohouse-sftp-error", "ERROR", "api", null, {
-        error,
+        ...error,
       });
     } finally {
       sftp.end();
@@ -169,7 +181,7 @@ exports.default = async (req, res) => {
       subject: `Autohouse Report ${moment().format("MM-DD-YY")}`,
       text: `Errors: ${allErrors.map((e) => JSON.stringify(e, null, 2))}
       Uploaded: ${JSON.stringify(
-        allxmlsToUpload.map((x) => x.filename),
+        allxmlsToUpload.map((x) => ({ filename: x.filename, count: x.count })),
         null,
         2
       )}
@@ -185,7 +197,11 @@ const CreateRepairOrderTag = (job, errorCallback) => {
   //Level 2
 
   if (!job.job_totals) {
-    errorCallback({ job, error: { toString: () => "No job totals for RO." } });
+    errorCallback({
+      jobid: job.id,
+      ro_number: job.ro_number,
+      error: { toString: () => "No job totals for RO." },
+    });
     return {};
   }
 
@@ -658,7 +674,7 @@ const CreateRepairOrderTag = (job, errorCallback) => {
       error,
     });
 
-    errorCallback({ job, error });
+    errorCallback({ jobid: job.id, ro_number: job.ro_number, error });
   }
 };
 
@@ -889,7 +905,9 @@ const GenerateDetailLines = (job, line, statuses) => {
     OriginalCost: null,
     OriginalInvoiceNumber: null,
     PriceEach: line.act_price || 0,
-    PartNumber: _.escape(line.oem_partno),
+    PartNumber: line.oem_partno
+      ? line.oem_partno.replace(/[^\x00-\x7F]/g, "")
+      : "",
     ProfitPercent: null,
     PurchaseOrderNumber: null,
     Qty: line.part_qty || 0,

setadmin.js

@@ -0,0 +1,23 @@
+var { admin } = require("./server/firebase/firebase-handler");
+
+const uidToMakeAdmin = "yTvpfkcNnGckLd1JnoXC7bTdvtu1";
+
+admin
+  .auth()
+  .getUser(uidToMakeAdmin)
+  .then((user) => {
+    admin
+      .auth()
+      .setCustomUserClaims(uidToMakeAdmin, {
+        ...user.customClaims,
+        "https://hasura.io/jwt/claims": {
+          "x-hasura-default-role": "admin",
+          "x-hasura-allowed-roles": ["admin"],
+          "x-hasura-user-id": uidToMakeAdmin,
+        },
+        ioadmin: true,
+      })
+      .then(() => console.log("Success."))
+      .catch((error) => console.log("Error updating claims.", error));
+  })
+  .catch((error) => console.log("Error fetching user.", error));