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.

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

PublicDashboards: Enable creation when dashboard has template variables #64560

Merged
merged 5 commits into from Mar 10, 2023

Conversation

juanicabanas
Copy link
Contributor

What is this feature?

It lets a user create a public dashboard even if the dashboard has template variables, taking into account not all panels may apply those template variables.
A warning message is still shown.

Which issue(s) does this PR fix?:

Fixes #64437

Special notes for your reviewer:

template-variables.mov

@juanicabanas juanicabanas added this to the 9.5.0 milestone Mar 9, 2023
@juanicabanas juanicabanas requested a review from a team as a code owner March 9, 2023 19:48
@juanicabanas juanicabanas self-assigned this Mar 9, 2023
@juanicabanas juanicabanas requested review from axelavargas and polibb and removed request for a team March 9, 2023 19:48
Copy link
Contributor

@evictorero evictorero left a comment

Choose a reason for hiding this comment

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

LGTM.
nit: Now validation uses only the dto this should be the first thing to do in both methods, create and update.

@juanicabanas
Copy link
Contributor Author

LGTM. nit: Now validation uses only the dto this should be the first thing to do in both methods, create and update.

@evictorero you're right. gonna make that change

@juanicabanas juanicabanas merged commit 222ad02 into main Mar 10, 2023
4 checks passed
@juanicabanas juanicabanas deleted the juanicabanas/pubdashCreationWithTemplateVar branch March 10, 2023 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: pubdash creation with template variables
3 participants