Comment 19 for bug 1846042

Revision history for this message
Michele Morgan (mmorgan) wrote :

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?