From 8f6118c0e128f61247d70713f67fdd2f488e6ec4 Mon Sep 17 00:00:00 2001 From: Diego Imbert Date: Wed, 30 Sep 2026 20:19:05 +0200 Subject: [PATCH] fix: Archive button for archive-only deletes, single delete doc, inert Multiple triggers label in View Co-Authored-By: Claude Opus 5.5 (1M context) --- .../src/lib/components/assets/AssetGraph/AssetNode.svelte | 6 ++++++ .../src/lib/components/assets/AssetGraph/assetsOnlyView.ts | 1 - .../routes/(root)/(logged)/pipeline/[folder]/+page.svelte | 6 +++++- 3 files changed, 11 insertions(+), 2 deletions(-) diff --git a/frontend/src/lib/components/assets/AssetGraph/AssetNode.svelte b/frontend/src/lib/components/assets/AssetGraph/AssetNode.svelte index 37006afa0b..6e6c8e03f9 100644 --- a/frontend/src/lib/components/assets/AssetGraph/AssetNode.svelte +++ b/frontend/src/lib/components/assets/AssetGraph/AssetNode.svelte @@ -468,6 +468,12 @@ fix={data.foldedFixes[0]} label={`${data.foldedFixes.length} error${data.foldedFixes.length === 1 ? '' : 's'} ยท Fix`} /> + {:else if upstream && coTriggers.length > 1 && !coTriggers.some((t) => t.onOpen)} + + Multiple triggers {:else if upstream && coTriggers.length > 1} diff --git a/frontend/src/lib/components/assets/AssetGraph/assetsOnlyView.ts b/frontend/src/lib/components/assets/AssetGraph/assetsOnlyView.ts index ac2b366a93..c7bf85d144 100644 --- a/frontend/src/lib/components/assets/AssetGraph/assetsOnlyView.ts +++ b/frontend/src/lib/components/assets/AssetGraph/assetsOnlyView.ts @@ -19,7 +19,6 @@ export type ViewEdge = { export const noAssetNodeId = (runnableId: string) => `no-asset:${runnableId}` export const isNoAssetNode = (id: string) => id.startsWith('no-asset:') -/** The script and trigger an assets-only asset's delete removes. */ /** What an assets-only delete removes: the script, its trigger, or both. */ export type AssetUpstreamDelete = { script?: { path: string; unsaved: boolean } diff --git a/frontend/src/routes/(root)/(logged)/pipeline/[folder]/+page.svelte b/frontend/src/routes/(root)/(logged)/pipeline/[folder]/+page.svelte index 46b11dc76f..9541b77fc7 100644 --- a/frontend/src/routes/(root)/(logged)/pipeline/[folder]/+page.svelte +++ b/frontend/src/routes/(root)/(logged)/pipeline/[folder]/+page.svelte @@ -3531,7 +3531,11 @@ : assetDeleteTarget?.script ? `${canHardDeleteScripts ? 'Delete' : 'Archive'} script?` : 'Delete trigger?'} - confirmationText="Delete" + confirmationText={assetDeleteTarget?.script && !canHardDeleteScripts + ? assetDeleteTarget.trigger + ? 'Archive and delete' + : 'Archive' + : 'Delete'} onConfirmed={confirmAssetUpstreamDelete} onCanceled={() => { if (!assetDeleteLoading) assetDeleteTarget = undefined