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

Swagger: Fix duplicate path parameters #658

Merged
merged 5 commits into from
Dec 11, 2024
Merged

Swagger: Fix duplicate path parameters #658

merged 5 commits into from
Dec 11, 2024

Conversation

mejroslav
Copy link
Contributor

@mejroslav mejroslav commented Dec 10, 2024

  • Path parameters taken from path are not added if they are described in route docstring.
  • The first line of docstring is not duplicated in name and description.
  • asab and bspump endpoints are at the bottom.
  • Enhanced description for asab endpoints.

Example of path parameter taken from docstring:
image

Example of path parameter taken only from path :
image

Example of sorted tags in bspump application:
image

Examples of asab endpoints:
image

image

@mejroslav mejroslav self-assigned this Dec 10, 2024
@mejroslav mejroslav requested a review from byewokko December 10, 2024 14:52
@mejroslav mejroslav added the bug Something isn't working label Dec 10, 2024
Copy link
Collaborator

@byewokko byewokko left a comment

Choose a reason for hiding this comment

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

looks bžutiful!

@mejroslav mejroslav merged commit 91efe34 into master Dec 11, 2024
10 checks passed
@mejroslav mejroslav deleted the fix/swagger-2412 branch December 11, 2024 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants