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

Need to Update the Docs or change the API #120

Closed
Fosol opened this issue Apr 6, 2023 · 1 comment
Closed

Need to Update the Docs or change the API #120

Fosol opened this issue Apr 6, 2023 · 1 comment

Comments

@Fosol
Copy link

Fosol commented Apr 6, 2023

Describe the bug

CHES has a bug in the API or Docs. When you return a 422 on the EmailMerge endpoint. The response status is a number, not a string. This will result in errors during deserialization.

To Reproduce

Steps to reproduce the behavior:

  1. Submit an invalid request to generate the error response model

Expected behavior

The documentation states that the status property is a string, but it is returned as a number.

You should just update your documentation as changing it to a string will most likely break any solutions expecting a number.

@jujaga
Copy link
Member

jujaga commented May 18, 2023

Resolved in #121

@jujaga jujaga closed this as completed May 18, 2023
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

No branches or pull requests

2 participants