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

Chained variables #60

Merged
merged 5 commits into from
Sep 10, 2024
Merged

Chained variables #60

merged 5 commits into from
Sep 10, 2024

Conversation

heidmotron
Copy link
Contributor

@heidmotron heidmotron commented Sep 7, 2024

What does this PR do?

  • Adds support for chained template variables.
  • Allows for scopedVariables to override template variables when a panel has a repeat option set

image

How does this impact users?

Users can now create template variables with UQL filter expressions. Those filter expressions can have template variables as well.

For example, when a service variable exists and we're defining an operation variable we can make it dependent on the service variable with the following UQL filter expression:

service == $service

This will then scope the operation values to the value(s) in $service

What needed to change in the code and why?

  • Introduced a new form element for updating the scope expression
  • When requesting attributes values, we'll always serialize the template variables like we do when querying.

How did you test this?

Manual

@heidmotron heidmotron marked this pull request as ready for review September 10, 2024 17:26
</div>

<div>
<Field label="UQL Filter Expression">
Copy link
Member

Choose a reason for hiding this comment

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

Oh nice, this is a great simple way to provide this 🏆

Copy link
Member

@DHedgecock DHedgecock left a comment

Choose a reason for hiding this comment

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

🎉 awesome upgrade

@heidmotron heidmotron merged commit 138ff49 into main Sep 10, 2024
1 check passed
@heidmotron heidmotron deleted the chained-variables branch September 11, 2024 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants