Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

/search 'layers' params #22

Closed
missinglink opened this issue Sep 25, 2014 · 4 comments
Closed

/search 'layers' params #22

missinglink opened this issue Sep 25, 2014 · 4 comments
Labels

Comments

@missinglink
Copy link
Member

regression: /search 'layers' param no longer working after the latest query changes.

https://pelias.mapzen.com/search?input=oregon&lat=40.6863924&lon=-73.9881961&layers=admin0

@hkrishna
Copy link
Contributor

Yeah, I think this happened when we switched queries to use https://github.com/geopipes/elasticsearch-backend/blob/master/query/geo_distance.js. We no longer use params.layers in https://github.com/pelias/api/blob/master/query/search.js

Notice how it doesn't take layers as one of the options. So, maybe we add support for layers in https://github.com/geopipes/elasticsearch-backend/blob/master/query/reverse_geo_base.js#L8 ? Thoughts?

@hkrishna hkrishna added the bug label Oct 13, 2014
@missinglink
Copy link
Member Author

I don't think the query needs to be changed, instead if we add a new param after this line named type as per this reference then we can control which types are searched.

@hkrishna
Copy link
Contributor

yep, that works 👍

@missinglink
Copy link
Member Author

resolved by #31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants