A tile whose owned query is missing or shared cannot be removed from any surface
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Da chiarire
- Stato di attività
- Tranquilla
- Stack tecnologico
- typescript
- Ambito
- frontend
Direzione di ricerca
Inizia da src/dashboard/application/dashboard-removal.ts:128-131, quindi segui i percorsi del cestino dell’albero Dashboards e di Remove tile nell’intestazione del riquadro, incluso il comando remove-tile esistente. Determina quale opzione documentata si applica alle query mancanti, preservando il rifiuto per le query condivise. Il lavoro è completato quando un percorso di rimozione scelto funziona per i riquadri orfani dalla superficie pertinente senza eliminare una query.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
removeDashboardPanel proves ownership before deleting anything and refuses
ownership-unproven when the tile's queryId names no query in
workspace.queries, or names one with more than one owner
(src/dashboard/application/dashboard-removal.ts:128-131). Both the Dashboards
tree and — since #537 — the tile header route their delete through it.
That is the correct fail-closed behaviour #494/#537 specify. The consequence is
that a tile whose query is missing (a dangling queryId) or shared by a
second tile now has no removal path from any surface:
- the Dashboards-tree trash is disabled with a reason;
- the tile header's
⋯→ Remove tile row is disabled with the same reason; - and #537 deleted the last escape hatch — the tile head's old document-only
remove-tiledispatch, which removed such a tile without touchingqueries.
How to reach it
A queries-only bundle import (#448) or a corrupt aggregate can produce a tile
pointing at an id no query carries. A legacy or hand-edited workspace can carry
two tiles referencing one query, which #427 forbids reaching but does not
retroactively repair.
Why it was deferred
#537's acceptance criteria require exactly this fail-closed behaviour, so the
alternative belongs in its own decision rather than smuggled into that change.
Options
- A separate, explicitly-labelled "Remove tile only" action, offered only
when removal is refused because the query is missing — a dangling reference has
no second resource to be atomic about. Would keep theremove-tilecommand,
which currently has no UI dispatcher left. - Give
removeDashboardPanelan explicit tile-only arm for the missing-query
case, so one path still covers it. - Accept it, and handle these workspaces through Import/Replace only.
Option 1 or 2 seems right for the missing-query case; a shared query arguably
should stay unremovable, since either tile still renders it.
- Lingua principale
- TypeScript
- Stelle
- 8
- Fork
- 2
- Merge medio
- 1h 34m
- PR unite (30g)
- 6
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Altinity/altinity-sql-browser
-
inbox
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Altinity/altinity-sql-browser#605 ·
-
inbox
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Altinity/altinity-sql-browser#509 ·
-
inbox
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Altinity/altinity-sql-browser#489 ·
-
flamegraph Apertaenhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
Altinity/altinity-sql-browser#684 ·
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 68/100
Altinity/altinity-sql-browser#680 · 2 commenti ·
Tutte le issue di Altinity/altinity-sql-browser
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
vercel-labs/just-bash#464 ·
-
looksLikeSlug() is ASCII-only, so non-Latin entity slugs (e.g. Korean) skip exact match and collapse Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
TanStack/tanstack.com#1293 ·