Jonathan Beliën

Results 216 comments of Jonathan Beliën

> Can you tell us (all who need the solution and all who read this issue, and so on ...) where in which file of geocoder-php/nominatim we have to change...

Please dive in our code a little bit more (or check how it's done in other providers), we have the `GeocodeQuery::withData()` function exactly for that purpose: https://github.com/geocoder-php/Geocoder/blob/master/src/Common/Query/GeocodeQuery.php#L131

Hello @AntoineLemaire , Thank you for your contribution. It looks good to me. Could you add a test for this new feature ? Thanks.

Thanks for your feedback @marcosfreitas ! It should work with `"minimum-stability": "stable"`, I'll check why it doesn't !

Hello, Can you make sure your PR passes all the tests ? Thanks a lot !

Could you also add some specific tests ? Check another [IP provider](https://github.com/geocoder-php/Geocoder#ip) to have "examples".

@benemohamed Would you have time to finish this PR (and merge our `master` to fix the conflict) ? Thanks.

Thanks for the notice @Chris53897, I guess we can close this PR then.

Hello @oichou , I'm a bit confused ... Why did you close #1132 to open a new PR with the exact same modification ?

Oh okay, it was not necessary but not worries. I'll check it ASAP.