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

Now will distinguish between status codes the expected inferred type #1957

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

carloscbl
Copy link

@carloscbl carloscbl commented Aug 1, 2024

Fixes #1956 and #1850 .

Changes proposed in this pull request:

  • Now instead merging 2 different responses as available options, will check for the status code the available options, after that if you still have multiple content types it will raise the error, and users should specify in headers the correct one

@RobbeSneyders
Copy link
Member

Thanks @carloscbl!

Tests are failing though. Not sure if this is due to an issue with your code, or because the failing tests depend on the old behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failing to infer content when application/problem+json is present in another response code
2 participants