Skip to content
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 filter keyword in where clauses. (#74) #82

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

thromer
Copy link

@thromer thromer commented Apr 13, 2024

FieldFilter support only, not composite filters.

Doesn't support composite filters.
# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
#
# On branch master
# Your branch is up to date with 'origin/master'.
#
# Changes to be committed:
#	modified:   mockfirestore/collection.py
#	modified:   mockfirestore/query.py
#	modified:   tests/test_collection_reference.py
#
@thromer
Copy link
Author

thromer commented Apr 13, 2024

Tests pass locally, including a new test that exercises the filter keyword argument to Collection.where and Query.where.

@eddie
Copy link

eddie commented Jun 21, 2024

This looks good, @mdowds any chance this could be merged?

@ritesh-researchify-labs

Is there any plan to merge this code and release a new version

@ckanaar
Copy link

ckanaar commented Aug 1, 2024

Any status on this PR? Would be really nice to get this merged so this package works with modern .where() syntax.

@mohammadkotb
Copy link

Any updates on this?! Can this be merged and create a new release for the package?

@eddie
Copy link

eddie commented Aug 28, 2024

I have some local changes based of this PR that also implements compound queries, if I have time next week I'll fork this and make it available until this repo becomes active again.

@mohammadkotb
Copy link

Thanks @eddie that would be great!
For now I cloned the repo to my code and made the code changes and depending on it as a package.

Let me know if you'd like to fork it and publish it as a package, I can collaborate and help, I see the value in this.

@eddie
Copy link

eddie commented Sep 5, 2024

I have forked the repo, added @thromer 's changes and also initial compound query support, please test as its a bit crude! Hopefully we can get this merged back into the main repo in time. Thanks all.

https://github.com/eddie/python-mock-firestore

https://pypi.org/project/mock-firestore-plus/0.12.0/

Also, update Python version numbers and bump our version number.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants