@7365admin1/layer-common 4.1.2-staging.247 → 4.1.2

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # @iservice365/layer-common
 
+## 4.1.2
+
+### Patch Changes
+
+- 89deaab: Check the permission the estate stores for the Bulletin Board gates
+
+  `useBulletinBoardPermission` asked for `bulletin-board:<action>`, a resource held
+  by 8-9 of the 428 roles in the live catalogue, while the same five actions under
+  `bulletin-board-mgmt:` are held by 80-82. `-hygiene`, `-security` and
+  `-pest-control` leave `canViewBulletinBoardDetails` unbound, so this composable
+  gates their sidebar-linked bulletin board page and ~74 roles that hold the right
+  were refused it.
+
+  Both resources are now accepted, and the list gate also accepts the singular
+  `see-all-bulletin-board` action that 80 roles store. No key is renamed, nothing
+  is granted and no role document changes.
+
 ## 4.1.1
 
 ### Patch Changes
@@ -313,11 +330,11 @@
   `utils/console-tier.ts` + `composables/useConsoleTier.ts` mirror the server's
   own rule from two endpoints the console already calls, unprojected:
 
-            GET /api/members/user/:user/app/admin    the Seven365 staff membership
-            GET /api/roles/id/:role                  that membership's role document
+              GET /api/members/user/:user/app/admin    the Seven365 staff membership
+              GET /api/roles/id/:role                  that membership's role document
 
-            owner  = member.type === "admin" && role.type === "admin" && role.default === true
-            staff  = member.type === "admin" && role.type === "admin"
+              owner  = member.type === "admin" && role.type === "admin" && role.default === true
+              staff  = member.type === "admin" && role.type === "admin"
 
   `role.default` is the marker because it is the only property of a platform
   staff role no API caller can set - `role.controller.ts` validates create and

package/components/Input/DatePicker.vue

@@ -35,15 +35,9 @@ const dateFormattedReadOnly = ref<string | null>(null)
 
 const dateInput = ref<HTMLInputElement | null>(null)
 const datePickerRef = ref<HTMLInputElement | null>(null)
-let nativeInput: HTMLInputElement | null = null
 
 const isInitialLoad = ref(true)
 
-function handleNativeInputClick(e: MouseEvent) {
-    e.stopPropagation()
-    openDatePicker()
-}
-
 function clearDate() {
     date.value = null
     dateFormattedReadOnly.value = null
@@ -94,17 +88,15 @@ watch(date, () => {
 onMounted(async () => {
     await nextTick()
     isInitialLoad.value = false
-    nativeInput = (datePickerRef.value as any)?.$el?.querySelector('input')
+    const nativeInput = (datePickerRef.value as any)?.$el?.querySelector('input')
     if (nativeInput) {
-        nativeInput.addEventListener('click', handleNativeInputClick)
+        nativeInput.addEventListener('click', (e: MouseEvent) => {
+            e.stopPropagation()
+            openDatePicker()
+        })
     }
 })
 
-onBeforeUnmount(() => {
-    nativeInput?.removeEventListener('click', handleNativeInputClick)
-    nativeInput = null
-})
-
 defineExpose({
     validate
 })

package/components/Input/DateTimePicker.vue

@@ -70,15 +70,9 @@ const inputPlaceholder = computed(
 
 const dateInput = ref<HTMLInputElement | null>(null);
 const dateTimePickerRef = ref<HTMLInputElement | null>(null);
-let nativeInput: HTMLInputElement | null = null;
 
 const isInitialLoad = ref(true);
 
-function handleNativeInputClick(e: MouseEvent) {
-  e.stopPropagation();
-  openDatePicker();
-}
-
 function openDatePicker() {
   setTimeout(() => {
     dateInput.value?.showPicker?.();
@@ -183,19 +177,17 @@ onMounted(async () => {
   await nextTick();
   isInitialLoad.value = false;
   // Wait until Vuetify renders its internal input
-  nativeInput = (dateTimePickerRef.value as any)?.$el?.querySelector(
+  const nativeInput = (dateTimePickerRef.value as any)?.$el?.querySelector(
     "input"
   );
   if (nativeInput) {
-    nativeInput.addEventListener("click", handleNativeInputClick);
+    nativeInput.addEventListener("click", (e: MouseEvent) => {
+      e.stopPropagation();
+      openDatePicker();
+    });
   }
 });
 
-onBeforeUnmount(() => {
-  nativeInput?.removeEventListener("click", handleNativeInputClick);
-  nativeInput = null;
-});
-
 defineExpose({
   validate,
 });

package/components/VisitorManagement.vue

@@ -1758,14 +1758,9 @@ function closeApproveRejectVisitorDialog() {
 async function updateVisitorApprovalStatus() {
   isApprovingRejectingVisitor.value = true;
   try {
-    const approval = {
-      approvedBy: currentUser.value?._id,
+    await updateVisitor(selectedVisitorForApproval.value._id, {
       status: approveRejectVisitorDialog.value.status,
       remarks: visitorApproveRejectRemarks.value,
-      approvedAt: new Date(),
-    };
-    await updateVisitor(selectedVisitorForApproval.value._id, {
-      approval,
       updatedBy: currentUser.value?._id,
     } as any);
     showMessage(
@@ -1896,9 +1891,7 @@ const {
       // A self-service submission awaiting Approve/Reject (Self-Service
       // Auto-Approvals off) lands here as "pending" alongside already
       // "approved" guests, rather than being hidden until someone acts on it.
-      // "rejected" is included too, so a rejected submission stays visible
-      // on this tab instead of disappearing once acted on.
-      params.status = "approved,pending,rejected";
+      params.status = "approved,pending";
     } else if (activeTab.value === "resident-transactions") {
       params.status = "registered";
       params.type = "resident,tenant";
@@ -2436,7 +2429,7 @@ function buildReportParams(): any {
     // report generated from this tab would silently omit contractors and
     // pending rows the screen is actually showing.
     params.type = "guest,contractor";
-    params.status = "approved,pending,rejected";
+    params.status = "approved,pending";
   } else if (activeTab.value === "resident-transactions") {
     params.status = "registered";
     params.type = "resident,tenant";

package/package.json

@@ -2,7 +2,7 @@
   "name": "@7365admin1/layer-common",
   "license": "MIT",
   "type": "module",
-  "version": "4.1.2-staging.247",
+  "version": "4.1.2",
   "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.",