From c7b3a9453391123370f0fd5ab11e8fe356e31523 Mon Sep 17 00:00:00 2001
From: Patrick Fic <>
Date: Tue, 12 Oct 2021 14:18:09 -0700
Subject: [PATCH] IO-1441 Conversion safety check.

---
 bodyshop_translations.babel                   | 23 ++++++++++++++++++-
 .../jobs-convert-button.component.jsx         |  5 ++++
 .../jobs-detail.page.component.jsx            |  6 ++++-
 client/src/translations/en_us/common.json     |  1 +
 client/src/translations/es/common.json        |  1 +
 client/src/translations/fr/common.json        |  1 +
 6 files changed, 35 insertions(+), 2 deletions(-)

diff --git a/bodyshop_translations.babel b/bodyshop_translations.babel
index 7b07e622e..81517d230 100644
--- a/bodyshop_translations.babel
+++ b/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
@@ -25960,6 +25960,27 @@
                                             </translation>
                                         </translations>
                                     </concept_node>
+                                    <concept_node>
+                                        <name>savebeforeconversion</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>scheduledinchange</name>
                                         <definition_loaded>false</definition_loaded>
diff --git a/client/src/components/jobs-convert-button/jobs-convert-button.component.jsx b/client/src/components/jobs-convert-button/jobs-convert-button.component.jsx
index b0f30aacf..cb68d7d57 100644
--- a/client/src/components/jobs-convert-button/jobs-convert-button.component.jsx
+++ b/client/src/components/jobs-convert-button/jobs-convert-button.component.jsx
@@ -35,6 +35,7 @@ export function JobsConvertButton({
   refetch,
   jobRO,
   insertAuditTrail,
+  parentFormIsFieldsTouched,
 }) {
   const [visible, setVisible] = useState(false);
   const [loading, setLoading] = useState(false);
@@ -43,6 +44,10 @@ export function JobsConvertButton({
   const [form] = Form.useForm();
 
   const handleConvert = async (values) => {
+    if (parentFormIsFieldsTouched()) {
+      alert(t("jobs.labels.savebeforeconversion"));
+      return;
+    }
     setLoading(true);
     const res = await mutationConvertJob({
       variables: { jobId: job.id, ...values },
diff --git a/client/src/pages/jobs-detail/jobs-detail.page.component.jsx b/client/src/pages/jobs-detail/jobs-detail.page.component.jsx
index 75baa16bf..7d305e45c 100644
--- a/client/src/pages/jobs-detail/jobs-detail.page.component.jsx
+++ b/client/src/pages/jobs-detail/jobs-detail.page.component.jsx
@@ -196,7 +196,11 @@ export function JobsDetailPage({
         <PrinterFilled />
         {t("jobs.actions.printCenter")}
       </Button>
-      <JobsConvertButton job={job} refetch={refetch} />
+      <JobsConvertButton
+        job={job}
+        refetch={refetch}
+        parentFormIsFieldsTouched={form.isFieldsTouched}
+      />
       <JobsDetailHeaderActions key="actions" job={job} refetch={refetch} />
       <Button
         type="primary"
diff --git a/client/src/translations/en_us/common.json b/client/src/translations/en_us/common.json
index ba295c6bc..0aa4c41a4 100644
--- a/client/src/translations/en_us/common.json
+++ b/client/src/translations/en_us/common.json
@@ -1526,6 +1526,7 @@
 				"sale_labor": "Sales - Labor",
 				"sale_parts": "Sales - Parts & Sublet",
 				"sales": "Sales",
+				"savebeforeconversion": "You have unsaved changes on the job.  Please save them before converting it. ",
 				"scheduledinchange": "The scheduled in is based off the latest appointment. To change this date,  please schedule or reschedule the job. ",
 				"specialcoveragepolicy": "Special Coverage Policy Applies",
 				"state_tax_amt": "Provincial/State Taxes",
diff --git a/client/src/translations/es/common.json b/client/src/translations/es/common.json
index 679ce223d..949ddfbfa 100644
--- a/client/src/translations/es/common.json
+++ b/client/src/translations/es/common.json
@@ -1526,6 +1526,7 @@
 				"sale_labor": "",
 				"sale_parts": "",
 				"sales": "",
+				"savebeforeconversion": "",
 				"scheduledinchange": "",
 				"specialcoveragepolicy": "",
 				"state_tax_amt": "",
diff --git a/client/src/translations/fr/common.json b/client/src/translations/fr/common.json
index aab0aa3b3..ada933476 100644
--- a/client/src/translations/fr/common.json
+++ b/client/src/translations/fr/common.json
@@ -1526,6 +1526,7 @@
 				"sale_labor": "",
 				"sale_parts": "",
 				"sales": "",
+				"savebeforeconversion": "",
 				"scheduledinchange": "",
 				"specialcoveragepolicy": "",
 				"state_tax_amt": "",