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
Filtered Rows and Columns in variables list #26196
Filtered Rows and Columns in variables list #26196
Conversation
(variable) => variable.fieldType !== 'Column' && variable.fieldType !== 'Row' | ||
); | ||
|
||
return { variables: variables }; |
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.
return { variables }
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.
+1
vm$ = this.store.vm$.pipe( | ||
map((res) => { | ||
const variables = res.variables.filter( | ||
(variable) => variable.fieldType !== 'Column' && variable.fieldType !== 'Row' |
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.
I wonder if we should do an enum for Column/Row
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.
Also, can we add a test for this filter?
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.
+1
vm$ = this.store.vm$.pipe( | ||
map((res) => { | ||
const variables = res.variables.filter( | ||
(variable) => variable.fieldType !== 'Column' && variable.fieldType !== 'Row' |
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.
Also, can we add a test for this filter?
vm$ = this.store.vm$.pipe( | ||
map((res) => { | ||
const variables = res.variables.filter( | ||
(variable) => variable.fieldType !== 'Column' && variable.fieldType !== 'Row' |
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.
+1
(variable) => variable.fieldType !== 'Column' && variable.fieldType !== 'Row' | ||
); | ||
|
||
return { variables: variables }; |
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.
+1
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.
Some changes have been requested
SonarQube Quality Gate |
π€ Generated by Copilot at 07b0f3a
Summary
πΊοΈπ°π
Improved container code editor to allow filtering variables by
fieldType
. Modifieddot-add-variable.component.ts
to usemap
operator for filtering.Walkthrough
map
operator fromrxjs
to transform the store data (link)Additional Info
** any additional useful context or info **
Screenshots