fix: stop the Finish tab prescribing business process it invented (KAN-323)

frontend/src/components/job/JobFinishTab.vue

@@ -32,9 +32,7 @@
       class="bg-white rounded-lg border border-red-200 p-6 text-center"
     >
       <p class="text-red-700 font-medium">Could not load this job's financials.</p>
-      <p class="text-sm text-slate-600 mt-1">
-        Reload the page. Do not quote the customer a figure until it loads.
-      </p>
+      <p class="text-sm text-slate-600 mt-1">Reload the page.</p>
     </div>
 
     <template v-else-if="summary">
@@ -98,8 +96,11 @@
             data-automation-id="JobFinishTab-over-invoiced"
             class="mt-3 rounded-md bg-amber-50 border border-amber-200 p-3 text-sm text-amber-900"
           >
-            <strong>Over-invoiced by {{ formatCurrency(summary.over_invoiced_excl_gst) }}</strong>
-            (excl GST). Resolve it with a credit note in Xero.
+            <strong
+              >Job appears over-invoiced by
+              {{ formatCurrency(summary.over_invoiced_excl_gst) }}</strong
+            >
+            (excl GST). The job value may be out of date, or invoicing may need correcting.
           </div>
         </section>
 
@@ -118,7 +119,7 @@
         class="bg-white rounded-xl border border-slate-200 p-4"
       >
         <h3 class="text-base font-semibold text-gray-900">Completion checklist</h3>
-        <p class="text-xs text-slate-500 mb-3">Recorded against the job. Never blocks invoicing.</p>
+        <p class="text-xs text-slate-500 mb-3">Self-checklist.</p>
 
         <div class="space-y-2">
           <label
@@ -137,15 +138,6 @@
             <span class="text-slate-800">{{ item.label }}</span>
           </label>
         </div>
-
-        <p
-          v-if="props.pricingMethodology === 'time_materials'"
-          data-automation-id="JobFinishTab-tm-urgency"
-          class="mt-3 text-sm text-slate-600"
-        >
-          On a Time &amp; Materials job the time and materials are what the customer pays. Get them
-          right before invoicing.
-        </p>
       </section>
     </template>
 
@@ -375,14 +367,21 @@ const checklist = reactive<JobCompletionChecklist>({
 })
 const savingChecklistKey = ref<ChecklistItemKey | null>(null)
 
-const checklistItems: Array<{ key: ChecklistItemKey; label: string }> = [
-  { key: 'foreman_signed_off', label: 'Has the foreman signed off the job?' },
-  { key: 'timesheets_collected', label: 'Have you collected the timesheet entries?' },
-  { key: 'materials_checked', label: 'Have you checked the materials on the job?' },
+const allChecklistItems: Array<{ key: ChecklistItemKey; label: string }> = [
+  { key: 'foreman_signed_off', label: 'Has the supervisor signed off the job?' },
+  { key: 'timesheets_collected', label: "Have you collected today's timesheet entries?" },
+  { key: 'materials_checked', label: 'Are all materials used written correctly on the job sheet?' },
   { key: 'customer_called', label: 'Have you called the customer?' },
-  { key: 'released', label: 'Has the job been released?' },
+  { key: 'released', label: 'Has the job been handed over?' },
 ]
 
+// Timesheets only exist to invoice on a T&M job; a quoted job is never asked.
+const checklistItems = computed(() =>
+  props.pricingMethodology === 'time_materials'
+    ? allChecklistItems
+    : allChecklistItems.filter((item) => item.key !== 'timesheets_collected'),
+)
+
 const jobValueLabel = computed(() =>
   props.pricingMethodology === 'fixed_price'
     ? 'Quote total (excl GST)'

frontend/src/components/job/__tests__/JobFinishTab.checklist.test.ts

@@ -60,33 +60,23 @@ describe('JobFinishTab completion checklist', () => {
 
   afterEach(resetFinishTab)
 
-  it('asks the same five questions whatever the pricing methodology', async () => {
-    for (const methodology of ['fixed_price', 'time_materials']) {
-      const wrapper = mountTab(methodology)
-      await flushPromises()
-
-      for (const key of ITEMS) {
-        expect(item(wrapper, key).exists(), `${methodology}/${key}`).toBe(true)
-      }
-
-      resetFinishTab()
-    }
-  })
-
-  it('warns that time and materials are what a T&M customer pays', async () => {
+  it('asks all five questions on a T&M job', async () => {
     const wrapper = mountTab('time_materials')
     await flushPromises()
 
-    expect(wrapper.find('[data-automation-id="JobFinishTab-tm-urgency"]').text()).toContain(
-      'before invoicing',
-    )
+    for (const key of ITEMS) {
+      expect(item(wrapper, key).exists(), key).toBe(true)
+    }
   })
 
-  it('does not show the T&M urgency note on a quoted job', async () => {
+  it('does not ask about timesheets on a quoted job', async () => {
     const wrapper = mountTab('fixed_price')
     await flushPromises()
 
-    expect(wrapper.find('[data-automation-id="JobFinishTab-tm-urgency"]').exists()).toBe(false)
+    expect(item(wrapper, 'timesheets_collected').exists()).toBe(false)
+    for (const key of ITEMS.filter((k) => k !== 'timesheets_collected')) {
+      expect(item(wrapper, key).exists(), key).toBe(true)
+    }
   })
 
   it('reflects ticks already recorded on the job', async () => {