search

chairemobilite / trRouting

Transit Routing server app using Connection Scan Algorithm and flexible parameters
GNU General Public License v3.0
24 stars 10 forks source link

APIv2: Make sure tripStep is well defined in the docs #294

Open tahini opened 4 months ago

tahini commented 4 months ago

See screenshot. I would expect to have the definition of the tripSteps here, not just an empty array. There's probably something missing in our doc to make the link.

Capture d’écran du 2024-05-13 13-43-52

greenscientist commented 4 months ago

Running redocly cli (via https://redocly.com/docs/cli/installation/) lint gives nothing related to that (many other error but nothing about steps

tahini commented 4 months ago

Would it then be too deep in the hierarchy? Or too complex to describe? But indeed the example output gives something plausible, so not sure why it's like this...

greenscientist commented 4 months ago

Current guess: something about the discriminator might not work at that level or is missing information

greenscientist commented 4 months ago

Ou bien qqch avec le path de la référence. C'est juste weird que les tools ne donne pas d'erreur.

greenscientist commented 4 months ago

swagger-ui seems fine with it. https://petstore.swagger.io/?_gl=1*mz8a3z*_gcl_au*MzM0OTc0MTguMTcxMjg2ODEwMg..&_ga=2.60220772.872043090.1715623353-1558485238.1712868102#/default/get_v2_route

greenscientist commented 4 months ago

Tested with this tool: https://authress-engineering.github.io/openapi-explorer/?#?route=get-/v2/route and it's there. So something is making redoc unhappy