@7365admin1/module-hygiene 4.29.1-staging.12 → 4.29.1-staging.14

package/.changeset/area-import-unit-ids.md

@@ -0,0 +1,5 @@
+---
+"@7365admin1/module-hygiene": patch
+---
+
+Area import: units listed on an imported area are saved. The import passed each unit id as an ObjectId, which the area schema refuses, so every imported area that listed units failed to save.

package/.changeset/checklist-photo-set-finish-approve-reject.md

@@ -0,0 +1,9 @@
+---
+"@7365admin1/module-hygiene": patch
+---
+
+Decide which approve finishes a checklist set from approve/reject, the same rule
+the units list uses, instead of the unit `status` field. Units at rest carry
+"open" or "closed", so the old check never saw a set as finished: a set holding a
+rejected unit could not be completed, and the finishing tick was not recognised.
+No change to when a photo is required, and no schema or data change.

package/.changeset/photo-required-only-on-completion.md

@@ -0,0 +1,9 @@
+---
+"@7365admin1/module-hygiene": patch
+---
+
+Require the completion photo only on the approve that FINISHES a set, not on
+every unit tick. A task in this product is a set: both the web checklist and the
+five field apps tick units one at a time and open the photographs dialog on the
+tick that completes the set. The previous rule demanded a photo on every
+approve, which refused every intermediate tick on both platforms.

package/dist/index.d.ts

@@ -388,6 +388,10 @@ declare function useAreaChecklistRepo(): {
     updateAreaChecklist: (_id: string | ObjectId, value: TAreaChecklistUpdate, session?: ClientSession) => Promise<number>;
     updateAreaChecklistStatus: (_id: string | ObjectId, status: (typeof allowedStatus)[number], session?: ClientSession) => Promise<number>;
     updateAreaChecklistUnits: (_id: string | ObjectId, set: number, unitId: string | ObjectId, value: TAreaChecklistUnitsUpdate, session?: ClientSession) => Promise<number>;
+    readSetCompletionState: (_id: string | ObjectId, set: number, unitId: string) => Promise<{
+        completesSet: boolean;
+        hasEvidence: boolean;
+    }>;
     getMaxSetNumberForArea: (areaId: string | ObjectId, session?: ClientSession) => Promise<any>;
     closeExpiredAreaChecklists: () => Promise<number>;
     pushScheduleTaskSets: (scheduleId: string | ObjectId, serviceType: TAppServiceType, areaId: string | ObjectId, scheduleTaskId: string | ObjectId, newSets: TAreaChecklist[]) => Promise<number>;