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

added single request feature #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sentd94
Copy link

@sentd94 sentd94 commented Mar 31, 2023

This PR adds a new method, singleWithin, that allows users to retrieve documents near a given point as a single Future, instead of a Stream. This feature is particularly useful for users who only need to access the data once, as it simplifies the process and reduces overhead.

@sentd94 sentd94 requested a review from zeusbaba as a code owner March 31, 2023 14:36
@sentd94
Copy link
Author

sentd94 commented Mar 31, 2023

Fixes #6

@kosukesaigusa
Copy link

kosukesaigusa commented Mar 31, 2023

I also like the idea of enabling the package to fetch geo locations once as Future!

And actually, the package below, heavily inspired by geoflutterfire/geoflutterfire2, is released a few weeks ago with full respect!

https://pub.dev/packages/geoflutterfire_plus

https://github.com/KosukeSaigusa/geoflutterfire_plus

  • Almost all the existing features are available.
  • But whole codes are redesigned and refactored for future extensibility and easy maintenance.
  • Some of the definitions and names of classes, methods, and variables are different, so once you may need to refactor you code🙏
  • The fetching once as Future method is already available.
  • Some other new features such as custom query are available, and unit tests are being added aiming for 100% coverage.

If you are interested, please check geoflutterfire_plus!

Of course, feature/pull requests are always welcome🙌

Sorry for the comments from outside!

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.

2 participants