-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: implement metricFindQuery #11
Conversation
…ni/systemlink-grafana-plugins into users/grchoi/systems-datasource
…ni/systemlink-grafana-plugins into users/grchoi/systems-datasource
Co-authored-by: Cameron Waterman <cameron.waterman@ni.com>
Co-authored-by: Cameron Waterman <cameron.waterman@ni.com>
…ni/systemlink-grafana-plugins into users/grchoi/systems-datasource
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.
We'll need to add tests for this stuff at some point but for the sake of time we can defer them for now
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.
Changing my review until open questions are resolved
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.
Changing my review until open questions are resolved
…ni/systemlink-grafana-plugins into users/grchoi/systems-datasource
# [1.8.0](v1.7.0...v1.8.0) (2023-08-11) ### Features * implement metricFindQuery ([#11](#11)) ([a5abed2](a5abed2))
🎉 This PR is included in version 1.8.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Pull Request
🤨 Rationale
When users create a query variable, the dropdown list needs to populate with all the systems the user has access to. The user should also be able to look up a system by typing in the system alias. This PR satisfies this user story.
👩💻 Implementation
The metricFindQuery method was implemented using the /query-systems route. Two projections were made to extract the id and the alias. Additionally, a new VariableQuery interface was created in types.ts.
🧪 Testing
The web dev tools were used to test this change. We checked to see if the query variable populated with all the systems that were present in the response.
✅ Checklist