Weitere API-Endpunkte

[mvglasow]

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.

[wirthual]

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.

[mvglasow]

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

[mvglasow]

Done.

[t-huyeng]

Sorry, just run the openapi check again locally, and this either needs to be a string' not a number' or line 496 needs to be changed.

 496:20    error  oas3-valid-schema-example  "counter" property type must be number       components.schemas.RoadCounter.example.counter

You can run the spectral lint by yourself if you have docker installed with
docker run --rm -it -v $(pwd):/tmp stoplight/spectral lint --ruleset "/tmp/.spectral.yaml" "/tmp/openapi.yaml"

[t-huyeng]

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?

[wirthual]

Yes, thats correct 👍

[mvglasow]

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]

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.