@conduction/nextcloud-vue 2.70.0 → 2.72.0

package/dist/nextcloud-vue.cjs.js.map

@@ -1,4 +1,4 @@
 [diffend] Oversized file quarantined before diffing.
 name: package/dist/nextcloud-vue.cjs.js.map
-size: 35617052 bytes
-sha256: 8d17da3a5f3f43dba63932341520f63af085b4d61c1b614631488abb6ecd41d8
+size: 35684577 bytes
+sha256: 40220f92c6defbd478644408d9d23a88047c47798fb52eefee28e6d70e0fbc9a

package/dist/nextcloud-vue.css

@@ -2841,6 +2841,50 @@ a.cn-kpi-card--stacked:focus-visible {
 .cn-requires-app.cn-requires-app .empty-content__action {
 	margin-top: var(--default-grid-baseline, 4px);
 }
+
+/* NcEmptyContent forces its icon's svg to 64px with !important while the
+ * rule above lets the icon box shrink, so the svg spilled out of its box and
+ * over the name. Hold the svg to the size the caller asked for. */
+.cn-requires-app.cn-requires-app .empty-content__icon svg {
+	width: 32px !important;
+	height: 32px !important;
+	max-width: 32px !important;
+	max-height: 32px !important;
+}
+
+/* The dashboard's compact "Install <app>" placeholder: two lines of text and
+ * the button, nothing else, so it fits a two-row tile even under a title row.
+ * `safe center` keeps the top in view when a tile is smaller still, and the
+ * body scrolls rather than clipping the button. */
+.cn-requires-app--compact {
+	display: flex;
+	flex-direction: column;
+	align-items: center;
+	justify-content: safe center;
+	gap: calc(2 * var(--default-grid-baseline, 4px));
+	box-sizing: border-box;
+	height: 100%;
+	min-height: 0;
+	overflow: auto;
+	padding: calc(2 * var(--default-grid-baseline, 4px));
+	text-align: center;
+}
+
+.cn-requires-app--compact .cn-requires-app__description {
+	display: -webkit-box;
+	margin: 0;
+	overflow: hidden;
+	color: var(--color-text-maxcontrast);
+	font-size: 0.9em;
+	line-height: 1.4;
+	-webkit-box-orient: vertical;
+	-webkit-line-clamp: 2;
+	line-clamp: 2;
+}
+
+.cn-requires-app--compact .button-vue {
+	flex-shrink: 0;
+}
 /* ========================================
    @conduction/nextcloud-vue — CnDetailPage Styles
    ======================================== */
@@ -15645,6 +15689,15 @@ a.cn-xwiki-card__title[data-v-54e85fe3]:hover {
 }
 
 
+/* A select option breaks between words only, never inside one. */
+.cn-form-dialog__option[data-v-c3eeb0a2] {
+	display: block;
+	min-width: 0;
+	white-space: normal;
+	overflow-wrap: normal;
+	word-break: normal;
+	hyphens: manual;
+}
 .cn-form-dialog__draft-actions[data-v-c3eeb0a2] {
 	display: flex;
 	gap: 8px;
@@ -26529,6 +26582,18 @@ a.cn-flow-runs-widget__row-content:hover .cn-flow-runs-widget__name[data-v-75628
 	font-size: 0.85em;
 	color: var(--color-text-maxcontrast);
 }
+
+/* Full width, like the text fields around it: at its natural width the
+   picker cut every option label in half. */
+.cn-flow-node-edit__select[data-v-e107484a] {
+	width: 100%;
+}
+.cn-flow-node-edit__field-help[data-v-e107484a] {
+	margin: 4px 0 0;
+	padding-inline-start: 8px;
+	font-size: 0.85em;
+	color: var(--color-text-maxcontrast);
+}
 .cn-flow-node-edit__advanced summary[data-v-e107484a] {
 	cursor: pointer;
 	color: var(--color-text-maxcontrast);
@@ -27983,6 +28048,58 @@ to {
 }
 
 
+.cn-flow-step-outcomes[data-v-5ac5ad4f] {
+	margin: 4px 0 0;
+	font-size: 0.9em;
+}
+.cn-flow-step-outcomes__list[data-v-5ac5ad4f] {
+	margin: 0;
+	padding: 0;
+}
+
+/* Stacked, name over value: the run sidebar is narrow, and a side-by-side
+   row wrapped mid-list. The host's dt/dd margins and alignment are reset. */
+.cn-flow-step-outcomes__row[data-v-5ac5ad4f] {
+	display: block;
+	margin-block: 4px;
+	padding-block: 2px;
+	padding-inline-start: 8px;
+	border-inline-start: 3px solid var(--color-border);
+	text-align: start;
+}
+.cn-flow-step-outcomes__row--success[data-v-5ac5ad4f] {
+	border-inline-start-color: var(--color-success);
+}
+.cn-flow-step-outcomes__row--warning[data-v-5ac5ad4f] {
+	border-inline-start-color: var(--color-warning);
+}
+.cn-flow-step-outcomes__row--error[data-v-5ac5ad4f] {
+	border-inline-start-color: var(--color-error);
+}
+.cn-flow-step-outcomes__name[data-v-5ac5ad4f] {
+	display: block;
+	white-space: normal;
+	overflow-wrap: anywhere;
+	margin: 0;
+	padding: 0;
+	font-weight: bold;
+	text-align: start;
+}
+.cn-flow-step-outcomes__value[data-v-5ac5ad4f] {
+	display: block;
+	white-space: normal;
+	margin: 0;
+	padding: 0;
+	text-align: start;
+	color: var(--color-main-text);
+	overflow-wrap: anywhere;
+}
+.cn-flow-step-outcomes__truncated[data-v-5ac5ad4f] {
+	margin: 4px 0 0;
+	color: var(--color-text-maxcontrast);
+}
+
+
 .cn-run-sidebar[data-v-48e781f0] {
 	display: flex;
 	flex-direction: column;

package/l10n/en.json

@@ -1106,7 +1106,21 @@
     "Loading…": "Loading…",
     "A register and schema are required": "A register and schema are required",
     "Change the search or try again.": "Change the search or try again.",
-    "Try again later.": "Try again later."
+    "Try again later.": "Try again later.",
+    "Who this step reached": "Who this step reached",
+    "Each list shows its first entries only.": "Each list shows its first entries only.",
+    "Delivered": "Delivered",
+    "Opted out": "Opted out",
+    "Not sent, the opt-out check did not answer": "Not sent, the opt-out check did not answer",
+    "Skipped by their notification settings": "Skipped by their notification settings",
+    "Skipped, sending is switched off": "Skipped, sending is switched off",
+    "Held back by the send limit": "Held back by the send limit",
+    "Refused by the step’s address rule": "Refused by the step’s address rule",
+    "Not a known user or group": "Not a known user or group",
+    "Failed": "Failed",
+    "external recipients are off": "external recipients are off",
+    "not on the item": "not on the item",
+    "not a valid address": "not a valid address"
   },
   "plurals": {
     "%n year": ["%n year", "%n years"],

package/l10n/nl.json

@@ -1150,7 +1150,21 @@
     "Loading…": "Laden…",
     "A register and schema are required": "Een register en schema zijn verplicht",
     "Change the search or try again.": "Pas de zoekopdracht aan of probeer het opnieuw.",
-    "Try again later.": "Probeer het later opnieuw."
+    "Try again later.": "Probeer het later opnieuw.",
+    "Who this step reached": "Wie deze stap bereikte",
+    "Each list shows its first entries only.": "Elke lijst toont alleen de eerste vermeldingen.",
+    "Delivered": "Bezorgd",
+    "Opted out": "Afgemeld",
+    "Not sent, the opt-out check did not answer": "Niet verstuurd, de afmeldcontrole gaf geen antwoord",
+    "Skipped by their notification settings": "Overgeslagen door hun meldingsinstellingen",
+    "Skipped, sending is switched off": "Overgeslagen, verzenden staat uit",
+    "Held back by the send limit": "Tegengehouden door de verzendlimiet",
+    "Refused by the step’s address rule": "Geweigerd door de adresregel van de stap",
+    "Not a known user or group": "Geen bekende gebruiker of groep",
+    "Failed": "Mislukt",
+    "external recipients are off": "externe ontvangers staan uit",
+    "not on the item": "staat niet op het item",
+    "not a valid address": "geen geldig adres"
   },
   "plurals": {
     "%n year": ["%n jaar", "%n jaar"],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@conduction/nextcloud-vue",
-  "version": "2.70.0",
+  "version": "2.72.0",
   "description": "Shared Vue component library for Conduction Nextcloud apps — complements @nextcloud/vue with higher-level components, OpenRegister integration, and NL Design System support",
   "homepage": "https://github.com/ConductionNL/nextcloud-vue#readme",
   "bugs": {

package/src/components/CnAppRoot/CnAppRoot.vue

@@ -513,8 +513,10 @@
 					:manifest="manifest"
 					:seenVersion="walkthroughSeenVersion"
 					:resume="walkthroughResume"
+					:autoStart="!walkthroughPaused"
 					:translate="translate"
 					@complete="onWalkthroughComplete"
+					@pause="onWalkthroughPause"
 					@progress="onWalkthroughProgress" />
 			</slot>
 			<!--
@@ -934,8 +936,11 @@ export default {
 			 * Restart entry for the product walkthrough (ADR-043). Descendants
 			 * (a menu/settings "Replay walkthrough" entry, or a manifest menu
 			 * `action: "replay-walkthrough"`) call this to re-run a tour. With no
-			 * `tourId` the first declared tour is used.
+			 * `tourId` the first declared tour is used. An unfinished tour
+			 * continues where the user was (a paused tour, else the saved
+			 * step); "Start over" in the user settings goes back to step 1.
 			 *
+			 * @spec openspec/changes/audit-round-lib-fixes/specs/cn-walkthrough/spec.md
 			 * @param {string} [tourId] The tour to restart.
 			 * @return {void}
 			 */
@@ -945,9 +950,18 @@ export default {
 				}
 				const wt = useWalkthrough(this.appId, this.manifest)
 				const id = tourId || (this.manifest.walkthrough.tours[0] && this.manifest.walkthrough.tours[0].id)
-				if (id) {
-					wt.restart(id)
+				if (!id) {
+					return
+				}
+				const progress = this.walkthroughProgressValue
+				this.unpauseWalkthroughProgress()
+				if (wt.paused.value && wt.activeTour.value && wt.activeTour.value.id === id && wt.resumePaused()) {
+					return
+				}
+				if (progress && progress.tourId === id && wt.resumeAt(id, progress.stepId)) {
+					return
 				}
+				wt.restart(id)
 			},
 
 			/**
@@ -1528,7 +1542,7 @@ export default {
 		},
 	},
 
-	emits: ['setup-complete', 'walkthrough-complete'],
+	emits: ['setup-complete', 'setup-wizard-dismissed', 'walkthrough-complete'],
 
 	/**
 	 * Component-instance state for the capabilities guard.
@@ -2599,9 +2613,21 @@ export default {
 			} catch {
 				// No URL token; fall through to the remembered progress.
 			}
-			// An unfinished tour continues at the step the user reached.
+			// An unfinished tour continues at the step the user reached,
+			// unless the user paused it: then it waits for "Continue".
 			const progress = this.walkthroughProgressValue
-			return progress ? { tourId: progress.tourId, stepId: progress.stepId } : null
+			return progress && !progress.paused ? { tourId: progress.tourId, stepId: progress.stepId } : null
+		},
+
+		/**
+		 * Whether the user paused the tour (X, ESC or the dim). A paused tour
+		 * stays hidden across page loads until the user picks "Continue".
+		 *
+		 * @spec openspec/changes/audit-round-lib-fixes/specs/cn-walkthrough/spec.md
+		 * @return {boolean} True while the saved progress is paused.
+		 */
+		walkthroughPaused() {
+			return !!(this.walkthroughProgressValue && this.walkthroughProgressValue.paused)
 		},
 
 		phase() {
@@ -3281,7 +3307,7 @@ export default {
 			// the server keeps reporting it unmet — so without this the wizard
 			// auto-opens again on every visit. Cancel must not be the only way to
 			// get peace.
-			this.persistSetupWizardDismissal()
+			this.persistSetupWizardDismissal(true)
 			if (this.setupState && typeof this.setupState.refresh === 'function') {
 				this.setupState.refresh()
 			}
@@ -3302,7 +3328,7 @@ export default {
 		 * @return {void}
 		 */
 		dismissSetupWizard() {
-			this.persistSetupWizardDismissal()
+			this.persistSetupWizardDismissal(false)
 			this.setupWizardOpen = false
 		},
 
@@ -3325,7 +3351,7 @@ export default {
 		 * @return {boolean}
 		 */
 		isSetupWizardDismissed() {
-			if (this.setupWizardDismissed) {
+			if (this.setupWizardDismissed || this.isSetupWizardDismissedOnServer()) {
 				return true
 			}
 			try {
@@ -3336,18 +3362,87 @@ export default {
 		},
 
 		/**
-		 * Record the non-gating setup wizard as dismissed for this manifest
-		 * `setup.version` so it doesn't auto-open again.
+		 * Whether the app's setup status says the wizard was closed, so it
+		 * stays closed in every browser and on every device. The status
+		 * (`GET /api/setup/status`) may carry `dismissed: true`, or the
+		 * `setup.version` it was closed at (`dismissed: 2`), which re-opens the
+		 * wizard after a version bump. An app whose status carries neither
+		 * keeps today's per-browser behaviour. An app may also answer the
+		 * outstanding steps itself when it records the close (OpenRegister's
+		 * `dismiss-setup`), which needs nothing here.
+		 *
+		 * @spec openspec/changes/setup-wizard-close-on-server/specs/cn-setup-wizard/spec.md
+		 * @return {boolean} True when the server recorded the close for this version.
+		 */
+		isSetupWizardDismissedOnServer() {
+			const s = this.setupState
+			const status = s && s.status && s.status.value
+			const dismissed = status && typeof status === 'object' ? status.dismissed : undefined
+			if (dismissed === true) {
+				return true
+			}
+			if (typeof dismissed === 'number' && Number.isFinite(dismissed)) {
+				const version = (this.manifest && this.manifest.setup && this.manifest.setup.version) || 0
+				return dismissed >= version
+			}
+			return false
+		},
+
+		/**
+		 * The setup action that records a closed wizard on the server, from
+		 * `manifest.setup.dismissAction`, or '' when the app declares none.
+		 *
+		 * @spec openspec/changes/setup-wizard-close-on-server/specs/cn-setup-wizard/spec.md
+		 * @return {string} The action id.
+		 */
+		setupDismissAction() {
+			const action = this.manifest && this.manifest.setup && this.manifest.setup.dismissAction
+			return typeof action === 'string' && /^[a-z0-9][a-z0-9-]*$/.test(action) ? action : ''
+		},
+
+		/**
+		 * Record the setup wizard as closed for this manifest `setup.version`
+		 * so it doesn't auto-open again: in this browser (localStorage) and,
+		 * when the manifest declares `setup.dismissAction`, on the server
+		 * through the app's own setup action endpoint
+		 * (`POST /apps/{appId}/api/setup/action/{dismissAction}`, admin-only
+		 * like every setup endpoint, so the record is per instance). Then
+		 * emits `setup-wizard-dismissed`. Once per page load: finishing
+		 * records it, and the result screen's Close that follows does not
+		 * post again.
 		 *
+		 * @spec openspec/changes/setup-wizard-close-on-server/specs/cn-setup-wizard/spec.md
+		 * @param {boolean} [finished] True when the user finished the wizard, false when they closed it.
 		 * @return {void}
 		 */
-		persistSetupWizardDismissal() {
+		persistSetupWizardDismissal(finished = false) {
 			try {
 				window.localStorage.setItem(this.setupWizardDismissKey(), '1')
 			} catch {
 				// Best-effort persistence (private mode / no storage).
 			}
+			if (this.setupWizardDismissed) {
+				return
+			}
 			this.setupWizardDismissed = true
+			const action = this.setupDismissAction()
+			if (action) {
+				try {
+					Promise.resolve(axios.post(generateUrl(`/apps/${this.appId}/api/setup/action/${action}`), { finished: finished === true }))
+						.catch(() => { /* the local record still holds in this browser */ })
+				} catch {
+					// Never let a failed record break the close.
+				}
+			}
+			/**
+			 * @event setup-wizard-dismissed Emitted once when the user closes or finishes the setup wizard.
+			 * @type {{ appId: string, version: number, finished: boolean }}
+			 */
+			this.$emit('setup-wizard-dismissed', {
+				appId: this.appId,
+				version: (this.manifest && this.manifest.setup && this.manifest.setup.version) || 0,
+				finished: finished === true,
+			})
 		},
 
 		/**
@@ -3434,6 +3529,49 @@ export default {
 			persistWalkthroughProgress(this.appId, this.walkthroughConfigKey, value)
 		},
 
+		/**
+		 * Remember that the user paused the tour, at the step they were on,
+		 * so the next page load keeps it hidden until they pick "Continue".
+		 *
+		 * @spec openspec/changes/audit-round-lib-fixes/specs/cn-walkthrough/spec.md
+		 * @param {object} [where] `{ tourId, stepId, index }` from CnWalkthrough.
+		 * @return {void}
+		 */
+		onWalkthroughPause(where) {
+			const current = this.walkthroughProgressValue || {}
+			const tourId = (where && where.tourId) || current.tourId
+			if (!tourId) {
+				return
+			}
+			const value = {
+				tourId,
+				stepId: (where && where.tourId) ? (where.stepId || '') : (current.stepId || ''),
+				index: (where && where.tourId && Number.isInteger(where.index)) ? where.index : (current.index || 0),
+				version: String((this.manifest && this.manifest.version) || ''),
+				paused: true,
+			}
+			this.walkthroughProgressValue = value
+			persistWalkthroughProgress(this.appId, this.walkthroughConfigKey, value)
+		},
+
+		/**
+		 * Clear the paused mark when the user continues the tour. The step
+		 * stays; the next step change writes fresh progress anyway.
+		 *
+		 * @spec openspec/changes/audit-round-lib-fixes/specs/cn-walkthrough/spec.md
+		 * @return {void}
+		 */
+		unpauseWalkthroughProgress() {
+			const progress = this.walkthroughProgressValue
+			if (!progress || !progress.paused) {
+				return
+			}
+			const value = { ...progress }
+			delete value.paused
+			this.walkthroughProgressValue = value
+			persistWalkthroughProgress(this.appId, this.walkthroughConfigKey, value)
+		},
+
 		/**
 		 * Continue an unfinished tour from the user-settings dialog: show a
 		 * paused tour again, or start the remembered tour at its step.
@@ -3448,10 +3586,11 @@ export default {
 			}
 			setTimeout(() => {
 				const wt = useWalkthrough(this.appId, this.manifest)
+				const progress = this.walkthroughProgressValue
+				this.unpauseWalkthroughProgress()
 				if (wt.resumePaused()) {
 					return
 				}
-				const progress = this.walkthroughProgressValue
 				if (progress) {
 					wt.resumeAt(progress.tourId, progress.stepId)
 				}
@@ -3478,6 +3617,7 @@ export default {
 			setTimeout(() => {
 				const id = this.manifest.walkthrough.tours[0] && this.manifest.walkthrough.tours[0].id
 				if (id) {
+					this.unpauseWalkthroughProgress()
 					useWalkthrough(this.appId, this.manifest).restart(id)
 				}
 			}, 50)

package/src/components/CnDashboardPage/CnDashboardPage.vue

@@ -330,24 +330,26 @@
 					v-if="missingRequiredApp(item)"
 					:title="getWidgetTitle(item)"
 					:showTitle="widgetShowTitle(item)">
-					<NcEmptyContent
-						:name="installAppLabel(missingRequiredApp(item))"
-						:description="t('nextcloud-vue', 'This widget shows data from another app that isn\'t installed yet.')"
-						class="cn-dashboard-page__requires-app cn-requires-app">
-						<template #icon>
-							<Download :size="24" />
-						</template>
-						<template #action>
-							<NcButton
-								variant="primary"
-								:href="appInstallUrl(missingRequiredApp(item))">
-								<template #icon>
-									<Download :size="18" />
-								</template>
-								{{ installAppLabel(missingRequiredApp(item)) }}
-							</NcButton>
-						</template>
-					</NcEmptyContent>
+					<!-- Compact on purpose: a two-row tile is about 150px, less
+					     a title row. NcEmptyContent's 64px icon, 20px name,
+					     description and button did not fit, so the icon ran out
+					     at the top and the button at the bottom. The button
+					     already says "Install {app}", so there is no separate
+					     name, and the text keeps to two lines. -->
+					<div class="cn-dashboard-page__requires-app cn-requires-app cn-requires-app--compact"
+						data-testid="cn-requires-app">
+						<p class="cn-requires-app__description">
+							{{ t('nextcloud-vue', 'This widget shows data from another app that isn\'t installed yet.') }}
+						</p>
+						<NcButton
+							variant="primary"
+							:href="appInstallUrl(missingRequiredApp(item))">
+							<template #icon>
+								<Download :size="18" />
+							</template>
+							{{ installAppLabel(missingRequiredApp(item)) }}
+						</NcButton>
+					</div>
 				</CnWidgetWrapper>
 
 				<!-- Tile widget -->

package/src/components/CnFlowDetail/CnRunDetailSidebar.vue

@@ -226,6 +226,10 @@
 					<strong>{{ step.transition }}</strong>
 					<span class="cn-run-sidebar__hint"> · {{ step.status }}</span>
 					<span v-if="step.error" class="cn-run-sidebar__error"> · {{ step.error }}</span>
+					<!-- A send step's outcome lists, by name: who got it, who
+					     opted out, who the opt-out check could not answer for. -->
+					<CnFlowStepOutcomes v-if="step.report && step.report.messaging"
+						:report="step.report.messaging" />
 				</li>
 			</ol>
 		</component>
@@ -241,6 +245,7 @@ import CheckboxMarkedOutline from 'vue-material-design-icons/CheckboxMarkedOutli
 import DatabaseOutline from 'vue-material-design-icons/DatabaseOutline.vue'
 import FormatListBulleted from 'vue-material-design-icons/FormatListBulleted.vue'
 import Replay from 'vue-material-design-icons/Replay.vue'
+import CnFlowStepOutcomes from '../CnFlowStepOutcomes/CnFlowStepOutcomes.vue'
 import { FLOW_RUN_ACTIVE_STATUSES, useFlowStore } from '../../composables/useFlowStore.js'
 
 export default {
@@ -249,6 +254,7 @@ export default {
 	components: {
 		ArrowLeft,
 		CheckboxMarkedOutline,
+		CnFlowStepOutcomes,
 		DatabaseOutline,
 		FormatListBulleted,
 		NcAppSidebarTab,