Fix type error when returning items (#1525) (#1548)

* optional chaining wip

* resolve property of undefined error

* remove error toast when there is no error

* revert makeMutation

* apply suggestions from code review

* revert newline

* code review suggestions

* remove unnecessary optional chaining

* fix orderRefund undefined errors

src/orders/views/OrderRefund/OrderRefund.tsx

@@ -108,7 +108,7 @@ const OrderRefund: React.FC<OrderRefundProps> = ({ orderId }) => {
       }
     });
 
-    return response.errors || [];
+    return response?.errors || [];
   };
 
   const handleSubmitProductsRefund = async (
@@ -127,7 +127,7 @@ const OrderRefund: React.FC<OrderRefundProps> = ({ orderId }) => {
       }
     });
 
-    return response.errors || [];
+    return response?.errors || [];
   };
 
   const handleSubmit = async (formData: OrderRefundSubmitData) =>

src/orders/views/OrderReturn/OrderReturn.tsx

@@ -5,6 +5,7 @@ import OrderReturnPage from "@saleor/orders/components/OrderReturnPage";
 import { OrderReturnFormData } from "@saleor/orders/components/OrderReturnPage/form";
 import { useOrderReturnCreateMutation } from "@saleor/orders/mutations";
 import { useOrderQuery } from "@saleor/orders/queries";
+import { FulfillmentReturnProducts_orderFulfillmentReturnProducts } from "@saleor/orders/types/FulfillmentReturnProducts";
 import { orderUrl } from "@saleor/orders/urls";
 import { OrderErrorCode } from "@saleor/types/globalTypes";
 import React from "react";
@@ -56,9 +57,11 @@ const OrderReturn: React.FC<OrderReturnProps> = ({ orderId }) => {
         });
 
         navigateToOrder(replaceOrder?.id);
+
+        return;
       }
 
-      if (errors[0].code === OrderErrorCode.CANNOT_REFUND) {
+      if (errors.some(err => err.code === OrderErrorCode.CANNOT_REFUND)) {
         notify({
           autohide: 5000,
           status: "error",
@@ -90,8 +93,10 @@ const OrderReturn: React.FC<OrderReturnProps> = ({ orderId }) => {
     });
 
     const {
-      data: { orderFulfillmentReturnProducts }
-    } = result;
+      data: {
+        orderFulfillmentReturnProducts = {} as FulfillmentReturnProducts_orderFulfillmentReturnProducts
+      } = {}
+    } = result || {};
 
     return orderFulfillmentReturnProducts.errors;
   };