Skip to content

v3.2: (port of #4614) Clarify that Request Body Objects need a body #4726

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

Merged
merged 1 commit into from
Jun 15, 2025

Conversation

handrews
Copy link
Member

@handrews handrews commented Jun 15, 2025

Port of PR #4614, which AFAICT is the only 3.1 PR that still needed to be ported to 3.2.

We require content but failed to require it to be non-empty, even though a request body without a body does not make any sense.

  • schema changes are included in this pull request
  • schema changes are needed for this pull request but not done yet
  • no schema changes are needed for this pull request

We require `content` but failed to require it to be non-empty,
even though a request body without a body does not make any sense.
@handrews handrews added this to the v3.2.0 milestone Jun 15, 2025
@handrews handrews requested a review from a team as a code owner June 15, 2025 01:04
@handrews handrews added the approved pr port PRs that just port an approved PR to another version label Jun 15, 2025
@handrews handrews requested a review from a team as a code owner June 15, 2025 01:04
@ralfhandl ralfhandl changed the title v3.2: (port of #4601) Clarify that Request Body Objects need a body v3.2: (port of #4614) Clarify that Request Body Objects need a body Jun 15, 2025
@ralfhandl ralfhandl requested a review from a team June 15, 2025 10:54
Copy link
Contributor

@mikekistler mikekistler left a comment

Choose a reason for hiding this comment

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

Looks good. 👍

@handrews handrews merged commit 4972cc0 into OAI:v3.2-dev Jun 15, 2025
2 checks passed
@handrews handrews deleted the req-content-320 branch June 19, 2025 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved pr port PRs that just port an approved PR to another version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants