fastapi-filter icon indicating copy to clipboard operation
fastapi-filter copied to clipboard

added contains express support

Open iakashpatel opened this issue 2 years ago β€’ 3 comments

Why this PR?

Recently I started using https://sqlalchemy-utils.readthedocs.io/en/latest/_modules/sqlalchemy_utils/types/scalar_list.html to store array of enum.

for which I need to filter table with specific value inside array of enum (TEXT/BLOB at low level) which is feasible using contains expression.

What is changed/added ?

  • Added contains expression support for sqlalchemy and testcases to support it.
  • Updated Doc/Example

PS: :star2: My first open-source contribution :)

iakashpatel avatar May 26 '23 04:05 iakashpatel

Deploy Preview for fastapi-filter ready!

Name Link
Latest commit 0cab115fb601af80c4c61de4ee4521223bd6abaf
Latest deploy log https://app.netlify.com/sites/fastapi-filter/deploys/64703a823aa568000835e50c
Deploy Preview https://deploy-preview-395--fastapi-filter.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar May 26 '23 04:05 netlify[bot]

Codecov Report

Merging #395 (0cab115) into main (64c93b9) will not change coverage. The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #395   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines          180       180           
=========================================
  Hits           180       180           
Impacted Files Coverage Ξ”
fastapi_filter/contrib/sqlalchemy/filter.py 100.00% <ΓΈ> (ΓΈ)

codecov[bot] avatar May 29 '23 13:05 codecov[bot]

Hi @iakashpatel , I'm interested in the same feature, and was wondering if you still planned to work on this PR ? If not I might consider opening one myself

raphodn avatar Feb 27 '24 13:02 raphodn