-
Notifications
You must be signed in to change notification settings - Fork 58
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
Bad implementation #63
Comments
@alexandregiordanelli If you do have an issue with how the library performs I invite you to open a pull request with suggested changes, rather than just personal attacks towards me. Now in regards to some points you mention:
Point is I like your idea, but I do not appreciate the manner you expressed yourself and will ask that you try to be more courteous moving forward. Otherwise I will just close the issue and block you. |
Sorry man! I didnt see the startAt and endAt filter.. I think you are right. I edited my issue and remove bad words.. Really sorry.. It is sad firestore doesnt have native geoqueries. thanks for share yor thoughts |
Thank you for the apology @alexandregiordanelli , and I'm sorry if I came off a little strong. Anyway, I took your suggestion of using the |
There is not filter when query on firestore. In this project, you are getting all documents of collection and filter on client side. It is a big problem, because firestore has filters like .where and firebase realtime not.
https://www.youtube.com/watch?v=mx1mMdHBi5Q
The engineer of firebase/google recommends this code:
https://jsbin.com/mosiza/edit?js,output
This solutions is scalable..
The text was updated successfully, but these errors were encountered: