add $contains, $contained_by, and $overlap operators#236
add $contains, $contained_by, and $overlap operators#236michalgm wants to merge 2 commits intofeathersjs:masterfrom
Conversation
|
Thank you! Can we add tests for those as well? |
|
Odd - could have sworn I commented on this back in November. I looked into adding tests, but since these features only apply to postgres, I couldn't see how to add a test without making postgres a dependency for testing, at least with the current test framework. I based the addition on the ilike operator, which is also postgrest-only. If you give me any pointers on how to properly add tests for these operators, I'm happy to look into it. |
|
Any update on a release for this feature? Works completely as expect and would like to have this officially! |
|
Hi - checking in to see if I can get any guidance on what would be needed to get this merged? Thanks! |
|
Any guidance on how to use this while we wait on it being merged? |
|
this worked: yarn add https://github.com/michalgm/feathers-knex#c3caff90a63d1bcd07e6f19994139f9ec7220471 |
77b06b8 to
ee62eb0
Compare
Summary
Simply adds 3 new array-type operators to OPERATORS and whitelist, as well as documentation
Currently, interacting with array-type fields would require complex and/or queries
Issue Add array-type operators #235
No
Other Information
Tested to confirm queries are written as expected