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

OpenAPI Auto add Bad Request response for operation input #45965

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

phillip-kruger
Copy link
Member

Fix #27376

Copy link

github-actions bot commented Jan 30, 2025

🎊 PR Preview b7ba3e7 has been successfully built and deployed to https://quarkus-pr-main-45965-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

This comment has been minimized.

This comment has been minimized.

Copy link

@punkratz312 punkratz312 left a comment

Choose a reason for hiding this comment

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

nicely done, straight forward coding

@phillip-kruger phillip-kruger force-pushed the openapi-auto-400 branch 2 times, most recently from 9441e8d to dd3cb1f Compare January 30, 2025 22:11

This comment has been minimized.

This comment has been minimized.

@phillip-kruger phillip-kruger changed the title OpenAPI Auto add 400 response for operation input OpenAPI Auto add Bad Request response for operation input Jan 30, 2025

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@phillip-kruger phillip-kruger force-pushed the openapi-auto-400 branch 2 times, most recently from feafd6f to 89b932b Compare February 6, 2025 03:00

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

@MikeEdgar MikeEdgar left a comment

Choose a reason for hiding this comment

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

LGTM

This comment has been minimized.

This comment has been minimized.

Copy link

quarkus-bot bot commented Feb 10, 2025

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit cd15adb.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Copy link

quarkus-bot bot commented Feb 10, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit cd15adb.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@phillip-kruger phillip-kruger merged commit f3d7851 into quarkusio:main Feb 11, 2025
30 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.19 - main milestone Feb 11, 2025
@quarkus-bot quarkus-bot bot added the kind/enhancement New feature or request label Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OpenAPI should list a 400 error code on certain conditions
4 participants