fix(dashboard,core-flows,types,medusa): Allow editing Order metadata (#11285)

Resolves SUP-780

.changeset/unlucky-pumpkins-nail.md

@@ -0,0 +1,8 @@
+---
+"@medusajs/dashboard": patch
+"@medusajs/core-flows": patch
+"@medusajs/types": patch
+"@medusajs/medusa": patch
+---
+
+fix(dashboard,core-flows,types,medusa): Allow editing Order metadata

packages/admin/dashboard/src/providers/router-provider/route-map.tsx

@@ -355,6 +355,10 @@ export const RouteMap: RouteObject[] = [
                     lazy: () =>
                       import("../../routes/orders/order-edit-billing-address"),
                   },
+                  {
+                    path: "metadata/edit",
+                    lazy: () => import("../../routes/orders/order-metadata"),
+                  },
                 ],
               },
             ],

packages/admin/dashboard/src/routes/orders/order-detail/constants.ts

@@ -6,6 +6,7 @@ const DEFAULT_PROPERTIES = [
   "email",
   "display_id",
   "currency_code",
+  "metadata",
   // --- TOTALS ---
   "total",
   "item_total",

packages/admin/dashboard/src/routes/orders/order-detail/order-detail.tsx

@@ -72,6 +72,7 @@ export const OrderDetail = () => {
       }}
       data={order}
       showJSON
+      showMetadata
       hasOutlet
     >
       <TwoColumnPage.Main>

packages/admin/dashboard/src/routes/orders/order-metadata/index.ts

@@ -0,0 +1 @@
+export { OrderMetadata as Component } from "./order-metadata"

packages/admin/dashboard/src/routes/orders/order-metadata/order-metadata.tsx

@@ -0,0 +1,26 @@
+import { useParams } from "react-router-dom"
+import { MetadataForm } from "../../../components/forms/metadata-form/metadata-form"
+import { useOrder, useUpdateOrder } from "../../../hooks/api"
+
+export const OrderMetadata = () => {
+  const { id } = useParams()
+
+  const { order, isPending, isError, error } = useOrder(id!, {
+    fields: "id,metadata",
+  })
+
+  const { mutateAsync, isPending: isMutating } = useUpdateOrder(order?.id!)
+
+  if (isError) {
+    throw error
+  }
+
+  return (
+    <MetadataForm
+      metadata={order?.metadata}
+      hook={mutateAsync}
+      isPending={isPending}
+      isMutating={isMutating}
+    />
+  )
+}

packages/core/core-flows/src/order/workflows/update-order.ts

@@ -1,4 +1,9 @@
 import { OrderDTO, OrderWorkflow } from "@medusajs/framework/types"
+import {
+  MedusaError,
+  OrderWorkflowEvents,
+  validateEmail,
+} from "@medusajs/framework/utils"
 import {
   WorkflowData,
   WorkflowResponse,
@@ -11,19 +16,14 @@ import {
   RegisterOrderChangeDTO,
   UpdateOrderDTO,
 } from "@medusajs/types"
-import {
-  MedusaError,
-  OrderWorkflowEvents,
-  validateEmail,
-} from "@medusajs/framework/utils"
 
-import { throwIfOrderIsCancelled } from "../utils/order-validation"
+import { emitEventStep, useQueryGraphStep } from "../../common"
 import {
   previewOrderChangeStep,
   registerOrderChangesStep,
   updateOrdersStep,
 } from "../steps"
-import { emitEventStep, useQueryGraphStep } from "../../common"
+import { throwIfOrderIsCancelled } from "../utils/order-validation"
 
 /**
  * The data to validate the order update.
@@ -64,10 +64,7 @@ export type UpdateOrderValidationStepInput = {
  */
 export const updateOrderValidationStep = createStep(
   "update-order-validation",
-  async function ({
+  async function ({ order, input }: UpdateOrderValidationStepInput) {
-    order,
-    input,
-  }: UpdateOrderValidationStepInput) {
     throwIfOrderIsCancelled({ order })
 
     if (
@@ -133,6 +130,7 @@ export const updateOrderWorkflow = createWorkflow(
         "email",
         "shipping_address.*",
         "billing_address.*",
+        "metadata",
       ],
       filters: { id: input.id },
       options: { throwIfKeyNotFound: true },
@@ -223,6 +221,20 @@ export const updateOrderWorkflow = createWorkflow(
           })
         }
 
+        if (input.metadata !== undefined) {
+          changes.push({
+            change_type: "update_order" as const,
+            order_id: input.id,
+            created_by: input.user_id,
+            confirmed_by: input.user_id,
+            details: {
+              type: "metadata",
+              old: order.metadata,
+              new: input.metadata,
+            },
+          })
+        }
+
         return changes
       }
     )

packages/core/types/src/http/order/admin/payload.ts

@@ -11,6 +11,10 @@ export interface AdminUpdateOrder {
    * The order's billing address.
    */
   billing_address?: OrderAddress
+  /**
+   * The order's metadata.
+   */
+  metadata?: Record<string, unknown> | null
 }
 
 export interface AdminCreateOrderFulfillment {

packages/core/types/src/workflow/order/update-order.ts

@@ -26,6 +26,10 @@ export type UpdateOrderWorkflowInput = {
    * The new email of the order.
    */
   email?: string
+  /**
+   * The new metadata of the order.
+   */
+  metadata?: Record<string, unknown> | null
 }
 
 export type UpdateOrderShippingAddressWorkflowInput = {

packages/medusa/src/api/admin/orders/[id]/route.ts

@@ -7,11 +7,11 @@ import {
   MedusaResponse,
 } from "@medusajs/framework/http"
 import { AdminOrder, HttpTypes } from "@medusajs/framework/types"
+import { ContainerRegistrationKeys } from "@medusajs/framework/utils"
 import {
   AdminGetOrdersOrderParamsType,
   AdminUpdateOrderType,
 } from "../validators"
-import { ContainerRegistrationKeys } from "@medusajs/framework/utils"
 
 export const GET = async (
   req: AuthenticatedMedusaRequest<AdminGetOrdersOrderParamsType>,

packages/medusa/src/api/admin/orders/validators.ts

@@ -1,11 +1,11 @@
 import { z } from "zod"
+import { AddressPayload } from "../../utils/common-validators"
 import {
   createFindParams,
   createOperatorMap,
   createSelectParams,
   WithAdditionalData,
 } from "../../utils/validators"
-import { AddressPayload } from "../../utils/common-validators"
 
 export const AdminGetOrdersOrderParams = createSelectParams().merge(
   z.object({
@@ -144,4 +144,5 @@ export const AdminUpdateOrder = z.object({
   email: z.string().optional(),
   shipping_address: AddressPayload.optional(),
   billing_address: AddressPayload.optional(),
+  metadata: z.record(z.unknown()).nullish(),
 })