Skip to content

Add summary property in ApiResponse #2827

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

Closed
wants to merge 1 commit into from
Closed

Add summary property in ApiResponse #2827

wants to merge 1 commit into from

Conversation

ymohdriz
Copy link
Contributor

@ymohdriz ymohdriz commented Jun 1, 2018

Hi team,

I have added the summary property in ApiResponse class.
Detailed issue description: https://github.com/OAI/OpenAPI-Specification/issues/1591

I request you to do the code review and inform me if any changes are required.

Thanks,
Mohammed Rizwan

@frantuma
Copy link
Member

frantuma commented Jun 1, 2018

Link to issue above is broken, good one should be this one. However that points to an open discussion about spec enhancements which is not part of any released spec, it's therefore too early to address it in implementations, and I will close this PR for the time being

@frantuma frantuma closed this Jun 1, 2018
@ymohdriz
Copy link
Contributor Author

ymohdriz commented Jun 4, 2018

Hi @frantuma,

Thanks for the clarification.

Thanks,
Mohammed

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.

2 participants