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

including optional "message" field for "fail" status #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

arjunQ21
Copy link

@arjunQ21 arjunQ21 commented Apr 7, 2021

We usually send "fail" status if validation errors exist with request data.
Sometimes, the failed message may not be included with any key of request payload(for example: when the input data is included in request parameters, instead of request body).
In such situations, adding an optional "message" field with response( just like in error condition) would make this standard even more flexible.

@istrupin
Copy link

I'd like to see this change as well -- @xzilla what are your thoughts on this? It was also brought up in #9

@feyton
Copy link

feyton commented Jan 14, 2022

That would be the best implementation. Actually, maybe consider adding a message as a standard. Just sending error or data with no description...

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.

3 participants