You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Inconsistency or contradiction in the specification
Fault in the representation (rendering) of the specification
Other (please describe below)
Describe the bug
In most of the spec when we need geolocation we create an object called geoReference. Only in routes we call this object geoReferences. Is that just a typo?
The geographical representation of a route may be a list of coordinates, but that is still a singular geographical reference. Even the spec says it is one of a number of possible geographical representations.
Hello @chrissvo, thanks for your contribution. You are correct that this is inconsistent and should not be done this way. The problem is that parties might lean on the 'typo' and therefore we cannot just change it. However what we can do is introduce a new field geoReference and deprecate geoReferences. It does mean that both fields will be part of the specification, but OpenAPI can clearly indicate which one is deprecated. Would that work for you?
Type of bug
Describe the bug
In most of the spec when we need geolocation we create an object called
geoReference
. Only in routes we call this objectgeoReferences
. Is that just a typo?The geographical representation of a route may be a list of coordinates, but that is still a singular geographical reference. Even the spec says it is one of a number of possible geographical representations.
To Reproduce
https://github.com/opentripmodel/otm5-change-requests/blob/09b28ebfd1473a8a1c21a94ba2c9b96ca2b3d294/otm5_openapi_specification.json#L2595
Expected behavior
I would expect that a route has a property called
geoReference
(like locations and events).Screenshots
Additional context
I build a few OTM API's and realised that in one case I implemented this response for a geographical area:
And this response in another to describe a location:
"geoReference": {}
And this response in yet another to describe a route:
"geoReferences": {}
I think the first one is just wrong, but the last one conforms to the spec, yet feels wrong ;)
cc @robbertjanssen85
The text was updated successfully, but these errors were encountered: