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
DashboardScene: Fixes issue referring to library panel in dashboard data source #87125
Conversation
Hello @torkelo!
Please, if the current pull request addresses a bug fix, label it with the |
This PR must be merged before a backport PR will be created. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - tried locally and it fixes the bug!
…ashboard data source (#87173) DashboardScene: Fixes issue referring to library panel in dashboard data source (#87125) * DashboardScene: Fixes issue using a library panel as source for dashboard data source * Added test (cherry picked from commit 4034a26) Co-authored-by: Torkel Ödegaard <torkel@grafana.com>
…refresh logic (#87232) DashboardScene: adjust lib panel as a dashboard datasource refresh logic (#87120) * activate library viz panel and adjust id extractor * revert parts that are fixed in #87125 * clean up after activation handler (cherry picked from commit afbd02c) Co-authored-by: Sergej-Vlasov <37613182+Sergej-Vlasov@users.noreply.github.com>
Not sure how this has worked at all recently (since we moved the key from VizLibraryPanel down to VizPanel). But I could not create a new panel that was using dashboard data source and pick a library panel as the source without seeing "VizPanel key could not be translated to a legacy numeric panel id" errors and the panelId in the targets model ended up being 0.