Add Description to ProducesResponseType (and others) for better OpenAPI document creation by sander1095 · Pull Request #58193 · dotnet/aspnetcore (original) (raw)
Hey @captainsafia, ASP.NET Core team and community!
I'm extremely excited to be able to create this PR! I'm dedicating this to the .NET and OpenAPI community, and I shouldn't forget to mention hacktoberfest as well! I'd love to get this merged or marked as accepted in the month of October, so keep the PR feedback coming!
I've been working on this for a while since talking to Safia about it at the MVP Summit 2024, and I'm very grateful for the opportunity.
Now, let's talk about some details 😁.
The state of the PR
I believe the PR is coming along nicely, but I find it difficult to say if it is done. I would need help from the ASP.NET Core team or community to get the following answered:
- How can I test these changes? I have added unit tests, but because I do not know how I can test this functionally in an ASP.NET Core project, I do not know what I still might need to do.
- I couldn't find any docs about this, so perhaps it'd be useful to add some documentation about this in the future?
- How can
Swashbuckle.AspNetCore,NSwagandMicrosoft.AspNetCore.OpenAPIbenefit from this change? Are there code changes needed on their side, if so, how do we coordinate this? Library authors, let me know if I can help! - I've already mentioned this in the PR description, but I believe we also need to get feature parity on the Minimal API side (Allow OpenAPI's response descriptions to be set using Minimal API #57963). However, this still needs some discussion as implementing it could apparently lead to source incompatibility issues. I'd love assistance with knowing how we will tackle this; after that I'd love a crack at implementing it!
And finally:
Please let me know what other changes need to be made to make this work functionally! Again, my goal is to get this to an accepted (or even merged!) state in october, if possible!