Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Dashboards: Allow to retrieve dashboards by uid #48522

Merged
merged 3 commits into from
May 17, 2022
Merged

Dashboards: Allow to retrieve dashboards by uid #48522

merged 3 commits into from
May 17, 2022

Conversation

spinillos
Copy link
Member

It allows to retrieve dashboards given the uids. We need this change for Enterprise.

@spinillos spinillos added the no-backport Skip backport of PR label Apr 29, 2022
@spinillos spinillos added this to the 9.0.0 milestone Apr 29, 2022
@spinillos spinillos requested a review from a team as a code owner April 29, 2022 11:17
@spinillos spinillos requested review from papagian, kylebrandt and ying-jeanne and removed request for a team April 29, 2022 11:17
@spinillos spinillos marked this pull request as draft April 29, 2022 11:17
@spinillos spinillos marked this pull request as ready for review May 6, 2022 11:20
@spinillos spinillos added the no-changelog Skip including change in changelog/release notes label May 6, 2022
joanlopez
joanlopez approved these changes May 6, 2022
View reviewed changes
Copy link
Contributor

@joanlopez joanlopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@spinillos spinillos merged commit 43e34f3 into main May 17, 2022
@spinillos spinillos deleted the report/update-api-for-multiple-dashboards branch May 17, 2022 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joanlopez joanlopez joanlopez approved these changes

@papagian papagian Awaiting requested review from papagian papagian was automatically assigned from grafana/backend-platform

@kylebrandt kylebrandt Awaiting requested review from kylebrandt kylebrandt was automatically assigned from grafana/backend-platform

@ying-jeanne ying-jeanne Awaiting requested review from ying-jeanne ying-jeanne was automatically assigned from grafana/backend-platform

@Clarity-89 Clarity-89 Awaiting requested review from Clarity-89

@undef1nd undef1nd Awaiting requested review from undef1nd

Assignees
No one assigned
Labels
area/backend no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes
Projects
None yet
Milestone
9.0.0-beta1
Development

Successfully merging this pull request may close these issues.
3 participants
@spinillos @joanlopez @grafanabot