Skip to content

Conversation

@samjewell
Copy link
Contributor

What is this feature?

This PR removes the dashboardDsAdHocFiltering feature toggle from the Grafana codebase. The functionality it controlled (AdHoc filtering for the dashboard datasource) is now permanently enabled.

Why do we need this feature?

The dashboardDsAdHocFiltering feature toggle was already enabled by default (Expression: "true") and had reached General Availability. Its removal simplifies the codebase by eliminating unnecessary conditional logic and feature toggle management overhead.

Who is this feature for?

This change is primarily for Grafana developers and maintainers, as it refactors and cleans up the codebase. End-users will experience no change in functionality, as the feature was already active.

Which issue(s) does this PR fix?:

Fixes #

Special notes for your reviewer:

Please check that:

  • It works as expected from a user's perspective. (The AdHoc filtering for dashboard datasource should always be available and functional).
  • If this is a pre-GA feature, it is behind a feature toggle. (N/A, removing a GA toggle)
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc. (Docs updated to remove toggle from list. Not a notable improvement for What's New as it's a refactor.)

Open in Cursor Open in Web

The dashboardDsAdHocFiltering feature toggle has been enabled by default
and is now in General Availability stage. This commit removes the feature
toggle and makes the AdHoc filtering functionality for the dashboard
datasource permanently available.

Changes:
- Remove feature toggle from registry.go
- Regenerate feature toggle files
- Remove conditional checks in frontend code
- Update tests to reflect permanent enablement
- Always show AdHoc Filters toggle in dashboard query editor
- Always enable dashboard datasource in DataSourcePicker for variables
@cursor
Copy link

cursor bot commented Sep 27, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions bot added area/backend area/frontend type/docs Flags the technical writing team for documentation support; auto adds to org-wide docs project labels Sep 27, 2025
@github-actions
Copy link
Contributor

@samjewell samjewell marked this pull request as ready for review September 27, 2025 21:12
@samjewell samjewell requested review from a team as code owners September 27, 2025 21:12
@samjewell samjewell requested review from grafakus and harisrozajac and removed request for a team September 27, 2025 21:12
@github-actions github-actions bot added this to the 12.3.x milestone Sep 27, 2025
@samjewell samjewell marked this pull request as draft September 29, 2025 08:45
@github-actions
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update or ping for review. Thank you for your contributions!

@github-actions github-actions bot added the stale Issue with no recent activity label Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/backend area/frontend stale Issue with no recent activity type/docs Flags the technical writing team for documentation support; auto adds to org-wide docs project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants