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

Validate payload in UI? #2119

Closed
atdiff opened this issue Apr 26, 2016 · 4 comments
Closed

Validate payload in UI? #2119

atdiff opened this issue Apr 26, 2016 · 4 comments

Comments

@atdiff
Copy link

atdiff commented Apr 26, 2016

It would be good if the JSON payload was being validated within the UI against the Swagger schema. Is this in place anywhere or are there any plans for it?

@webron
Copy link
Contributor

webron commented May 5, 2016

Looks like the wrong project. If relevant, please open on swagger-editor. Can't say I see validation as part of the tool though.

@webron webron closed this as completed May 5, 2016
@atdiff
Copy link
Author

atdiff commented May 5, 2016

@webron I see this as being a part of Swagger UI actually. (I can edit my comment above). For instance, on http://petstore.swagger.io/#!/pet/addPet, I think having the payload validated when clicking "Try it out" makes sense. Thoughts?
I can open a new iissue if this one can't be reopened.

@webron
Copy link
Contributor

webron commented May 5, 2016

I can reopen the issue, but it still doesn't make sense to me, no. swagger-ui is not intended as a full-fledged testing tool and at the moment it falls out of scope. We may change that in the future.

@atdiff
Copy link
Author

atdiff commented May 20, 2016

@webron I don't think it needs to be a full-fledged testing tool. Validation of the payload seems like a simple thing to incorporate. If a person is going to click "Try it out", it seems logical to know whether the request and payload were valid or not as a part of that information. Seems useful.

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

No branches or pull requests

3 participants