Skip to content

feat: persist explore chart settings in URL state#9169

Merged
djbarnwal merged 2 commits intomainfrom
feat/url-state-chart-config
Apr 3, 2026
Merged

feat: persist explore chart settings in URL state#9169
djbarnwal merged 2 commits intomainfrom
feat/url-state-chart-config

Conversation

@djbarnwal
Copy link
Copy Markdown
Member

Currently, the Dynamic Y-Axis Scale and Always-On Line Chart settings in Explore Dashboards are ephemeral local component state. When a user configures these options and shares the dashboard URL, recipients see the default view instead of the intended configuration.

This PR promotes dynamicYAxisScale and forceLineChart from local component state in MetricsTimeSeriesCharts.svelte to the ExploreState store and persists them in the URL as dyn_y and line_chart params. The full established pipeline is used: proto ExplorePreset → URL → V1ExplorePresetExploreState round-trip.

  • Added chart_dynamic_y_axis and chart_force_line fields to the proto ExplorePreset message
  • Registered params as view-specific for Explore and TDD views (not Pivot), with cross-view preservation
  • Default behavior unchanged when settings are absent from the URL

https://linear.app/rilldata/issue/APP-806/add-dynamic-y-axis-scale-and-always-line-chart-configs-as-part-of

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@djbarnwal djbarnwal requested a review from AdityaHegde April 2, 2026 15:59
@djbarnwal djbarnwal merged commit c1c6ca1 into main Apr 3, 2026
23 of 30 checks passed
@djbarnwal djbarnwal deleted the feat/url-state-chart-config branch April 3, 2026 07:49
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.

2 participants