pelias: update Pelias provider #365
Conversation
| 'point.lon': location.lng | ||
| }); | ||
| const data = await getJSON<any>(this.options.serviceUrl + '/reverse', params); | ||
| return this._parseResults(data, 'bounds'); |
There was a problem hiding this comment.
I don't recall the history of this but the bbox param isn't named bounds, someone has put effort into supporting alternative key names and I'm not sure why 🤷
|
I think this looks good, @perliedman anything else which needs cleaning up while I'm at it? If you'd like a production key for testing/whatever you can grab one from https://geocode.earth if you'd like to have it longer than the free trial just email me and I'll convert it to a forever trial. |
missinglink
force-pushed
the
pelias
branch
6 times, most recently
from
8badec1 to
8e79901
Compare
|
Hi @missinglink ! Thanks for taking the time to contribute this. Unfortunately, although I'm technically the owner of this repository, I haven't touched the code in many, many years, so I'm reluctant to review or merge anything. |
|
Hi @perliedman, ugh ok, @simon04 said he'd be happy to accept this in #364 (comment) |
Uh oh!
There was an error while loading. Please reload this page.