Add boundary.country filter to /v1/autocomplete #634

Merged
merged 1 commit into from Aug 17, 2016

Conversation

Projects
None yet
4 participants
@orangejulius
Member

orangejulius commented Aug 12, 2016

It works exactly the same as in /search, no surprises here!

Connects pelias/pelias#403

@orangejulius orangejulius self-assigned this Aug 12, 2016

@orangejulius orangejulius referenced this pull request in pelias/acceptance-tests Aug 12, 2016

Merged

Add test for autocomplete boundary.country #281

@orangejulius

This comment has been minimized.

Show comment
Hide comment
@orangejulius

orangejulius Aug 12, 2016

Member

It works!
screenshot from 2016-08-12 16-35-53

Member

orangejulius commented Aug 12, 2016

It works!
screenshot from 2016-08-12 16-35-53

@trescube

This comment has been minimized.

Show comment
Hide comment
@trescube

trescube Aug 12, 2016

Contributor

:shipit:

Contributor

trescube commented Aug 12, 2016

:shipit:

@dianashk

This comment has been minimized.

Show comment
Hide comment
@dianashk

dianashk Aug 12, 2016

Contributor

LGTM Wow that was quick!

Contributor

dianashk commented Aug 12, 2016

LGTM Wow that was quick!

@missinglink

This comment has been minimized.

Show comment
Hide comment
Member

missinglink commented Aug 15, 2016

cc/ @campreb

@orangejulius orangejulius merged commit a320d9a into master Aug 17, 2016

3 checks passed

approvals/lgtm this commit looks good
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@orangejulius orangejulius removed the in review label Aug 17, 2016

@orangejulius orangejulius deleted the autocomplete-boundary.country branch Aug 19, 2016

@missinglink

This comment has been minimized.

Show comment
Hide comment
@missinglink

missinglink Aug 22, 2016

Member

in production now @campreb

eg:

/v1/autocomplete?boundary.country=NZ&text=shortland st, auckland
Member

missinglink commented Aug 22, 2016

in production now @campreb

eg:

/v1/autocomplete?boundary.country=NZ&text=shortland st, auckland

je-l pushed a commit to nlsfi/pelias-api that referenced this pull request Aug 31, 2017

Merge pull request #634 from pelias/autocomplete-boundary.country
Add boundary.country filter to /v1/autocomplete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment