Nominatim.API icon indicating copy to clipboard operation
Nominatim.API copied to clipboard

Published 20 hours ago • verified publisher icon f1ana

adding search API

Open sonofsatoshi opened this issue 1 year ago • 4 comments

Adding https://nominatim.org/release-docs/latest/api/Search/

sonofsatoshi avatar Oct 24 '23 23:10 sonofsatoshi

Hello, could you add test coverage please? Specifically integration tests so we can confirm its functionality prior to bundling.

f1ana avatar Nov 12 '23 17:11 f1ana

During the tests, in "GeocoderTests.cs", on line 54, I have the following error "System.NullReferenceException: Object reference not set to an instance of an object." (r is null). Is this what is planned?

J-Lacerte avatar Nov 13 '23 13:11 J-Lacerte

Oops!, the error is on line 88.

J-Lacerte avatar Nov 13 '23 14:11 J-Lacerte

Tests added, fixed existing test to make all green

sonofsatoshi avatar Nov 21 '23 12:11 sonofsatoshi