Skip to content

Distance query#54

Draft
apricot13 wants to merge 1 commit intodevelopfrom
feature/sort-by-distance-query
Draft

Distance query#54
apricot13 wants to merge 1 commit intodevelopfrom
feature/sort-by-distance-query

Conversation

@apricot13
Copy link
Member

requirements:

If there is just a keyword, order by keyword relevance
If there is just a location, search within a 5 mile radius and order by distance
If there is a keyword(s) AND location, search within a 5 mile for results that match the keyword(s), and order by distance

This PR puts back the logic to the old way of doing this, its less efficient and when location is returned it shows results nearest to furthest disregarding keyword score

Todo:

  • tests will need updating/reverting
  • the countQuery needs refining when location is present

These should auto deploy to heroku test environments when merged

companion PR for this is:

When merged into develop, then staging, then production they will automatically deploy to

https://staging--bucks-bod-directory.netlify.app/
https://staging--bucks-fis-directory.netlify.app/
https://staging--bucks-fis-send-directory.netlify.app/

and then

https://directory.buckinghamshire.gov.uk/
https://directory.familyinfo.buckinghamshire.gov.uk/
https://directory.send.buckinghamshire.gov.uk/

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.

1 participant