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

SP(0.5): Add 409 response in Swagger docs for user patch #1027

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

sandrvvu
Copy link
Contributor

@sandrvvu sandrvvu commented Dec 4, 2024

Status code 409 Conflict wasn't documented in the Swagger documentation for endpoint {server}/users/{id} (PATCH method):

image

After fix:
image
image

@sandrvvu sandrvvu added bug Something isn't working Backend labels Dec 4, 2024
@sandrvvu sandrvvu self-assigned this Dec 4, 2024
@sandrvvu sandrvvu closed this Dec 4, 2024
@sandrvvu sandrvvu deleted the bugfix/996/add-swagger-patch-user-409-response branch December 4, 2024 13:57
@sandrvvu sandrvvu restored the bugfix/996/add-swagger-patch-user-409-response branch December 4, 2024 14:04
@sandrvvu sandrvvu reopened this Dec 4, 2024
Copy link

sonarcloud bot commented Dec 4, 2024

@sandrvvu sandrvvu merged commit bc960bc into develop Dec 4, 2024
6 checks passed
@sandrvvu sandrvvu deleted the bugfix/996/add-swagger-patch-user-409-response branch December 4, 2024 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Swagger] The endpoint PATCH /users/{id} doesn't contain 409 response status.
4 participants