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

feat: Add custom decorator support #837

Merged
merged 6 commits into from
Nov 22, 2024

Conversation

R-Campbell
Copy link
Contributor

This adds the ability to customize the Fastify decorator while defaulting to the existing swagger. This enables generating multiple swagger docs with different transform functions, allowing more flexibility for e.g. creating multiple docs based on custom route config options. I believe this solves #631 if the set of user scopes is known ahead of time.

Checklist

R-Campbell and others added 2 commits November 7, 2024 21:02
fix inverted transform examples

Signed-off-by: Robert Campbell <[email protected]>
@R-Campbell
Copy link
Contributor Author

I'd love to get this in to be able to use it, anything that would help push it through?

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

@Fdawgs Fdawgs requested a review from Copilot November 20, 2024 10:58

Choose a reason for hiding this comment

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

Copilot reviewed 4 out of 4 changed files in this pull request and generated no suggestions.

@R-Campbell
Copy link
Contributor Author

@Fdawgs Copilot said this was the perfect PR, anything else needed to get it merged?

@Fdawgs
Copy link
Member

Fdawgs commented Nov 21, 2024

@Fdawgs Copilot said this was the perfect PR, anything else needed to get it merged?

I'll give it a proper read through tomorrow. 😊

Edit: stuff it, read it today.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Signed-off-by: Frazer Smith <[email protected]>
Signed-off-by: Frazer Smith <[email protected]>
Signed-off-by: Frazer Smith <[email protected]>
@Fdawgs Fdawgs merged commit 0b9c8d3 into fastify:master Nov 22, 2024
11 checks passed
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.

4 participants