autobahn-api icon indicating copy to clipboard operation
autobahn-api copied to clipboard

Published 20 hours ago • verified publisher icon bundesAPI

Weitere API-Endpunkte

Open mvglasow opened this issue 2 years ago • 7 comments

Wie in #20 beschrieben, außerdem /parking für Pkw-Rastplätze (/parking_lorry ist auf Rastplätze mit Lkw-Stellplätzen beschränkt).

Hierzu musste /autobahn aus der Basis-URL in die bereits dokumentierten URLs verschoben werden.

Syntaxprüfung habe ich keine durchgeführt – wenn etwas nicht passt, bitte melden.

mvglasow avatar Mar 16 '23 20:03 mvglasow

Hi, danke dir fuer den Merge Request.

Da haben sich ein paar copy and paste fehler eingeschlichen glaube ich. Wenn roadId in der beschreibung unter path ist, dann muss es auch im path als {roadId} zu finden sein.

Kannst du versuchen deine version in den editor unter https://editor.swagger.io/ einzufuegen, dann solltest du die Fehler sehen.

image

wirthual avatar Mar 20 '23 02:03 wirthual

Danke für den Link, jetzt sollte alles passen.

mvglasow avatar Mar 20 '23 18:03 mvglasow

Done.

mvglasow avatar May 23 '23 17:05 mvglasow

If the small error in line 496 is fixed, I think we can merge. @wirthual, I guess it will change some of the python code. So I think we need a new release of the python package (update version in generator_config.yaml) after the merge?

t-huyeng avatar May 25 '23 06:05 t-huyeng

Yes, thats correct :+1:

wirthual avatar May 25 '23 16:05 wirthual

Line 496 is correct, the API does in fact return a string representation of the number. This is similar to other (already documented) cases where Boolean values are passed as strings, not Boolean values.

mvglasow avatar May 26 '23 12:05 mvglasow

Should be OK now, except for the server not reporting the expected content type – but that doesn’t look like any of my changes introduced it.

mvglasow avatar May 26 '23 14:05 mvglasow