I've tested this patch with production data and the functionality is looking great!
I'm prepared to add my signoff, but have come across a few issues that would be great to get resolved if they're easy fixes.
- Text filters (Contains, Starts with, Ends with, etc.) are case sensitive, which could cause confusion. Case INsensitive filters would be much more useful
- Columns with Yes/No values have filter options of True/False/Any. Filter options should match the column values.
- Interval data types aren't filterable. Instead of the filter button, 'I don't know how to filter <column name>' appears. Examples: Circulation Duration Rules, Circulation Recurring Fine Rules.
Since this fix provides much needed missing functionality, I am willing to signoff and open new bugs for the above issues, but might some of these be quick to fix?
I've tested this patch with production data and the functionality is looking great!
I'm prepared to add my signoff, but have come across a few issues that would be great to get resolved if they're easy fixes.
- Text filters (Contains, Starts with, Ends with, etc.) are case sensitive, which could cause confusion. Case INsensitive filters would be much more useful
- Columns with Yes/No values have filter options of True/False/Any. Filter options should match the column values.
- Interval data types aren't filterable. Instead of the filter button, 'I don't know how to filter <column name>' appears. Examples: Circulation Duration Rules, Circulation Recurring Fine Rules.
Since this fix provides much needed missing functionality, I am willing to signoff and open new bugs for the above issues, but might some of these be quick to fix?