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.

Sign up for GitHub

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

Jump to bottom

/place endpoint should accept short or full source names #441

Closed
orangejulius opened this issue Mar 9, 2016 · 1 comment
Closed

/place endpoint should accept short or full source names #441

orangejulius opened this issue Mar 9, 2016 · 1 comment
Assignees
@avulfson17
Labels
help wanted low hanging fruit
Milestone
Who's on First (P...

Comments

@orangejulius
Copy link
Member

orangejulius commented Mar 9, 2016 •
edited by dianashk
Loading

Compare http://pelias.github.io/compare/#/v1/place%3Fids=gn:venue:2797943

to

http://pelias.github.io/compare/#/v1/place%3Fids=geonames:venue:2797943
@orangejulius orangejulius added this to the Who's on First milestone Mar 9, 2016
@riordan
Copy link
Contributor

riordan commented Mar 9, 2016

👍

@avulfson17 avulfson17 mentioned this issue Jun 30, 2016
Merged
avulfson17 added a commit to pelias/acceptance-tests that referenced this issue Jul 6, 2016
@avulfson17
Added description for failing test
594aa99 
The shortened sources acceptance test currently fails when it should pass.
Fixes pelias/api#441
@avulfson17 avulfson17 mentioned this issue Jul 7, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@avulfson17 avulfson17

Labels
help wanted low hanging fruit
Projects
None yet
Milestone
Who's on First (Phase 2)
Development

No branches or pull requests
5 participants
@orangejulius @riordan @dianashk @avulfson17 and others