IO-2568 Button Padding in Print Center Label Modal

Signed-off-by: Allan Carr <allan.carr@thinkimex.com>

client/src/components/payment-mark-export-button/payment-mark-export-button-component.jsx

@@ -7,12 +7,10 @@ import { createStructuredSelector } from "reselect";
 import { INSERT_EXPORT_LOG } from "../../graphql/accounting.queries";
 import { UPDATE_PAYMENT } from "../../graphql/payments.queries";
 import { setModalContext } from "../../redux/modals/modals.actions";
-import { selectPayment } from "../../redux/modals/modals.selectors";
 import { selectCurrentUser } from "../../redux/user/user.selectors";
 
 const mapStateToProps = createStructuredSelector({
   currentUser: selectCurrentUser,
-  paymentModal: selectPayment,
 });
 
 const mapDispatchToProps = (dispatch) => ({
@@ -26,7 +24,6 @@ const PaymentMarkForExportButton = ({
   refetch,
   setPaymentContext,
   currentUser,
-  paymentModal,
 }) => {
   const { t } = useTranslation();
   const [insertExportLog, { loading: exportLogLoading }] =
@@ -68,22 +65,16 @@ const PaymentMarkForExportButton = ({
           refetch,
         },
         context: {
-          ...paymentModal.context,
           ...payment,
           exportedat: today,
         },
       });
 
-      if (refetch) {
-        if (paymentModal.context.refetchRequiresContext) {
-          refetch(
-            paymentUpdateResponse &&
-              paymentUpdateResponse.data.update_payments.returning[0]
-          );
-        } else {
-          refetch();
-        }
-      }
+      if (refetch)
+        refetch(
+          paymentUpdateResponse &&
+            paymentUpdateResponse.data.update_payments.returning[0]
+        );
     } else {
       notification["error"]({
         message: t("payments.errors.exporting", {

client/src/components/payment-modal/payment-modal.container.jsx

@@ -1,5 +1,4 @@
 import { useMutation } from "@apollo/client";
-
 import { Button, Form, Modal, notification, Space } from "antd";
 import React, { useEffect, useState } from "react";
 import { useTranslation } from "react-i18next";
@@ -97,21 +96,16 @@ function PaymentModalContainer({
         });
 
         if (!!!updatedPayment.errors) {
-          notification["success"]({ message: t("payments.successes.paymentupdate") });
+          notification["success"]({ message: t("payments.successes.payment") });
         } else {
-          notification["error"]({ message: t("payments.errors.paymentupdate") });
+          notification["error"]({ message: t("payments.errors.payment") });
         }
       }
 
-      if (actions.refetch) {
-        if (context.refetchRequiresContext) {
-          actions.refetch(
-            updatedPayment && updatedPayment.data.update_payments.returning[0]
-          );
-        } else {
-          actions.refetch();
-        }
-      }
+      if (actions.refetch)
+        actions.refetch(
+          updatedPayment && updatedPayment.data.update_payments.returning[0]
+        );
 
       if (enterAgain) {
         const prev = form.getFieldsValue(["date"]);
@@ -164,7 +158,7 @@ function PaymentModalContainer({
       }}
       afterClose={() => form.resetFields()}
       footer={
-        <span>
+        <Space>
           <Button onClick={handleCancel}>{t("general.actions.cancel")}</Button>
           <Button loading={loading} onClick={() => form.submit()}>
             {t("general.actions.save")}
@@ -180,7 +174,7 @@ function PaymentModalContainer({
               {t("general.actions.saveandnew")}
             </Button>
           )}
-        </span>
+        </Space>
       }
     >
       {!context || (context && !context.id) ? null : (

client/src/components/payment-reexport-button/payment-reexport-button.component.jsx

@@ -3,25 +3,15 @@ import { Button, notification } from "antd";
 import React from "react";
 import { useTranslation } from "react-i18next";
 import { connect } from "react-redux";
-import { createStructuredSelector } from "reselect";
 import { UPDATE_PAYMENT } from "../../graphql/payments.queries";
 import { setModalContext } from "../../redux/modals/modals.actions";
-import { selectPayment } from "../../redux/modals/modals.selectors";
 
-const mapStateToProps = createStructuredSelector({
-  paymentModal: selectPayment,
-});
 const mapDispatchToProps = (dispatch) => ({
   setPaymentContext: (context) =>
     dispatch(setModalContext({ context: context, modal: "payment" })),
 });
 
-const PaymentReexportButton = ({
-  paymentModal,
-  payment,
-  refetch,
-  setPaymentContext,
-}) => {
+const PaymentReexportButton = ({ payment, refetch, setPaymentContext }) => {
   const { t } = useTranslation();
   const [updatePayment, { loading }] = useMutation(UPDATE_PAYMENT);
 
@@ -45,21 +35,15 @@ const PaymentReexportButton = ({
           refetch,
         },
         context: {
-          ...paymentModal.context,
           ...payment,
           exportedat: null,
         },
       });
-      if (refetch) {
-        if (paymentModal.context.refetchRequiresContext) {
-          refetch(
-            paymentUpdateResponse &&
-              paymentUpdateResponse.data.update_payments.returning[0]
-          );
-        } else {
-          refetch();
-        }
-      }
+      if (refetch)
+        refetch(
+          paymentUpdateResponse &&
+            paymentUpdateResponse.data.update_payments.returning[0]
+        );
     } else {
       notification["error"]({
         message: t("payments.errors.exporting", {
@@ -80,7 +64,4 @@ const PaymentReexportButton = ({
   );
 };
 
-export default connect(
-  mapStateToProps,
-  mapDispatchToProps
-)(PaymentReexportButton);
+export default connect(null, mapDispatchToProps)(PaymentReexportButton);

client/src/components/payments-list-paginated/payment-list-paginated.component.jsx

@@ -14,11 +14,11 @@ import { selectBodyshop } from "../../redux/user/user.selectors";
 import CurrencyFormatter from "../../utils/CurrencyFormatter";
 import { DateFormatter, DateTimeFormatter } from "../../utils/DateFormatter";
 import { TemplateList } from "../../utils/TemplateConstants";
-import { pageLimit } from "../../utils/config";
 import { alphaSort } from "../../utils/sorters";
 import CaBcEtfTableModalContainer from "../ca-bc-etf-table-modal/ca-bc-etf-table-modal.container";
 import OwnerNameDisplay from "../owner-name-display/owner-name-display.component";
 import PrintWrapperComponent from "../print-wrapper/print-wrapper.component";
+import {pageLimit} from "../../utils/config";
 
 const mapStateToProps = createStructuredSelector({
   //currentUser: selectCurrentUser
@@ -184,10 +184,7 @@ export function PaymentsListPaginated({
                       }
                     : refetch,
                 },
-                context: {
-                  ...(apolloResults ? apolloResults : record),
-                  refetchRequiresContext: true,
-                },
+                context: apolloResults ? apolloResults : record,
               });
             }}
           >

client/src/components/print-center-jobs-labels/print-center-jobs-labels.component.jsx

@@ -6,6 +6,7 @@ import {
   notification,
   Popover,
   Radio,
+  Space,
 } from "antd";
 import React, { useState } from "react";
 import { useTranslation } from "react-i18next";
@@ -115,10 +116,16 @@ export function PrintCenterJobsLabels({ bodyshop, jobId }) {
         >
           <InputNumber min={1} precision={0} max={99} />
         </Form.Item>
-        <Button type="primary" loading={loading} onClick={handleOk}>
-          {t("general.actions.print")}
-        </Button>
-        <Button onClick={handleCancel}>{t("general.actions.cancel")}</Button>
+        <div style={{ display: "flex", justifyContent: "flex-end" }}>
+          <Space>
+            <Button type="primary" loading={loading} onClick={handleOk}>
+              {t("general.actions.print")}
+            </Button>
+            <Button onClick={handleCancel}>
+              {t("general.actions.cancel")}
+            </Button>
+          </Space>
+        </div>
       </Form>
     </Card>
   );

client/src/translations/en_us/common.json

@@ -2332,7 +2332,6 @@
 				"markexported": "Payment(s) marked exported.",
 				"markreexported": "Payment marked for re-export successfully",
 				"payment": "Payment created successfully. ",
-				"paymentupdate": "Payment updated successfully. ",
 				"stripe": "Credit card transaction charged successfully."
 			}
 		},

client/src/translations/es/common.json

@@ -2331,7 +2331,6 @@
 				"markexported": "",
 				"markreexported": "",
 				"payment": "",
-				"paymentupdate": "",
 				"stripe": ""
 			}
 		},

client/src/translations/fr/common.json

@@ -2331,7 +2331,6 @@
 				"markexported": "",
 				"markreexported": "",
 				"payment": "",
-				"paymentupdate": "",
 				"stripe": ""
 			}
 		},