Skip to content

Conversation

@badsyntax
Copy link

There's no "uid" type defined by the spec. "uid" should be treated as type "string" with format "uid".

What does it do?

Fixes a bug where the documentation plugin generates an invalid spec when a UID field type is used.

Why is it needed?

OpenAPI code generators will error out as they cannot handle an invalid data type of "uid".

How to test it?

Add a "UID" field and re-generate the API documentation, then use https://editor.swagger.io/ to validate the spec.

Related issue(s)/PR(s)

I couldn't find any related github issues but found a bug report in the forums: https://forum.strapi.io/t/errors-on-generated-documentation-swagger-openapi/2802

There's no "uid" type defined by the spec. "uid" should be treated as type "string" with format "uid".
@derrickmehaffy
Copy link
Member

This pull request has been mentioned on Strapi Community Forum. There might be relevant details there:

https://forum.strapi.io/t/errors-on-generated-documentation-swagger-openapi/2802/2

@derrickmehaffy
Copy link
Member

This pull request has been mentioned on Strapi Community Forum. There might be relevant details there:

https://forum.strapi.io/t/are-strapi-maintainers-accepting-pull-requests/9157/1

@derrickmehaffy
Copy link
Member

Hi 👋
We now have released the v4. As of now, the v3 is in maintenance mode for 6 months. We will only be fixing critical and security issues on v3 from now on.
I will go ahead and close this PR. Thank you for contributing to Strapi 🔥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants