Peter Johnson

Results 1014 comments of Peter Johnson

Heya, I just got back from Kosovo and super exhausted, just wanted to send you a quick note that this is on my todo list.

These API params are available for `/v1/autocomplete` but not `/v1/search`, we've been moving toward having parity between the two APIs so I think it might be nice to have this...

Hi @michaelkirk, I've merged https://github.com/pelias/query/pull/134, can you please update this PR so the `package.json` dependency points to the latest [pelias/query](https://www.npmjs.com/package/pelias-query?activeTab=versions) (which should be 11.2): > npm install pelias-query@latest --save .....

You didn't miss the `macrocounty` one, I added that a few weeks ago in https://github.com/pelias/api/pull/1552 🙏

Hi @sathibault unfortunately this is contrary to the feature request to have these rows deduplicated. Most users expect only a single copy of a place in the results, even when...

Can `boundary.gid` be used to replace your forked version which filters on `locality_id` and `county_id`? eg. https://pelias.github.io/compare/#/v1/autocomplete?boundary.gid=whosonfirst%3Alocality%3A1495123997&text=cafe

Hi @stefanocudini, I've fixed the core issue in https://github.com/pelias/api/pull/1642, we never intended this use-case (use as a library) but it should now be possible. I don't want to encourage users...

I would support this, the new SQLite code is designed in a way which can fairly easy accommodate this. We would not be able to change the existing behaviour for...

If we do this, I'd also like to move this code somewhere else: https://github.com/pelias/whosonfirst/blob/master/utils/download_data.js#L17-L50 It got kind of messy and I regret it being piled in that file when it...

The existing code checks the timestamp on disk vs the timestamp on the server and doesn't re-download data if it's not required.