raphiniert-com / ra-data-postgrest

react admin client for postgREST
MIT License
110 stars 34 forks source link

feat: add logical operator filter parsing #82

Closed jordiyapz closed 1 year ago

jordiyapz commented 1 year ago

Support logical operators OR and AND.

This pull request add the ability to use or filter such as:

<FilterListItem label="Under 18 or over 21" value={{ "@or": {  "age@lt": 18, "age@gt": 21 } }}/>
coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 5205371005


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/urlBuilder.ts 14 15 93.33%
<!-- Total: 14 15 93.33% -->
Totals Coverage Status
Change from base Build 5188225735: 0.3%
Covered Lines: 253
Relevant Lines: 265

💛 - Coveralls
scheiblr commented 1 year ago

Okay, as soon as you added the one requested test case and renamed the logics variable, I would love to merge it :) Also I would appreciate, if you could do a merge upstream, as I merged #81