@open-mercato/core 0.7.0 → 0.7.1-develop.7102.1.b41f7e3e51

package/src/modules/sales/components/documents/ReturnsSection.tsx

@@ -12,7 +12,7 @@ import { flash } from '@open-mercato/ui/backend/FlashMessages'
 import { RowActions } from '@open-mercato/ui/backend/RowActions'
 import { useConfirmDialog } from '@open-mercato/ui/backend/confirm-dialog'
 import { useOrganizationScopeDetail } from '@open-mercato/shared/lib/frontend/useOrganizationScope'
-import { useT } from '@open-mercato/shared/lib/i18n/context'
+import { useT, useLocale } from '@open-mercato/shared/lib/i18n/context'
 import {
   emitSalesDocumentTotalsRefresh,
   subscribeSalesDocumentTotalsRefresh,
@@ -41,15 +41,16 @@ type SalesReturnsSectionProps = {
   documentUpdatedAt?: string | null
 }
 
-function formatDisplayDate(value: string | null | undefined): string | null {
+export function formatDisplayDate(value: string | null | undefined, locale?: string): string | null {
   if (!value) return null
   const date = new Date(value)
   if (Number.isNaN(date.getTime())) return null
-  return new Intl.DateTimeFormat(undefined, { dateStyle: 'medium' }).format(date)
+  return new Intl.DateTimeFormat(locale, { dateStyle: 'medium' }).format(date)
 }
 
 export function SalesReturnsSection({ orderId, currencyCode, documentUpdatedAt }: SalesReturnsSectionProps) {
   const t = useT()
+  const locale = useLocale()
   const { organizationId, tenantId } = useOrganizationScopeDetail()
   const { confirm, ConfirmDialogElement } = useConfirmDialog()
   const [returns, setReturns] = React.useState<ReturnRow[]>([])
@@ -328,10 +329,10 @@ export function SalesReturnsSection({ orderId, currencyCode, documentUpdatedAt }
                 </div>
               </div>
               <div className="whitespace-nowrap text-right text-sm text-muted-foreground">
-                {formatDisplayDate(ret.returnedAt) ?? t('sales.returns.notSet', 'Not set')}
+                {formatDisplayDate(ret.returnedAt, locale) ?? t('sales.returns.notSet', 'Not set')}
               </div>
               <div className="whitespace-nowrap text-right text-sm font-medium">
-                {formatMoney(ret.total, currencyCode ?? null)}
+                {formatMoney(ret.total, currencyCode ?? null, locale)}
               </div>
               <div className="flex justify-end">
                 <RowActions

package/src/modules/sales/components/documents/SalesDocumentsTable.tsx

@@ -55,6 +55,7 @@ type ApiDocument = {
   customerEntityId?: string | null
   customerSnapshot?: Record<string, unknown> | null
   channelId?: string | null
+  channelName?: string | null
   lineItemCount?: number | null
   grandTotalNetAmount?: number | null
   grandTotalGrossAmount?: number | null
@@ -70,6 +71,7 @@ type DocumentsResponse = {
   items?: Array<Record<string, unknown>>
   total?: number
   totalPages?: number
+  totalIsCapped?: boolean
 }
 
 type SalesDocumentRow = {
@@ -79,6 +81,7 @@ type SalesDocumentRow = {
   customerName?: string | null
   customerEmail?: string | null
   channelId?: string | null
+  channelName?: string | null
   lineItemCount?: number | null
   totalNet?: number | null
   totalGross?: number | null
@@ -89,6 +92,20 @@ type SalesDocumentRow = {
 
 const PAGE_SIZE = 20
 
+// Not a uuid on purpose: the API drops non-uuid entries from channelIds, so a leak of this sentinel
+// into the id list narrows the filter instead of erroring.
+const UNASSIGNED_CHANNEL_VALUE = '__unassigned__'
+
+// Perspectives persist filterValues, so snapshots saved while this filter was a single-value select
+// hold a bare string. Without this the multi-select would read it as an unknown shape and silently
+// clear the user's saved filter.
+function normalizeChannelSelection(value: unknown): string[] {
+  const raw = Array.isArray(value) ? value : typeof value === 'string' ? [value] : []
+  return raw
+    .map((entry) => (typeof entry === 'string' ? entry.trim() : String(entry || '').trim()))
+    .filter((entry) => entry.length > 0)
+}
+
 function resolveCustomerName(snapshot: CustomerSnapshot | null | undefined, fallback?: string | null) {
   if (!snapshot) return fallback ?? null
   const base = snapshot.customer?.displayName ?? null
@@ -157,6 +174,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
   const [page, setPage] = React.useState(1)
   const [total, setTotal] = React.useState(0)
   const [totalPages, setTotalPages] = React.useState(1)
+  const [totalIsCapped, setTotalIsCapped] = React.useState(false)
   const [sorting, setSorting] = React.useState<SortingState>([{ id: 'createdAt', desc: true }])
   const [search, setSearch] = React.useState('')
   const [filterValues, setFilterValues] = React.useState<FilterValues>({})
@@ -178,9 +196,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
     'Review documents with customer context, totals, and channels.'
   )
 
-  const fetchChannelOptions = React.useCallback(async (query?: string): Promise<FilterOption[]> => {
-    const params = new URLSearchParams({ page: '1', pageSize: '50' })
-    if (query && query.trim()) params.set('search', query.trim())
+  const fetchChannelOptionsBy = React.useCallback(async (params: URLSearchParams): Promise<FilterOption[]> => {
     try {
       const call = await apiCall<{ items?: unknown[] }>(`/api/sales/channels?${params.toString()}`)
       if (!call.ok) return []
@@ -198,6 +214,32 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
     }
   }, [])
 
+  const fetchChannelOptions = React.useCallback(
+    (query?: string) => {
+      const params = new URLSearchParams({ page: '1', pageSize: '50' })
+      if (query && query.trim()) params.set('search', query.trim())
+      return fetchChannelOptionsBy(params)
+    },
+    [fetchChannelOptionsBy]
+  )
+
+  // A selected channel outside the first options page — restored from a perspective, or simply the
+  // 51st channel — would otherwise render as a bare uuid in the filter chip and the selected tag,
+  // which is the same defect this change removes from the column. The route supports `ids=`, so
+  // fetch exactly the selected ones and merge them in. See
+  // `.ai/lessons/async-edit-selects-must-be-hydrated-as-value-plus.md`.
+  const seedChannelOptionsForIds = React.useCallback(
+    async (ids: string[]) => {
+      const missing = ids.filter((id) => id !== UNASSIGNED_CHANNEL_VALUE)
+      if (!missing.length) return
+      const params = new URLSearchParams({ page: '1', pageSize: String(missing.length) })
+      params.set('ids', missing.join(','))
+      const opts = await fetchChannelOptionsBy(params)
+      if (opts.length) setChannelOptions((prev) => mergeOptions(prev, opts))
+    },
+    [fetchChannelOptionsBy]
+  )
+
   const fetchTagOptions = React.useCallback(async (query?: string): Promise<FilterOption[]> => {
     const params = new URLSearchParams({ page: '1', pageSize: '50' })
     if (query && query.trim()) params.set('search', query.trim())
@@ -278,6 +320,32 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
     [fetchChannelOptions]
   )
 
+  const unassignedChannelOption = React.useMemo<FilterOption>(
+    () => ({
+      value: UNASSIGNED_CHANNEL_VALUE,
+      label: t('sales.documents.list.filters.channelUnassigned', '(No channel)'),
+    }),
+    [t]
+  )
+
+  // The sentinel is offered alongside the real channels so "these channels or unassigned" is one
+  // selection. It is stripped back out when the query is built — the API takes it as a separate
+  // channelIdsEmpty flag, not as an id.
+  const channelFilterOptions = React.useMemo<FilterOption[]>(
+    () => [unassignedChannelOption, ...channelOptions],
+    [channelOptions, unassignedChannelOption]
+  )
+
+  const loadChannelFilterOptions = React.useCallback(
+    async (query?: string) => {
+      const opts = await loadChannelOptions(query)
+      // Only offer the sentinel when the user is browsing rather than searching — typing "web"
+      // should not suggest "(No channel)".
+      return query && query.trim() ? opts : [unassignedChannelOption, ...opts]
+    },
+    [loadChannelOptions, unassignedChannelOption]
+  )
+
   const loadTagOptions = React.useCallback(
     async (query?: string) => {
       const opts = await fetchTagOptions(query)
@@ -307,9 +375,14 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
     ...(channelsEnabled ? [{
       id: 'channelId',
       label: t('sales.documents.list.filters.channel', 'Channel'),
-      type: 'select',
-      options: channelOptions,
-      loadOptions: loadChannelOptions,
+      type: 'tags',
+      options: channelFilterOptions,
+      loadOptions: loadChannelFilterOptions,
+      placeholder: t('sales.documents.list.filters.channelPlaceholder', 'Search channels'),
+      formatValue: (val: string) =>
+        val === UNASSIGNED_CHANNEL_VALUE
+          ? t('sales.documents.list.filters.channelUnassigned', '(No channel)')
+          : channelOptions.find((opt) => opt.value === val)?.label ?? val,
     } satisfies FilterDef] : []),
     {
       id: 'date',
@@ -367,7 +440,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
       formatValue: (val: string) => tagOptions.find((o) => o.value === val)?.label ?? val,
       formatDescription: (val: string) => tagOptions.find((o) => o.value === val)?.description ?? null,
     },
-  ], [channelsEnabled, channelOptions, loadChannelOptions, customerOptions, loadCustomerOptions, loadTagOptions, tagOptions, t])
+  ], [channelsEnabled, channelFilterOptions, channelOptions, loadChannelFilterOptions, customerOptions, loadCustomerOptions, loadTagOptions, tagOptions, t])
 
   const queryParams = React.useMemo(() => {
     const params = new URLSearchParams()
@@ -379,8 +452,10 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
       params.set('sortField', sort.id)
       params.set('sortDir', sort.desc ? 'desc' : 'asc')
     }
-    const channelId = typeof filterValues.channelId === 'string' ? filterValues.channelId : ''
-    if (channelId) params.set('channelId', channelId)
+    const channelSelection = normalizeChannelSelection(filterValues.channelId)
+    const channelIds = channelSelection.filter((value) => value !== UNASSIGNED_CHANNEL_VALUE)
+    if (channelIds.length > 0) params.set('channelIds', channelIds.join(','))
+    if (channelSelection.includes(UNASSIGNED_CHANNEL_VALUE)) params.set('channelIdsEmpty', 'true')
     const customerIds = Array.isArray(filterValues.customerId)
       ? filterValues.customerId
           .map((value) => (typeof value === 'string' ? value.trim() : String(value || '').trim()))
@@ -469,6 +544,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
         customerName,
         customerEmail,
         channelId: doc.channelId ?? null,
+        channelName: doc.channelName ?? null,
         lineItemCount: doc.lineItemCount ?? null,
         totalNet,
         totalGross,
@@ -490,6 +566,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
         setRows([])
         setTotal(0)
         setTotalPages(1)
+        setTotalIsCapped(false)
         return
       }
       const payload = call.result ?? {}
@@ -501,6 +578,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
         ? payload.totalPages
         : Math.max(1, Math.ceil(count / PAGE_SIZE))
       setTotalPages(pages)
+      setTotalIsCapped(payload.totalIsCapped === true)
       setCacheStatus(call.cacheStatus ?? null)
     } catch (err) {
       logger.error('sales.documents.list', { err })
@@ -514,10 +592,22 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
     void loadDocuments()
   }, [loadDocuments, reloadToken, scopeVersion])
 
+  // Normalize at the hydration boundary, not only where the query string is built: a perspective
+  // saved while this filter was a single-value select restores `channelId` as a bare string, and
+  // the `tags` control renders nothing for a non-array value. Storing the normalized shape keeps
+  // the control, the active-filter chip and the query builder all reading one shape — otherwise the
+  // list is filtered while the field looks empty, and the next channel the user picks replaces the
+  // restored one instead of joining it.
   const handleFiltersApply = React.useCallback((values: FilterValues) => {
-    setFilterValues(values)
+    if (values.channelId === undefined) {
+      setFilterValues(values)
+    } else {
+      const channelSelection = normalizeChannelSelection(values.channelId)
+      setFilterValues({ ...values, channelId: channelSelection })
+      void seedChannelOptionsForIds(channelSelection)
+    }
     setPage(1)
-  }, [])
+  }, [seedChannelOptionsForIds])
 
   const handleFiltersClear = React.useCallback(() => {
     setFilterValues({})
@@ -625,9 +715,10 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
       cell: ({ row }) => {
         const channelId = row.original.channelId
         if (!channelId) return <span className="text-xs text-muted-foreground">{t('sales.documents.list.table.unassigned', 'Unassigned')}</span>
-        const channel = channelOptions.find((opt) => opt.value === channelId)
+        // Server-resolved, so this no longer depends on the channel being inside the options page.
+        // The uuid remains only for a channel the list response could not resolve.
         return (
-          <span className="text-sm">{channel?.label ?? channelId}</span>
+          <span className="text-sm">{row.original.channelName ?? channelId}</span>
         )
       },
       enableSorting: false,
@@ -665,7 +756,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
           ? <span className="text-xs text-muted-foreground">{new Date(row.original.date).toLocaleString()}</span>
           : <span className="text-xs text-muted-foreground">—</span>,
     },
-  ], [channelsEnabled, channelOptions, kind, statusMap, t])
+  ], [channelsEnabled, kind, statusMap, t])
 
   const emptyLabel = kind === 'order'
     ? t('sales.documents.list.table.emptyOrders', 'No orders yet.')
@@ -713,6 +804,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
             pageSize: PAGE_SIZE,
             total,
             totalPages,
+            totalIsCapped,
             onPageChange: setPage,
             cacheStatus,
           }}

package/src/modules/sales/components/documents/SalesOrderDraftLines.tsx

@@ -7,7 +7,7 @@ import { DataTable } from '@open-mercato/ui/backend/DataTable'
 import { RowActions } from '@open-mercato/ui/backend/RowActions'
 import { EmptyState } from '@open-mercato/ui/primitives/empty-state'
 import { Button } from '@open-mercato/ui/primitives/button'
-import { useT } from '@open-mercato/shared/lib/i18n/context'
+import { useLocale, useT } from '@open-mercato/shared/lib/i18n/context'
 import { LineItemDialog } from './LineItemDialog'
 import type { SalesLineRecord } from './lineItemTypes'
 import { formatMoney, normalizeNumber } from './lineItemUtils'
@@ -93,6 +93,7 @@ export function SalesOrderDraftLines({
   onChange,
 }: SalesOrderDraftLinesProps) {
   const t = useT()
+  const locale = useLocale()
   const [dialogOpen, setDialogOpen] = React.useState(false)
   const [editing, setEditing] = React.useState<SalesOrderLineDraft | null>(null)
 
@@ -112,14 +113,14 @@ export function SalesOrderDraftLines({
     {
       id: 'unitPrice',
       header: t('sales.documents.items.table.unit', 'Unit price'),
-      cell: ({ row }) => formatMoney(row.original.record.unitPriceGross, row.original.record.currencyCode ?? currencyCode ?? undefined),
+      cell: ({ row }) => formatMoney(row.original.record.unitPriceGross, row.original.record.currencyCode ?? currencyCode ?? undefined, locale),
     },
     {
       id: 'total',
       header: t('sales.documents.items.table.total', 'Total'),
-      cell: ({ row }) => formatMoney(row.original.record.totalGross, row.original.record.currencyCode ?? currencyCode ?? undefined),
+      cell: ({ row }) => formatMoney(row.original.record.totalGross, row.original.record.currencyCode ?? currencyCode ?? undefined, locale),
     },
-  ], [currencyCode, t])
+  ], [currencyCode, locale, t])
 
   const openCreate = React.useCallback(() => {
     setEditing(null)
@@ -161,7 +162,7 @@ export function SalesOrderDraftLines({
           {t('sales.documents.items.add', 'Add item')}
         </Button>
       </div>
-      {error ? <p className="text-sm text-destructive" role="alert">{error}</p> : null}
+      {error ? <p className="text-sm text-status-error-text" role="alert">{error}</p> : null}
       <DataTable
         columns={columns}
         data={lines}

package/src/modules/sales/components/documents/ShipmentDialog.tsx

@@ -20,7 +20,7 @@ import { flash } from '@open-mercato/ui/backend/FlashMessages'
 import { cn } from '@open-mercato/shared/lib/utils'
 import { E } from '#generated/entities.ids.generated'
 import { emitSalesDocumentTotalsRefresh } from '@open-mercato/core/modules/sales/lib/frontend/documentTotalsEvents'
-import { useT } from '@open-mercato/shared/lib/i18n/context'
+import { useT, useLocale } from '@open-mercato/shared/lib/i18n/context'
 import { formatMoney, normalizeNumber } from './lineItemUtils'
 import type { OrderLine, ShipmentRow } from './shipmentTypes'
 import { formatAddressString, type AddressFormatStrategy, type AddressValue } from '@open-mercato/core/modules/customers/utils/addressFormat'
@@ -290,6 +290,7 @@ export function ShipmentDialog({
   onAddComment,
 }: ShipmentDialogProps) {
   const t = useT()
+  const locale = useLocale()
   const [formResetKey, setFormResetKey] = React.useState(0)
   const [shippingMethods, setShippingMethods] = React.useState<ShippingMethodOption[]>([])
   const [shippingMethodLoading, setShippingMethodLoading] = React.useState(false)
@@ -540,21 +541,21 @@ export function ShipmentDialog({
       if (option.avgPrice !== null) {
         parts.push(
           t('sales.documents.shipments.shippingMethodAvg', 'Avg {{price}}', {
-            price: formatMoney(option.avgPrice, currency ?? null),
+            price: formatMoney(option.avgPrice, currency ?? null, locale),
           }),
         )
       }
       if (option.minPrice !== null) {
         parts.push(
           t('sales.documents.shipments.shippingMethodMin', 'Min {{price}}', {
-            price: formatMoney(option.minPrice, currency ?? null),
+            price: formatMoney(option.minPrice, currency ?? null, locale),
           }),
         )
       }
       if (parts.length === 0) return undefined
       return parts.join(' · ')
     },
-    [currencyCode, t],
+    [currencyCode, locale, t],
   )
 
   const mapShippingMethod = React.useCallback(

package/src/modules/sales/components/documents/ShipmentsSection.tsx

@@ -10,7 +10,7 @@ import { apiCall, withScopedApiRequestHeaders } from '@open-mercato/ui/backend/u
 import { buildOptimisticLockHeader } from '@open-mercato/ui/backend/utils/optimisticLock'
 import { deleteCrud } from '@open-mercato/ui/backend/utils/crud'
 import { useOrganizationScopeDetail } from '@open-mercato/shared/lib/frontend/useOrganizationScope'
-import { useT } from '@open-mercato/shared/lib/i18n/context'
+import { useT, useLocale } from '@open-mercato/shared/lib/i18n/context'
 import { useConfirmDialog } from '@open-mercato/ui/backend/confirm-dialog'
 import {
   emitSalesDocumentTotalsRefresh,
@@ -41,11 +41,11 @@ type SalesShipmentsSectionProps = {
   onAddComment?: (body: string) => Promise<void>
 }
 
-function formatDisplayDate(value: string | null | undefined): string | null {
+export function formatDisplayDate(value: string | null | undefined, locale?: string): string | null {
   if (!value) return null
   const date = new Date(value)
   if (Number.isNaN(date.getTime())) return null
-  return new Intl.DateTimeFormat(undefined, { dateStyle: 'medium' }).format(date)
+  return new Intl.DateTimeFormat(locale, { dateStyle: 'medium' }).format(date)
 }
 
 const formatShipmentAddress = (metadata?: Record<string, unknown> | null): string | null => {
@@ -123,6 +123,7 @@ export function SalesShipmentsSection({
   onAddComment,
 }: SalesShipmentsSectionProps) {
   const t = useT()
+  const locale = useLocale()
   const { organizationId, tenantId } = useOrganizationScopeDetail()
   const { confirm, ConfirmDialogElement } = useConfirmDialog()
   const resolvedOrganizationId = organizationIdProp ?? organizationId ?? null
@@ -486,8 +487,8 @@ export function SalesShipmentsSection({
       ) : (
         <div className="grid gap-3 md:grid-cols-2">
           {shipments.map((shipment) => {
-            const shippedAt = formatDisplayDate(shipment.shippedAt)
-            const deliveredAt = formatDisplayDate(shipment.deliveredAt)
+            const shippedAt = formatDisplayDate(shipment.shippedAt, locale)
+            const deliveredAt = formatDisplayDate(shipment.deliveredAt, locale)
             const addressSummary = formatShipmentAddress(shipment.metadata)
             const statusLabel =
               shipment.statusLabel ??

package/src/modules/sales/components/documents/lineItemShipmentLock.ts

@@ -0,0 +1,74 @@
+type ShippedLineSnapshot = {
+  quantity: number;
+  totalNetAmount?: number | null;
+  totalGrossAmount?: number | null;
+};
+
+const SHIPPED_LINE_IMMUTABLE_PAYLOAD_FIELDS = [
+  "kind",
+  "productId",
+  "productVariantId",
+  "quantityUnit",
+  "unitPriceNet",
+  "unitPriceGross",
+  "priceId",
+  "priceMode",
+  "taxRateId",
+  "taxRate",
+  "taxAmount",
+  "discountAmount",
+  "discountPercent",
+  "catalogSnapshot",
+  "metadata",
+] as const;
+
+function scaleTotal(
+  total: number | null | undefined,
+  previousQuantity: number,
+  nextQuantity: number,
+): number | undefined {
+  if (
+    !Number.isFinite(total) ||
+    !Number.isFinite(previousQuantity) ||
+    previousQuantity <= 0
+  ) {
+    return undefined;
+  }
+  return (total as number) * (nextQuantity / previousQuantity);
+}
+
+export function prepareShippedLineUpdatePayload(
+  payload: Record<string, unknown>,
+  currentLine: ShippedLineSnapshot | null,
+): Record<string, unknown> {
+  if (!currentLine) return payload;
+
+  const nextPayload = { ...payload };
+  for (const field of SHIPPED_LINE_IMMUTABLE_PAYLOAD_FIELDS) {
+    delete nextPayload[field];
+  }
+
+  delete nextPayload.totalNetAmount;
+  delete nextPayload.totalGrossAmount;
+
+  const nextQuantity = Number(nextPayload.quantity);
+  if (!Number.isFinite(nextQuantity) || nextQuantity === currentLine.quantity) {
+    return nextPayload;
+  }
+
+  const scaledNetTotal = scaleTotal(
+    currentLine.totalNetAmount,
+    currentLine.quantity,
+    nextQuantity,
+  );
+  const scaledGrossTotal = scaleTotal(
+    currentLine.totalGrossAmount,
+    currentLine.quantity,
+    nextQuantity,
+  );
+  if (scaledNetTotal !== undefined) nextPayload.totalNetAmount = scaledNetTotal;
+  if (scaledGrossTotal !== undefined)
+    nextPayload.totalGrossAmount = scaledGrossTotal;
+
+  return nextPayload;
+}

package/src/modules/sales/components/documents/lineItemUtils.ts

@@ -9,9 +9,14 @@ export function normalizeNumber(value: unknown, fallback = 0): number {
   return fallback
 }
 
-export function formatMoney(value: number, currency: string | null | undefined): string {
+/**
+ * Pass `locale` to format in the application locale (`useLocale()` in client components).
+ * Omitting it keeps the runtime default, which varies per machine and is therefore not
+ * assertable in tests.
+ */
+export function formatMoney(value: number, currency: string | null | undefined, locale?: string): string {
   if (!currency) return value.toFixed(2)
-  return new Intl.NumberFormat(undefined, { style: 'currency', currency }).format(value)
+  return new Intl.NumberFormat(locale, { style: 'currency', currency }).format(value)
 }
 
 export type LineDiscountDisplay = {

package/src/modules/sales/data/entities.ts

@@ -1790,6 +1790,9 @@ export class SalesPaymentAllocation {
 
 @Entity({ tableName: 'sales_notes' })
 @Index({ name: 'sales_notes_scope_idx', properties: ['organizationId', 'tenantId'] })
+@Index({ name: 'sales_notes_context_idx', properties: ['contextId'] })
+@Index({ name: 'sales_notes_order_idx', properties: ['order'] })
+@Index({ name: 'sales_notes_quote_idx', properties: ['quote'] })
 export class SalesNote {
   @PrimaryKey({ type: 'uuid', defaultRaw: 'gen_random_uuid()' })
   id!: string
@@ -1836,6 +1839,9 @@ export class SalesNote {
 
 @Entity({ tableName: 'sales_document_addresses' })
 @Index({ name: 'sales_document_addresses_scope_idx', properties: ['organizationId', 'tenantId'] })
+@Index({ name: 'sales_document_addresses_document_idx', properties: ['documentId'] })
+@Index({ name: 'sales_document_addresses_order_idx', properties: ['order'] })
+@Index({ name: 'sales_document_addresses_quote_idx', properties: ['quote'] })
 export class SalesDocumentAddress {
   @PrimaryKey({ type: 'uuid', defaultRaw: 'gen_random_uuid()' })
   id!: string

package/src/modules/sales/extension-points.ts

@@ -5,6 +5,7 @@ export const extensionPoints = defineModuleExtensionPoints({
   hosts: {
     ordersTable: dataTableExtensionHost({ tableId: 'sales.orders', source: 'components/documents/SalesDocumentsTable.tsx' }),
     quotesTable: dataTableExtensionHost({ tableId: 'sales.quotes', source: 'components/documents/SalesDocumentsTable.tsx' }),
+    paymentsTable: dataTableExtensionHost({ tableId: 'sales.payments', source: 'components/documents/PaymentsSection.tsx' }),
     orderItemColumns: injectionExtensionHost({
       family: 'detail',
       spotId: 'data-table:sales.order.items:columns',

package/src/modules/sales/i18n/de.json

@@ -1016,6 +1016,7 @@
   "sales.documents.items.errorProductRequired": "Wähle ein Produkt aus, um fortzufahren.",
   "sales.documents.items.errorQuantity": "Die Menge muss größer als 0 sein.",
   "sales.documents.items.errorQuantityBelowShipped": "Die Menge kann nicht unter die bereits versandten {{shipped}} gesenkt werden.",
+  "sales.documents.items.errorQuantityShipmentsUnknown": "Die Menge kann erst gesenkt werden, wenn die Sendungen dieses Auftrags geladen wurden. Öffnen Sie den Auftrag erneut, um es noch einmal zu versuchen.",
   "sales.documents.items.errorSave": "Position konnte nicht gespeichert werden.",
   "sales.documents.items.errorScope": "Organisation und Mandant sind erforderlich.",
   "sales.documents.items.errorUnitPrice": "Der Stückpreis muss größer als 0 sein.",
@@ -1059,6 +1060,8 @@
   "sales.documents.items.save": "Änderungen speichern",
   "sales.documents.items.selectedProduct": "Ausgewählt {{id}}",
   "sales.documents.items.selectedVariant": "Ausgewählt {{id}}",
+  "sales.documents.items.shippedLineLockPending": "Die Preisangaben sind gesperrt, bis die Sendungen dieses Auftrags geladen wurden. Öffnen Sie den Auftrag erneut, um es noch einmal zu versuchen — Name und eine Erhöhung der Menge können weiterhin bearbeitet werden.",
+  "sales.documents.items.shippedLineLocked": "Die Preisangaben dieser Position sind gesperrt, da bereits Artikel versandt wurden. Name und Menge können weiterhin bearbeitet werden.",
   "sales.documents.items.status": "Status",
   "sales.documents.items.statusEmpty": "Kein Status",
   "sales.documents.items.statusLoading": "Status werden geladen…",
@@ -1093,6 +1096,8 @@
   "sales.documents.items.variantSearch": "Variante suchen",
   "sales.documents.list.errors.load": "Dokumente konnten nicht geladen werden.",
   "sales.documents.list.filters.channel": "Kanal",
+  "sales.documents.list.filters.channelPlaceholder": "Kanäle suchen",
+  "sales.documents.list.filters.channelUnassigned": "(Kein Kanal)",
   "sales.documents.list.filters.customer": "Kunden-ID",
   "sales.documents.list.filters.customerPlaceholder": "Kunden-ID",
   "sales.documents.list.filters.date": "Datum",
@@ -1321,13 +1326,16 @@
   "sales.quotes.email.subject": "Angebot {quoteNumber}",
   "sales.quotes.email.total": "Gesamt: {amount} {currency}",
   "sales.quotes.email.validUntil": "Gültig bis: {date}",
+  "sales.quotes.exchange_rate_unsupported": "Der Wechselkurs ist für Angebote nicht verfügbar.",
   "sales.quotes.form.general": "Allgemein",
   "sales.quotes.form.lines": "Positionen",
   "sales.quotes.form.notes": "Notizen",
+  "sales.quotes.fulfillment_status_unsupported": "Der Fulfillment-Status ist für Angebote nicht verfügbar.",
   "sales.quotes.internal_notes_unsupported": "Interne Notizen sind für Angebote nicht verfügbar.",
   "sales.quotes.lines.delete": "Angebotsposition löschen",
   "sales.quotes.lines.upsert": "Angebotsposition aktualisieren",
   "sales.quotes.list.title": "Angebote",
+  "sales.quotes.payment_status_unsupported": "Der Zahlungsstatus ist für Angebote nicht verfügbar.",
   "sales.quotes.public.acceptButton": "Angebot annehmen",
   "sales.quotes.public.acceptFailed": "Angebot konnte nicht angenommen werden.",
   "sales.quotes.public.acceptedMessage": "Auftrag erstellt: {{orderNumber}}",

package/src/modules/sales/i18n/en.json

@@ -1016,6 +1016,7 @@
   "sales.documents.items.errorProductRequired": "Select a product to continue.",
   "sales.documents.items.errorQuantity": "Quantity must be greater than 0.",
   "sales.documents.items.errorQuantityBelowShipped": "You cannot lower the quantity below the {{shipped}} already shipped.",
+  "sales.documents.items.errorQuantityShipmentsUnknown": "The quantity cannot be lowered until this order's shipments have been read. Reopen the order to try again.",
   "sales.documents.items.errorSave": "Failed to save line.",
   "sales.documents.items.errorScope": "Organization and tenant are required.",
   "sales.documents.items.errorUnitPrice": "Unit price must be greater than 0.",
@@ -1059,6 +1060,8 @@
   "sales.documents.items.save": "Save changes",
   "sales.documents.items.selectedProduct": "Selected {{id}}",
   "sales.documents.items.selectedVariant": "Selected {{id}}",
+  "sales.documents.items.shippedLineLockPending": "Pricing is locked until this order's shipments have been read. Reopen the order to try again — you can still edit the name and raise the quantity.",
+  "sales.documents.items.shippedLineLocked": "Pricing is locked on this line because it already has shipped items. You can still edit the name and quantity.",
   "sales.documents.items.status": "Status",
   "sales.documents.items.statusEmpty": "No status",
   "sales.documents.items.statusLoading": "Loading statuses…",
@@ -1093,6 +1096,8 @@
   "sales.documents.items.variantSearch": "Search variant",
   "sales.documents.list.errors.load": "Failed to load documents.",
   "sales.documents.list.filters.channel": "Channel",
+  "sales.documents.list.filters.channelPlaceholder": "Search channels",
+  "sales.documents.list.filters.channelUnassigned": "(No channel)",
   "sales.documents.list.filters.customer": "Customer id",
   "sales.documents.list.filters.customerPlaceholder": "Customer id",
   "sales.documents.list.filters.date": "Date",
@@ -1321,13 +1326,16 @@
   "sales.quotes.email.subject": "Quote {quoteNumber}",
   "sales.quotes.email.total": "Total: {amount} {currency}",
   "sales.quotes.email.validUntil": "Valid until: {date}",
+  "sales.quotes.exchange_rate_unsupported": "Exchange rate is not available on quotes.",
   "sales.quotes.form.general": "General",
   "sales.quotes.form.lines": "Line Items",
   "sales.quotes.form.notes": "Notes",
+  "sales.quotes.fulfillment_status_unsupported": "Fulfillment status is not available on quotes.",
   "sales.quotes.internal_notes_unsupported": "Internal notes are not available on quotes.",
   "sales.quotes.lines.delete": "Delete quote line",
   "sales.quotes.lines.upsert": "Update quote line",
   "sales.quotes.list.title": "Quotes",
+  "sales.quotes.payment_status_unsupported": "Payment status is not available on quotes.",
   "sales.quotes.public.acceptButton": "Accept quote",
   "sales.quotes.public.acceptFailed": "Failed to accept quote.",
   "sales.quotes.public.acceptedMessage": "Order created: {{orderNumber}}",