OpenApi spec is broken

[yehonatan1202]

Semantic error at paths./comments/{id}
Declared path parameter "id" needs to be defined as a path parameter at either the path or operation level

missing

parameters:
        - $ref: '#/components/parameters/UUId'

in /comments/{id}

[linear]

EDU-293 OpenApi spec is broken

[kheiner]

I don't follow what you're saying here. The only place I can find /comments/{id} is here:

I checked the folders in comments and components manually to see if I could find any reference that matched what you designated as missing:

parameters:
        - $ref: '#/components/parameters/UUId'

Care to help clarify or provide a permalink to the file?

[yehonatan1202]

Sorry, let me try to clarify it better. I think I opened this issue when trying to generate a client for Directus using Orval and was getting an error related to the swagger (it was a year ago so I might be wrong...). anyway, when copying the directus open Api spec into https://editor.swagger.io/ i got the error that i mentioned.
Here is how to reproduce it:

1. copy the open api spec from https://sandbox.directus.io/server/specs/oas
2. paste it into https://editor.swagger.io/
3. between all the warnings you will see the following error

How to manually fix:

Add "$ref": "#/components/parameters/UUId" to GET, PATCH and DELETE, for GET and PATCH just append it to the parameters list and for DELETE you will also need to create a parameters list.

I hope this is more clear, Let me know if you have any other questions.

Thanks a lot :)

[kheiner]

I think this issue actually belongs with the directus/directus monorepo, as it relates to the actual API endpoint, not the documentation of the default API endpoints.

I found a spam PR that actually looks like it will partially address your concern, I'll mention this comment in there so the link will appear below. I leave it up to you on whether or not you want to create a new issue in the other repository or track the closed PR manually. I personally would make a new issue if I was in your boat and mention the closed PR in it; but it all depends no what you want to do.