-
Notifications
You must be signed in to change notification settings - Fork 0
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
Enhance catalogue item properties table view filter #1128 #1157
Enhance catalogue item properties table view filter #1128 #1157
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #1157 +/- ##
========================================
Coverage 99.15% 99.16%
========================================
Files 89 89
Lines 17785 17865 +80
Branches 2966 2973 +7
========================================
+ Hits 17635 17715 +80
Misses 146 146
Partials 4 4 ☔ View full report in Codecov by Sentry. |
4ab0040
to
67a2185
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.
Looks good overall, just have a few minor fixes to suggest.
@@ -117,6 +213,7 @@ export function CatalogueItemsPropertiesTable(props: PropertiesTableProps) { | |||
data: properties, | |||
// Features | |||
enableTopToolbar: true, | |||
enableColumnFilterModes: true, |
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.
enableColumnFilterModes: true, | |
enableColumnFilterModes: true, | |
filterFns: customFilterFunctions, |
The exclude functions currently don't work on list column values. Develop needs to be merged into this branch, since this PR had some changes to the utility functions that fixes it. |
Description
Testing instructions
Add a set up instructions describing how the reviewer should test the code
Agile board tracking
closes #1128