@7365admin1/layer-common 4.136.0 → 4.138.0

package/components/SiteSettings.vue

@@ -541,11 +541,6 @@ const saveANPRSettings = async () => {
   try {
     isSaving.value = true;
 
-    // Replace this with your actual API endpoint / store action
-    // await $fetch(`/api/sites/${siteId.value}/camera-settings`, {
-    //   method: 'POST',
-    //   body: cameraSettings.value
-    // })
     const data = {
       site: props.siteId,
       ANPRSwitches: ANPRSwitches.value,
@@ -556,6 +551,11 @@ const saveANPRSettings = async () => {
     show("Saved successfully", "success");
   } catch (error) {
     console.error("Failed to save settings:", error);
+    show(
+      (error as any)?.data?.message ||
+        "Could not save the camera (ANPR) settings. Please try again.",
+      "error"
+    );
   } finally {
     isSaving.value = false;
   }

package/components/StatusChip.vue

@@ -104,7 +104,7 @@ const toneClass = computed(() =>
   font-size: var(--fs-chip);
 }
 
-/* The role tag: `padding:3px 10px;border-radius:999px;font-size:11.5px`. */
+/* The role tag: `padding:3px 10px;border-radius:999px;font-size:12.5px`. */
 .status-chip--pill {
   padding: 3px 10px;
   font-size: var(--fs-chip);

package/components/VisitorsReportPreview.vue

@@ -397,12 +397,12 @@ onMounted(() => {
 
     :deep(.v-data-table thead tr th) {
         padding: 6px !important;
-        font-size: 11px !important;
+        font-size: 12px !important;
     }
 
     :deep(.v-data-table tbody tr td) {
         padding: 4px !important;
-        font-size: 10px !important;
+        font-size: 12px !important;
     }
 
     :deep(.v-img) {

package/composables/useKey.ts

@@ -132,21 +132,25 @@ export default function useKey() {
     keyLog.value = data || new MKeyLog();
   }
 
-  function addKeyLog(_keyLog: TKeyLog) {
+  // Await each request so a failed save reaches the caller's catch instead of showing "success".
+  async function addKeyLog(_keyLog: TKeyLog) {
     isAdding.value = true;
     delete _keyLog._id;
-    const data = useNuxtApp().$api<{
-      message: string | undefined;
-    }>(keyLogApiUrl, {
-      method: "POST",
-      body: JSON.stringify(_keyLog),
-    });
-    isAdding.value = false;
-    return data.message;
+    try {
+      const data = await useNuxtApp().$api<{
+        message: string | undefined;
+      }>(keyLogApiUrl, {
+        method: "POST",
+        body: JSON.stringify(_keyLog),
+      });
+      return data?.message;
+    } finally {
+      isAdding.value = false;
+    }
   }
 
-  function getKeyById(id: string) {
-    const data = useNuxtApp().$api<TKeyLog | undefined>(
+  async function getKeyById(id: string) {
+    const data = await useNuxtApp().$api<TKeyLog | undefined>(
       `${keyLogApiUrl}/${id}`,
       {
         method: "GET",
@@ -158,22 +162,22 @@ export default function useKey() {
     return data;
   }
 
-  function updateKeyById(id: string, payload: TKeyLog) {
+  async function updateKeyById(id: string, payload: TKeyLog) {
     payload.codes =
       payload.codes instanceof Array && payload.codes.length > 0
         ? payload.codes
         : [];
-    const data = useNuxtApp().$api<{
+    const data = await useNuxtApp().$api<{
       message: { acknowledged: boolean };
     }>(`${keyLogApiUrl}/${id}`, {
       method: "PUT",
       body: payload,
     });
-    return data.message;
+    return data?.message;
   }
 
-  function deleteKeyById(id: string) {
-    const data = useNuxtApp().$api<{ message: TKeyLog }>(
+  async function deleteKeyById(id: string) {
+    const data = await useNuxtApp().$api<{ message: TKeyLog }>(
       `${keyLogApiUrl}/${id}`,
       {
         method: "DELETE",

package/package.json

@@ -2,7 +2,7 @@
   "name": "@7365admin1/layer-common",
   "license": "MIT",
   "type": "module",
-  "version": "4.136.0",
+  "version": "4.138.0",
   "author": "7365admin1",
   "main": "./nuxt.config.ts",
   "//files": "What a consumer extending this layer actually loads. Without this npm ships the whole working tree - the changesets, the CI workflows, the render harness in tools/ and any scratch directory that happened to exist at publish time. Nuxt resolves a layer by directory, so every runtime directory below has to stay listed; adding a new top-level runtime directory means adding it here too.",

package/pages/unauthorized.vue

@@ -1,5 +1,6 @@
 <!--
-  The "you are not a member of an organization" gate.
+  The "no permission for this page" gate (wording only; non-members use
+  require-organization-membership).
 
   Was Vuetify's chrome: `text-subtitle-1` for the sentence, `text-subtitle-2`
   on the buttons, and `variant="tonal" rounded="xl"` - a 24px-cornered pill in
@@ -12,8 +13,8 @@
 <template>
   <v-row no-gutters class="fill-height" align="center" justify="center">
     <v-col cols="12" lg="6" md="6" sm="6" class="text-center gate-page__message">
-      You must be a member of an organization to view this page. Please join or
-      contact your administrator for access.
+      You don't have permission to open this page. If you need it for your
+      work, ask your administrator to give your role access.
       <v-row no-gutters justify="center" class="mt-2">
         <v-btn
           class="screen-btn-primary"

package/utils/breadcrumb.ts

@@ -25,6 +25,13 @@ const ACRONYMS: Record<string, string> = {
 const RENAMED: [string, string][] = [
   ["Building Mgmt", "Occupancy Mgmt"],
   ["Event Mgmt", "Task Calendar"],
+  // Schedule screens (owner 21 Oct): the same names the menu and the mobile apps use.
+  ["Schedule Task", "Extra Visits"],
+  ...["Cleaning", "Technician", "Pest", "Pool", "Landscape"].flatMap((t): [string, string][] => [
+    [`${t} Schedule Area`, "Area"],
+    [`${t} Schedule Unit`, "Unit"],
+    [`${t} Schedule`, "Daily Checklist"],
+  ]),
 ];
 
 export function moduleNameFromRoute(route: {

package/utils/nav-sections.ts

@@ -50,6 +50,8 @@ export const NAV_SECTIONS: Record<string, string> = {
   Area: "SERVICE DELIVERY",
   Unit: "SERVICE DELIVERY",
   "Schedule Task": "SERVICE DELIVERY",
+  "Daily Checklist": "SERVICE DELIVERY",
+  "Extra Visits": "SERVICE DELIVERY",
 
   // EQUIPMENT
   "Equipment Management": "EQUIPMENT",

package/utils/schedule-task.ts

@@ -176,12 +176,20 @@ export function validateSchedule(f: TScheduleForm, now: Date = new Date()): TSch
   return e;
 }
 
-/** "at 08:00 every weekday from 21 Oct until 31 Dec" - the When, as a sentence fragment. */
+/** "08:00" -> "8:00 am", "14:30" -> "2:30 pm". Anything else is returned as given. */
+export function fmtTime(time: string | null | undefined): string {
+  const m = TIME_RE.exec(time || "");
+  if (!m) return time || "";
+  const h = Number(m[1]);
+  return `${h % 12 || 12}:${m[2]} ${h < 12 ? "am" : "pm"}`;
+}
+
+/** "every weekday at 8:00 am from 21 Oct until 31 Dec" - the When, as a sentence fragment. */
 export function describeWhen(
   t: Partial<TRule> & { time?: string },
   nowYear?: string,
 ): string {
-  const time = t.time ? `at ${t.time} ` : "";
+  const at = t.time ? ` at ${fmtTime(t.time)}` : "";
   const repeat = t.repeat || "none";
   const dates = t.dates || [];
   const range =
@@ -189,23 +197,23 @@ export function describeWhen(
     (t.endDate ? ` until ${fmtDate(t.endDate, nowYear)}` : "");
   switch (repeat) {
     case "daily":
-      return `${time}every day${range}`;
+      return `every day${at}${range}`;
     case "weekdays":
-      return `${time}every weekday${range}`;
+      return `every weekday${at}${range}`;
     case "weekly": {
       const names = WEEK_DAYS.filter((d) => (t.daysOfWeek || []).includes(d.value)).map((d) => d.title);
-      return `${time}every ${names.join(", ") || "week"}${range}`;
+      return `every ${names.join(", ") || "week"}${at}${range}`;
     }
     case "monthly": {
       const day = t.startDate ? Number(t.startDate.slice(8, 10)) : null;
       const on = day === null ? "every month" : day === 31 ? "on the last day of every month" : `on day ${day} of every month`;
-      return `${time}${on}${range}`;
+      return `${on}${at}${range}`;
     }
     default: {
       const sorted = [...dates].sort();
-      if (!sorted.length) return `${time}on no date`.trim();
+      if (!sorted.length) return `on no date${at}`;
       const shown = sorted.slice(0, 3).map((d) => fmtDate(d, nowYear)).join(", ");
-      return `${time}on ${shown}${sorted.length > 3 ? ` +${sorted.length - 3} more` : ""}${
+      return `on ${shown}${sorted.length > 3 ? ` +${sorted.length - 3} more` : ""}${at}${
         t.endDate ? ` (until ${fmtDate(t.endDate, nowYear)})` : ""
       }`;
     }
@@ -226,33 +234,93 @@ export function buildSummary(
   return `Adds 1 extra visit to ${areas} (${units}) ${describeWhen(f, year)} — next: ${nextText}.`;
 }
 
-export const STATUS_LABELS: Record<string, string> = { active: "Active", paused: "Paused", ended: "Ended" };
+/** The state of an extra visit as a whole. "ended" reads "Finished": nothing is left to run. */
+export const STATUS_LABELS: Record<string, string> = { active: "Active", paused: "Paused", ended: "Finished" };
+
+/** What each word means, shown on the page so nobody has to guess. Same words as the mobile apps. */
+export const STATUS_HELP: { word: string; meaning: string }[] = [
+  { word: "Active", meaning: "It will add a visit at its next date and time." },
+  { word: "Paused", meaning: "Stopped for now. Nothing is added until you press Resume." },
+  { word: "Finished", meaning: "Its last date has passed, so nothing more is added. Press \"Extend or repeat\" to keep it going." },
+];
+export const RESULT_HELP: { word: string; meaning: string }[] = [
+  { word: "Not started", meaning: "Nothing is ticked yet." },
+  { word: "In progress", meaning: "Some units are ticked; the rest can still be done today." },
+  { word: "Done", meaning: "Staff ticked every unit of the visit." },
+  { word: "Needs redo", meaning: "A supervisor sent a unit back to be done again." },
+  { word: "Missed", meaning: "The day ended with units not done." },
+];
 
-/** "Done 20 Oct" / "Missed 20 Oct" / "Not run yet". */
+/** "Done 20 Oct" / "Missed 20 Oct" / "No visit yet". */
 export function lastRunLabel(row: { lastRun?: string | null; lastRunStatus?: string | null }): string {
-  if (!row.lastRun) return "Not run yet";
+  if (!row.lastRun) return "No visit yet";
   const m = /^(\d{4}-\d{2}-\d{2})/.exec(row.lastRun);
   const day = m ? fmtDate(m[1]) : "";
-  const word = { done: "Done", missed: "Missed", pending: "In progress" }[row.lastRunStatus || ""] || "Ran";
+  const word = { done: "Done", missed: "Missed", pending: "In progress" }[row.lastRunStatus || ""] || "Added";
   return `${word} ${day}`.trim();
 }
 
+/** Why an extra visit says Finished, in one sentence. */
+export function finishedReason(row: Partial<TRule>, nowYear?: string): string {
+  if (row.repeat && row.repeat !== "none" && row.endDate) {
+    return `Its end date (${fmtDate(row.endDate, nowYear)}) has passed.`;
+  }
+  return "All the dates it was set for have passed.";
+}
+
 /**
- * One name and one page path per trade. The names are the web menu names (M&E
- * is "Technician Schedule" - the mobile app mirrors it); the path is the folder
- * the trade app serves its daily schedule from.
+ * One sentence per list row: "Every Mon, Wed at 2:00 pm in Lobby D — next on Wed 22 Oct".
+ * Paused and finished rows say so instead of a next date.
  */
-export const SCHEDULE_NAMES: Record<string, { title: string; route: string }> = {
-  cleaning_services: { title: "Cleaning Schedule", route: "cleaning-schedule" },
-  landscaping_services: { title: "Landscape Schedule", route: "landscape-schedule" },
-  pest_control_services: { title: "Pest Schedule", route: "pest-schedule" },
-  pool_maintenance_services: { title: "Pool Schedule", route: "pool-schedule" },
-  mechanical_electrical_services: { title: "Technician Schedule", route: "technician-schedule" },
+export function describeRow(
+  row: Partial<TRule> & {
+    areas?: { name?: string }[];
+    nextRun?: string | null;
+    scheduleStatus?: string | null;
+    paused?: boolean;
+  },
+  nowYear?: string,
+): string {
+  const when = describeWhen(row, nowYear);
+  const names = (row.areas || []).map((a) => a?.name).filter(Boolean) as string[];
+  const where = names.length
+    ? ` in ${names.slice(0, 3).join(", ")}${names.length > 3 ? ` +${names.length - 3} more` : ""}`
+    : "";
+  const state = row.scheduleStatus || (row.paused ? "paused" : "active");
+  const m = /^(\d{4}-\d{2}-\d{2})/.exec(row.nextRun || "");
+  const tail =
+    state === "paused"
+      ? " — paused, nothing is added until you resume it"
+      : state === "ended"
+        ? " — finished"
+        : m
+          ? ` — next on ${DAYS[weekdayOf(m[1])]} ${fmtDate(m[1], nowYear)}`
+          : "";
+  return `${when.charAt(0).toUpperCase()}${when.slice(1)}${where}${tail}`;
+}
+
+/**
+ * One trade word and one page path per trade. Every trade's menu says "Daily
+ * Checklist" (owner 21 Oct; the mobile apps use the same name); the page title
+ * adds the trade ("Daily Checklist – Cleaning"). The path is the folder the
+ * trade app serves its daily checklist from - unchanged, so links and
+ * permissions keep working.
+ */
+export const SCHEDULE_NAMES: Record<string, { title: string; trade: string; route: string }> = {
+  cleaning_services: { title: "Daily Checklist", trade: "Cleaning", route: "cleaning-schedule" },
+  landscaping_services: { title: "Daily Checklist", trade: "Landscape", route: "landscape-schedule" },
+  pest_control_services: { title: "Daily Checklist", trade: "Pest", route: "pest-schedule" },
+  pool_maintenance_services: { title: "Daily Checklist", trade: "Pool", route: "pool-schedule" },
+  mechanical_electrical_services: { title: "Daily Checklist", trade: "Technician", route: "technician-schedule" },
 };
 
 export const scheduleNameFor = (serviceType?: string) =>
   SCHEDULE_NAMES[serviceType || ""] || SCHEDULE_NAMES.cleaning_services;
 
+/** "Daily Checklist – Cleaning" / "– Technician" / ... */
+export const dailyChecklistTitle = (serviceType?: string) =>
+  `Daily Checklist – ${scheduleNameFor(serviceType).trade}`;
+
 /** Singapore "today" as YYYY-MM-DD, for "is today's list there yet". */
 export const sgtToday = (now: Date = new Date()) => sgtParts(now).date;
 
@@ -265,13 +333,28 @@ export function hasListForToday(rows: { createdAt?: string | Date }[], now: Date
   });
 }
 
-/** One spelling for the in-flight state ("Ongoing" / "On-going" -> "In progress"); other words pass through. */
+/**
+ * The daily checklist's words, the same on web and mobile: Open -> Not started,
+ * Ongoing / On-going -> In progress, Completed -> Done. Closed stays Closed (the
+ * day ended; anything not ticked counts as Missed). Other words pass through.
+ */
 export function dailyStatusLabel(status: string | null | undefined): string {
   const s = String(status ?? "").trim();
   if (!s) return "No status";
-  return /^on-?going$/i.test(s) ? "In progress" : s;
+  if (/^on-?going$/i.test(s)) return "In progress";
+  if (/^open$/i.test(s)) return "Not started";
+  if (/^completed?$/i.test(s)) return "Done";
+  return s;
 }
 
+/** What the daily checklist's words mean, shown on the page. */
+export const DAILY_STATUS_HELP: { word: string; meaning: string }[] = [
+  { word: "Not started", meaning: "The day's list is open and nothing is ticked yet." },
+  { word: "In progress", meaning: "Staff have ticked some units; the rest can still be done." },
+  { word: "Done", meaning: "Every unit on the list was ticked." },
+  { word: "Closed", meaning: "The day ended at midnight. The list is view-only, and anything not ticked counts as Missed." },
+];
+
 /** A unit's result in the history view: Done, Needs redo (rejected by a supervisor) or Missed (never done). */
 export function historyUnitLabel(status: string | null | undefined): "Done" | "Needs redo" | "Missed" {
   const s = String(status ?? "").trim().toLowerCase();