-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[UI v2] feat: Adds flow run data table filter components #17148
Conversation
e3aa54a
to
91408cd
Compare
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 suggestions but looks good!
) { | ||
return false; | ||
} | ||
return Array.from(selectedFilters).every((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.
nice, didn't know these functions existed
); | ||
}, [selectedFilters]); | ||
|
||
const handleSelectAllExceptScheduled = () => { |
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.
Could these be reduced to use the handleSelectFilter
function?
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.
hmm, I can' think of how to do it while keeping it readable.
Mostly because these type of filter selections are deriving the state of multiple filters 🤔
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.
fair enough!
91408cd
to
826eaad
Compare
Adds filtering components to be used for the flow run data table
Screen.Recording.2025-02-14.at.3.11.48.PM.mov
Checklist
<link to issue>
"mint.json
.Relates to #15512