-
Notifications
You must be signed in to change notification settings - Fork 11.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Search: Include collapsed panels in search v2 #83047
Conversation
/deploy-to-hg |
|
Error: Contact #proj-ephemeral-hg-instances if it is not a compile error handling pull request comment event: running grafana-build deb: executing command: stdout= stderr=go: downloading dagger.io/dagger v0.9.8 Stdout: Stderr: |
9d002c4
to
03959a6
Compare
/deploy-to-hg |
|
|
What is this feature?
This is a bug fix for panel title search in search v2.
Why do we need this feature?
Panels in collapsed rows are currently not being included in panel title search in search v2. This PR fixes this bug.
Who is this feature for?
Users with the
panelTitleSearch
feature flag enabled.Which issue(s) does this PR fix?:
Fix for https://github.com/grafana/support-escalations/issues/9284
Special notes for your reviewer:
This is my understanding of how TestReadSummaries works. (1) It takes the
with-library-panels.json
and reads it. (2) The output of what is being read is expected to match the contents ofwith-library-panels-info.json
.As part of this PR, I added a new collapsed row and panel to
with-library-panels.json
. During the reading process, the other panels get a "ds" reference even though they themselves don't contain adatasource
field. The new panel I added doesn't get a "ds" reference unless I directly include adatasource
field inwith-library-panels.json
.This didn't seem like a key aspect of the test to me, but happy to look further into it if someone thinks it would be worthwhile to do so.
Please check that: