Using new layers view to filter layers #580

Merged
merged 1 commit into from Jul 19, 2016

Conversation

Projects
None yet
4 participants
@avulfson17
Contributor

avulfson17 commented Jul 7, 2016

Needs pelias/query#28 to be merged first
Fixes #579

@avulfson17 avulfson17 added the in review label Jul 7, 2016

@avulfson17 avulfson17 self-assigned this Jul 7, 2016

@dianashk dianashk added the in progress label Jul 7, 2016

@avulfson17 avulfson17 removed the in review label Jul 7, 2016

@trescube

This comment has been minimized.

Show comment
Hide comment
@trescube

trescube Jul 7, 2016

Contributor

:shipit:

Contributor

trescube commented Jul 7, 2016

:shipit:

@orangejulius

This comment has been minimized.

Show comment
Hide comment
@orangejulius

orangejulius Jul 19, 2016

Member

I rebased this PR after merging pelias/query#31 and #590, and the tests now pass!

LGTM

Member

orangejulius commented Jul 19, 2016

I rebased this PR after merging pelias/query#31 and #590, and the tests now pass!

LGTM

@orangejulius orangejulius merged commit 46d6c11 into master Jul 19, 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 deleted the layers-query branch Jul 22, 2016

orangejulius added a commit that referenced this pull request Jul 22, 2016

Set up layer filtering for autocomplete and reverse
This was missed by me when working on #580, but caught by the acceptance tests!

Unfortunately it was caught after going to production.

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

Merge pull request #580 from pelias/layers-query
Using new layers view to filter layers

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

Set up layer filtering for autocomplete and reverse
This was missed by me when working on pelias#580, but caught by the acceptance tests!

Unfortunately it was caught after going to production.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment