Compare commits

..
Author SHA1 Message Date
shawn 7dac01acb7 fix: restrict order quantity edits to pending orders
Runner Test / test (push) Successful in 36s
Runner Test / test (pull_request) Successful in 15s
2026-09-29 14:07:12 +10:00
3 changed files with 5 additions and 13 deletions
+3 -1
View File
@@ -1,6 +1,8 @@
import type { OrderStatus } from "@orderops/shared/order-status";
export { canEditOrderItems } from "@orderops/shared/order-status";
export function canEditOrderItems(status: OrderStatus) {
return status === "pending";
}
export function canEditShippingAddress(status: OrderStatus) {
return status === "pending" || status === "processing";
+2 -8
View File
@@ -1,6 +1,5 @@
import { useState } from "react";
import type { OrderDetail } from "@orderops/shared";
import { canEditOrderItems } from "@orderops/shared/order-status";
interface OrderItemsPanelProps {
order: OrderDetail;
@@ -40,14 +39,9 @@ function QuantityForm({
}
export function OrderItemsPanel({ order, onUpdateQuantity }: OrderItemsPanelProps) {
const canEditQuantity = canEditOrderItems(order.status);
return (
<section className="rounded-xl border border-slate-200 bg-white p-4 shadow-sm">
<h3 className="mb-3 text-sm font-semibold text-slate-700">Items</h3>
{!canEditQuantity ? (
<p className="mb-3 text-sm text-slate-600">Item quantities can only be changed while an order is pending.</p>
) : null}
<div className="space-y-3">
{order.items.map((item) => (
<div key={item.id} className="rounded-lg border border-slate-200 p-3">
@@ -57,10 +51,10 @@ export function OrderItemsPanel({ order, onUpdateQuantity }: OrderItemsPanelProp
<div className="text-sm text-slate-600">{item.sku}</div>
<div className="text-sm text-slate-500">Available: {item.availableQuantity}</div>
</div>
{canEditQuantity ? (
{order.status === "pending" ? (
<QuantityForm itemId={item.id} quantity={item.quantity} onSave={onUpdateQuantity} />
) : (
<div className="text-sm text-slate-700">Quantity: {item.quantity}</div>
<div className="text-sm text-slate-600">Quantity: {item.quantity}</div>
)}
</div>
</div>
-4
View File
@@ -1,7 +1,3 @@
export const orderStatuses = ["pending", "processing", "shipped", "cancelled", "refunded"] as const;
export type OrderStatus = (typeof orderStatuses)[number];
export function canEditOrderItems(status: OrderStatus) {
return status === "pending";
}