meilisearch / milli

Search engine library for Meilisearch ⚡️
MIT License
464 stars 81 forks source link

Geosearch for zero radius #721

Closed amab8901 closed 1 year ago

amab8901 commented 1 year ago

Pull Request

Related issue

Fixes #3167 (https://github.com/meilisearch/meilisearch/issues/3167)

What does this PR do?

PR checklist

Please check if your PR fulfills the following requirements:

Thank you so much for contributing to Meilisearch!

amab8901 commented 1 year ago

idk, maybe this will work, maybe not. But with each attempt, we're one step closer. Is it fine that I have like 2 commits instead of one? Or do you want me to squash those two into one? For some reason this one Merge branch 'meilisearch:main' into 3167-geosearch-zero doesn't show up in git log

irevoire commented 1 year ago

But I still can't see anything with your first commit :pleading_face: image

irevoire commented 1 year ago

Since you still have the same issue over there, let's fix the issue in your original commits! It shouldn't be too hard if we do it step by step :grin: