Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support ref relative pointers in openapi params #554

Merged
merged 2 commits into from
Mar 2, 2022

Conversation

elierotenberg
Copy link
Contributor

  • Add test for failing use case
  • Implement fix

Fix #553.

Checklist

* Add test for failing use case
* Implement fix

Fix fastify#553.
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

elierotenberg added a commit to elierotenberg/fastify-zod that referenced this pull request Mar 1, 2022
* Major rewrite
* Better support for referential transparency, notably for enums
* Deduplicate redundant type annotation when defining requests
* Better support for downstream openapitools-generators with automatic
  ref extraction
* Remove fastify-zod-examples in favor of fastify-zod tests as support
  for fastify-swagger is now first-class

Note:
  Pending merge of fastify/fastify-swagger#554,
  fastify-swagger in package.json references the forked commit.
@mcollina mcollina merged commit 112ecd4 into fastify:master Mar 2, 2022
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.

$ref relative references (Schema#/properties/...) in common params crashing in openapi mode
3 participants