-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
support for array_filters #2769
Conversation
Co-authored-by: Bastien Gérard <[email protected]>
Co-authored-by: Bastien Gérard <[email protected]>
Co-authored-by: Bastien Gérard <[email protected]>
Co-authored-by: Bastien Gérard <[email protected]>
Co-authored-by: Bastien Gérard <[email protected]>
Co-authored-by: Bastien Gérard <[email protected]>
…into array_filters # Conflicts: # docs/changelog.rst
Putting it in The 2 alternatives that I see are
or
The second one seems to be slightly better in my opinion as it's not too far from the native pymongo api
What do you think? |
@bagerard |
Argh indeed I misunderstood that, let me re comment it accordingly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix latest comments and it will be good to go
…into array_filters # Conflicts: # docs/changelog.rst
Looks good, thanks for the contribution 👍 |
This is just what I am now looking for. 👏 |
Add option to user array_filters in mongoengine