Skip to content

fix: respect saved Table view on Dashboard - #369

Merged
BorisTyshkevich merged 1 commit into
mainfrom
fix/dashboard-explicit-table-view-368
Jul 22, 2026
Merged

fix: respect saved Table view on Dashboard#369
BorisTyshkevich merged 1 commit into
mainfrom
fix/dashboard-explicit-table-view-368

Conversation

@BorisTyshkevich

@BorisTyshkevich BorisTyshkevich commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

What & why

Fixes the Dashboard presentation resolver so a saved spec.view: "table" becomes an explicit Table base presentation before runtime auto-detection, including legacy panel metadata without a cfg. Explicit panel.cfg remains authoritative; named variants and tile-local overrides continue to layer over the effective base.

Closes #368

Checklist

  • npm test passes (the per-file coverage gate is non-negotiable)
  • Tests added/updated in the same change as the code
  • npm run build succeeds (single-file dist/sql.html)
  • Layers kept honest: pure logic in src/core/, network in src/net/ (injected fetch), DOM in src/ui/
  • No new runtime dependency
  • CHANGELOG.md [Unreleased] updated
  • Reconciled affected tracked work; no roadmap or ADR change was needed for this isolated compatibility bug

Resolve legacy view: 'table' as an explicit Table presentation before automatic panel detection, while retaining panel, variant, and override precedence.

Co-Authored-By: OpenAI Codex <noreply@openai.com>

Claude-Session: unavailable (OpenAI Codex)
@BorisTyshkevich
BorisTyshkevich force-pushed the fix/dashboard-explicit-table-view-368 branch from dec4eeb to 770fc6b Compare July 22, 2026 10:07
@BorisTyshkevich
BorisTyshkevich merged commit 1c7c4b0 into main Jul 22, 2026
9 checks passed
BorisTyshkevich added a commit that referenced this pull request Jul 22, 2026
@BorisTyshkevich
BorisTyshkevich deleted the fix/dashboard-explicit-table-view-368 branch August 6, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dashboard tile ignores saved view: "table" and auto-renders a chart

1 participant