Skip to content

Conversation

@briangregoryholmes
Copy link
Contributor

@briangregoryholmes briangregoryholmes commented Dec 15, 2025

As a follow up to #8481, this PR hides time series specific options from the KPI component editor params when no time series is present in the underlying metrics view.

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!

@briangregoryholmes briangregoryholmes changed the title [APP-640] feat: hide time series options on KPI [APP-640] feat: hide KPI time series options when no time series present Dec 15, 2025
@briangregoryholmes briangregoryholmes merged commit 8580891 into main Dec 17, 2025
10 checks passed
@briangregoryholmes briangregoryholmes deleted the bgh/hide-time-series-options-kpi branch December 17, 2025 04:37
briangregoryholmes added a commit that referenced this pull request Dec 17, 2025
…ent (#8506)

* feat: hide time series options on KPI

* remove log

* init sequencing
k-anshul pushed a commit that referenced this pull request Dec 18, 2025
…ent (#8506)

* feat: hide time series options on KPI

* remove log

* init sequencing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants