Enhance catalogue category card filters #1124

[joshuadkitenge]

Description

see #1124

Testing instructions

Add a set up instructions describing how the reviewer should test the code

• Review code
• Check Actions build
• Review changes to test coverage
• {more steps here}

Agile board tracking

closes #1124

[codecov]

Codecov Report

Attention: Patch coverage is 85.05747% with 13 lines in your changes missing coverage. Please review.

Project coverage is 99.16%. Comparing base (718d659) to head (d24be52).
Report is 3 commits behind head on develop.

Files with missing lines	Patch %	Lines
src/utils.tsx	35.00%	13 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1156      +/-   ##
===========================================
- Coverage    99.23%   99.16%   -0.08%     
===========================================
  Files           88       88              
  Lines        17876    17932      +56     
  Branches      2123     2946     +823     
===========================================
+ Hits         17740    17782      +42     
- Misses         135      146      +11     
- Partials         1        4       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[asuresh-code]

Looks good overall, just have a few minor comments.

[asuresh-code]

I tested arrExcludesSome, and it doesn't seem to work properly on the catalogue category filters.

It should filter out Motorized Actuators which has all of these properties but does not.

When I implemented it, it wasn't meant to handle row values that are lists, so I think an implementation similar to what you have done for the 2 new functions should address it.

[joshuadkitenge]

I tested arrExcludesSome, and it doesn't seem to work properly on the catalogue category filters. It should filter out Motorized Actuators which has all of these properties but does not.

When I implemented it, it wasn't meant to handle row values that are lists, so I think an implementation similar to what you have done for the 2 new functions should address it.

I've refactored the code so now it just uses the MRT Table function. The types have been matched to same as MRT / tanstack table