-
Notifications
You must be signed in to change notification settings - Fork 1
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(system): Filter system metadata by workspace and variable chaining #43
feat(system): Filter system metadata by workspace and variable chaining #43
Conversation
commit eb3a231 Author: Cameron Waterman <cameron.waterman@ni.com> Date: Thu Oct 5 09:19:51 2023 -0500 Address review feedback commit 723138e Author: Cameron Waterman <cameron.waterman@ni.com> Date: Wed Oct 4 11:05:24 2023 -0500 Remove unnecessary clearAllMocks commit ec5db21 Author: Cameron Waterman <cameron.waterman@ni.com> Date: Wed Oct 4 10:58:21 2023 -0500 feat(tag): Workspace variable resolution
Co-authored-by: Melissa Hilliard <63749096+MelissaHilliard@users.noreply.github.com>
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.
Maybe we could add a test to SystemVariableQueryEditor.test.tsx
that tests if a user can select a variable option.
Yeah I thought this as well, but wasn't having much luck actually getting the current variable name in the context of the variable query editor. I'll spend a bit more time looking into this today. |
π This PR is included in version 3.7.0 π The release is available on:
Your semantic-release bot π¦π |
## [3.7.0](v3.6.0...v3.7.0) (2023-10-09) ### Features * **system:** Filter system metadata by workspace and variable chaining ([#43](#43)) ([14de6ed](14de6ed))
Pull Request
π€¨ Rationale
https://ni.visualstudio.com/DevCentral/_workitems/edit/2527391
https://ni.visualstudio.com/DevCentral/_workitems/edit/2541892
π©βπ» Implementation
metricFindQuery
.π§ͺ Testing
Added tests and existing tests continue to pass.
β Checklist