feat: Auto-collect params and response schemas from handlers #1510
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Collect component schemas for `IntoResponses` enums * `utoipa::path` macro flag to enable/disable automatic inclusion of `IntoParams` impls to request params (this is disabled by default) * Global config option to enable `IntoParams` impls inclusion (this is disabled by default) No breaking changes are excepted, all tests passed (at least locally).
Added missing `auto_into_params` method to config
Add missing schema collection for Result<T, E>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
IntoResponsesenums (this allows to avoid manually referring schemas in openapi derive, which causes inconsistency between automatic request body schema extraction and missing response schemas extraction in case of enum which implementsIntoResponsestrait).utoipa::pathmacro flag to enable/disable automatic inclusion ofIntoParamsimpls to request params (this is disabled by default)IntoParamsimpls inclusion (this is disabled by default)This changes shouldn't trigger #675 again
No breaking changes are excepted, all tests passed (at least locally).