Safia Abdalla
Safia Abdalla
@sander1095 Thanks for getting this issue together! The one gap I notice is that this proposal primarily targets the `ProducesResponseType` attributes associated with MVC and not the `Produces` metadata/attributes/extension methods...
@sander1095 The above API was approved in case you'd like to update your draft PR. Note: we're leaving the question on how to handle the `Produces` extension method for minimal...
> Which one is it? Please take a look at my work so you can give specific feedback :) That part of the API proposal was rejected because it would've...
> One thing we still need to discuss is the case when someone decorates their action with XML Comments with a response description and the ProducesResponseType attribute with a Description....
@gurustron Thanks for reporting this issue! Minimal API's binding logic processes all arguments that are annotated with `[FromHeader]` as string values (see [here](https://github.com/dotnet/aspnetcore/blob/207bc7d00c00bf454dc83789d6824609b1d8b02a/src/Http/Http.Extensions/src/RequestDelegateFactory.cs#L719)). On the other hand, MVC's `HeaderModelBinder` has...
@martincostello TY for the PR. We reference the Mvc.Testing package in other tests in the repo (Identity.FunctionalTests, MvcFunctionalTest, for example). We can probably verify whether or not this fix applies...
Mvc.TemplateTests are skipped and OpenAPI test fix is merged. Hopefully this goes through smoothly 🤞🏽 Edit: to that end I'll optimistically set auto-merge on this.
Just came across this issue and thought I would chime in with my two-cents here. With regard to supporting document generation via `Microsoft.AspNetCore.OpenApi`, you're already part of the way there....
> Work has started on this here #5418 This PR has been merged and nightly builds of the new `Aspire.Hosting.Azure.Functions` package are now available. I made a sample repo with...