@motor-cms/ui-admin 1.16.1 → 1.16.3

package/app/components/dashboard/DashboardOnboarding.vue

@@ -7,7 +7,8 @@ const { can } = usePermissions()
 const { user } = useSanctumAuth<User>()
 const router = useRouter()
 const { completeOnboarding } = useProfileApi()
-const { commitDone, isDone } = useOnboardingDone()
+const { commitDone } = useOnboardingDone()
+const { isEnabled } = useOnboardingEnabled()
 
 const { isCompleted: announcementsCompleted, markCompleted: markAnnouncementsDone } = useOnboardingState('dashboard-announcements')
 const { isCompleted: notificationsCompleted, markCompleted: markNotificationsDone } = useOnboardingState('notifications')
@@ -225,17 +226,14 @@ const stopWatch = watch(
     if (!announcementsW || !notificationsW || !searchW || !adminNavW || !userData) return
     stopWatch()
 
-    // Only reset if the user has NOT already committed a skip.
-    // isSkipCommitted() reads localStorage, so it survives hard reloads even
-    // when the completeOnboarding() API call was cancelled by the browser.
-    // resetAll() in useOnboardingResetAll clears the skip flag, so a
-    // deliberate "restart tour" from the profile page still works correctly.
-    if (userData.show_onboarding && !isDone()) {
+    const shouldRun = isEnabled.value
+
+    if (shouldRun) {
       userData.show_onboarding = false
       resetOnboardingState()
     }
 
-    if (!announcementsCompleted.value) {
+    if (shouldRun && !announcementsCompleted.value) {
       startAnnouncements()
     }
   },

package/app/composables/useOnboardingState.ts

@@ -129,3 +129,23 @@ export function useOnboardingDone() {
 
   return { commitDone, isDone }
 }
+
+/**
+ * Central guard: should the onboarding tour run for this user?
+ *
+ * Returns true only when BOTH conditions are met:
+ *   1. Backend says show_onboarding=true (fresh user or "restart tour" from profile)
+ *   2. User hasn't already completed/skipped the tour in this browser (isDone=false)
+ *
+ * Use this everywhere instead of hand-rolling the check.
+ */
+export function useOnboardingEnabled() {
+  const { user } = useSanctumAuth<User>()
+  const { isDone } = useOnboardingDone()
+
+  const isEnabled = computed(() => {
+    return !!user.value?.data?.show_onboarding && !isDone()
+  })
+
+  return { isEnabled }
+}

package/app/pages/profile.vue

@@ -185,6 +185,7 @@ async function onRestartTour() {
   try {
     await resetOnboarding()
     resetOnboardingState()
+    await refreshIdentity()
     success(t('motor-core.profile.toast_tour_reset_title'), t('motor-core.profile.toast_tour_reset_message'))
     await router.push('/')
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@motor-cms/ui-admin",
-  "version": "1.16.1",
+  "version": "1.16.3",
   "type": "module",
   "main": "./nuxt.config.ts",
   "files": [
@@ -20,7 +20,7 @@
     "zod": "^4.0.0"
   },
   "peerDependencies": {
-    "@motor-cms/ui-core": ">=1.16.1",
+    "@motor-cms/ui-core": ">=1.16.3",
     "nuxt": "^4.0.0",
     "vue": "^3.5.0"
   }