Closed psdcode closed 2 years ago
@psdcode the fixes look great, functionality is as expected.
You make some good points about the /count endpoint and adding phrases to the city search - I'll have Gustavo add these as issues. Thanks!
@psdcode the fixes look great, functionality is as expected.
You make some good points about the /count endpoint and adding phrases to the city search - I'll have Gustavo add these as issues. Thanks!
@brittwitham Sounds good! btw, in the original comment I think I meant OR instead of AND: "Backend seems to use OR on all words within a city search term"
This PR rebuilds and reenables the city filter. Resolves issues https://github.com/ajah/sks-interface/issues/49 and https://github.com/ajah/sks-interface/issues/34
Other fixes:
/
To confirm with @brittwitham :
/count
route currently does not seem to take the params other thanq
andoperator
, so count results returned do not take any other filter into account.Time spent: 2.5hrs