-
Notifications
You must be signed in to change notification settings - Fork 68
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 example to demonstrate use georadius. #104
Conversation
Codecov Report
@@ Coverage Diff @@
## master #104 +/- ##
==========================================
+ Coverage 76.63% 76.81% +0.18%
==========================================
Files 13 13
Lines 1100 1100
==========================================
+ Hits 843 845 +2
+ Misses 199 198 -1
+ Partials 58 57 -1
Continue to review full report at Codecov.
|
@mr-shitij GEOFILTER is expressed as long,lat as clearly stated on docs:
Same applies to the redis
Noneteless I'll fix your example, add another godoc example to make it clear. |
Kudos, SonarCloud Quality Gate passed! |
@mr-shitij notice I've updated your PR given there was no real bug on the client. I made usage of your example and fixed it. Also included the godoc variant so that people seeing our docs can check It. |
Thank You ..!! |
According to standards and as used in redis server we always specify latitude first and then longitude. That's why geofilter is not working in redisearch-go client. I Fixed that bug by changing their position and also added an example to demonstrate how to use geofilter filter in redisearch-go client.
I hope you will consider.