fix(core-flows): Unsetting reason on return items (#8470)

packages/admin-next/dashboard/src/routes/orders/order-create-return/components/return-create-form/return-create-form.tsx

@@ -436,34 +436,36 @@ export const ReturnCreateForm = ({
                 className="bg-ui-bg-field mt-4 block h-[56px] w-full rounded-lg border border-dashed"
               />
             )}
-            {items.map((item, index) => (
-              <ReturnItem
-                key={item.id}
-                item={itemsMap.get(item.item_id)!}
-                previewItem={previewItemsMap.get(item.item_id)!}
-                currencyCode={order.currency_code}
-                form={form}
-                onRemove={() => {
-                  const actionId = previewItems
-                    .find((i) => i.id === item.item_id)
-                    ?.actions?.find((a) => a.action === "RETURN_ITEM")?.id
+            {items
+              .filter((item) => !!previewItemsMap.get(item.item_id))
+              .map((item, index) => (
+                <ReturnItem
+                  key={item.id}
+                  item={itemsMap.get(item.item_id)!}
+                  previewItem={previewItemsMap.get(item.item_id)}
+                  currencyCode={order.currency_code}
+                  form={form}
+                  onRemove={() => {
+                    const actionId = previewItems
+                      .find((i) => i.id === item.item_id)
+                      ?.actions?.find((a) => a.action === "RETURN_ITEM")?.id
 
-                  if (actionId) {
-                    removeReturnItem(actionId)
-                  }
-                }}
-                onUpdate={(payload) => {
-                  const actionId = previewItems
-                    .find((i) => i.id === item.item_id)
-                    ?.actions?.find((a) => a.action === "RETURN_ITEM")?.id
+                    if (actionId) {
+                      removeReturnItem(actionId)
+                    }
+                  }}
+                  onUpdate={(payload) => {
+                    const actionId = previewItems
+                      .find((i) => i.id === item.item_id)
+                      ?.actions?.find((a) => a.action === "RETURN_ITEM")?.id
 
-                  if (actionId) {
-                    updateReturnItem({ ...payload, actionId })
-                  }
-                }}
-                index={index}
-              />
-            ))}
+                    if (actionId) {
+                      updateReturnItem({ ...payload, actionId })
+                    }
+                  }}
+                  index={index}
+                />
+              ))}
             {!showPlaceholder && (
               <div className="mt-8 flex flex-col gap-y-4">
                 {/*LOCATION*/}

packages/admin-next/dashboard/src/routes/orders/order-create-return/components/return-create-form/return-item.tsx

@@ -177,8 +177,8 @@ function ReturnItem({
                 className="flex-shrink"
                 variant="transparent"
                 onClick={() => {
-                  onUpdate({ reason_id: null }) // TODO BE: we should be able to set to unset reason here
-                  form.setValue(`items.${index}.reason_id`, "")
+                  onUpdate({ reason_id: null })
+                  form.setValue(`items.${index}.reason_id`, null)
                 }}
               >
                 <XMark className="text-ui-fg-muted" />

packages/core/core-flows/src/order/workflows/return/update-request-item-return.ts

@@ -5,13 +5,13 @@ import {
   OrderWorkflow,
   ReturnDTO,
 } from "@medusajs/types"
-import { ChangeActionType, OrderChangeStatus } from "@medusajs/utils"
+import { ChangeActionType, isDefined, OrderChangeStatus } from "@medusajs/utils"
 import {
-  WorkflowData,
-  WorkflowResponse,
   createStep,
   createWorkflow,
   transform,
+  WorkflowData,
+  WorkflowResponse,
 } from "@medusajs/workflows-sdk"
 import { useRemoteQueryStep } from "../../../common"
 import {
@@ -119,7 +119,9 @@ export const updateRequestItemReturnWorkflow = createWorkflow(
           id: input.action_id,
           details: {
             quantity: data.quantity ?? originalAction.details?.quantity,
-            reason_id: data.reason_id ?? originalAction.details?.reason_id,
+            reason_id: isDefined(data.reason_id)
+              ? data.reason_id
+              : originalAction.details?.reason_id,
           },
           internal_note: data.internal_note,
         }