Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix OpenAPI for query parameters #371

Merged
merged 9 commits into from
Aug 1, 2024
Merged

Fix OpenAPI for query parameters #371

merged 9 commits into from
Aug 1, 2024

Conversation

tarsil
Copy link
Collaborator

@tarsil tarsil commented Aug 1, 2024

Checklist

  • The code has 100% test coverage.
  • The documentation was properly created or updated (if applicable) following the correct guidelines and appropriate language.
  • I branched out from the latest main or is a sub-branch.

This addresses a regression introduced when we moved to Encoders where the query params when declared should be displayed in the OpenAPI documentation.

Adds missing documents (somehow this was missed....) and extra tests for various features.

@tarsil tarsil requested review from PeterChain and DJWOMS August 1, 2024 10:54
@tarsil tarsil merged commit a15908d into main Aug 1, 2024
11 checks passed
@tarsil tarsil deleted the fix/qp branch August 1, 2024 17:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants