From 4e54aa7cfe2b8bf09e941ef644d736a1af58cc53 Mon Sep 17 00:00:00 2001 From: Codex Review Integration Date: Wed, 16 Sep 2026 16:56:26 -0300 Subject: [PATCH] chore(work-orders): remove ticket keys from completion docs --- .../workorders/_hooks/use-work-orders-list-page.ts | 2 +- .../work-orders/utils/completion-doc-auto-complete.ts | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/app/(protected)/workorders/_hooks/use-work-orders-list-page.ts b/src/app/(protected)/workorders/_hooks/use-work-orders-list-page.ts index 08de1f5e..182a98e5 100644 --- a/src/app/(protected)/workorders/_hooks/use-work-orders-list-page.ts +++ b/src/app/(protected)/workorders/_hooks/use-work-orders-list-page.ts @@ -118,7 +118,7 @@ export function useWorkOrdersListPage() { return; } try { - // SH-341: generation is the one automatic completion — no confirmation dialog. + // Generation is the one automatic completion — no confirmation dialog. await patchFieldAsync(row.id, { status: "Completed" }, saved.row); } catch { // The document is saved; the table mutation already surfaced the status error and diff --git a/src/domain/work-orders/utils/completion-doc-auto-complete.ts b/src/domain/work-orders/utils/completion-doc-auto-complete.ts index 2337c8bd..e27f0318 100644 --- a/src/domain/work-orders/utils/completion-doc-auto-complete.ts +++ b/src/domain/work-orders/utils/completion-doc-auto-complete.ts @@ -6,9 +6,9 @@ import { } from "@/domain/work-orders/utils/wo-closability"; /** - * SH-115 / SH-341: completion is manual with confirmation, except that generating the - * completion document completes the WO — but only when the SH-114 closability gate passes - * on the persisted row. Not-required (NN) and drafts never complete; Past Due rows must be + * Completion is manual with confirmation, except that generating the completion document + * completes the WO — but only when the closability gate passes on the persisted row. + * Not-required (NN) and drafts never complete; Past Due rows must be * rescheduled before any status change, so they are left for the dispatcher. */ export function shouldCompleteAfterCompletionDoc(