Skip to content

Conversation

@captainsafia
Copy link
Member

Manual backport of #36439 to release/6.0

* Expose ParameterInfo for Minimal Actions API Explorer

Populate the ParameterDescriptor for Minimal Actions parameters with
a type that implements IParameterInfoParameterDescriptor so that any
custom attributes on the parameter can be inspected.
Relates to #36438.

* Remove file-scoped namespace

Addresses review feedback.
@captainsafia captainsafia requested a review from a team September 15, 2021 16:18
@ghost ghost added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Sep 15, 2021
@captainsafia captainsafia added old-area-web-frameworks-do-not-use *DEPRECATED* This label is deprecated in favor of the area-mvc and area-minimal labels and removed area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates labels Sep 15, 2021
@captainsafia captainsafia changed the title Expose ParameterInfo for Minimal Actions API Explorer (#36439) [release/6.0] Expose ParameterInfo for Minimal Actions API Explorer (#36439) Sep 15, 2021
@captainsafia
Copy link
Member Author

@dotnet/aspnet-build Can I get help setting this up with an automerge? For some reason my builds keep getting cancelled. This is a backport with no changes from the other PR.

@wtgodbe wtgodbe enabled auto-merge (squash) September 15, 2021 22:54
@wtgodbe
Copy link
Member

wtgodbe commented Sep 15, 2021

Force merging this since it was green & approved in main, and branches snap in an hour

@wtgodbe wtgodbe disabled auto-merge September 15, 2021 23:03
@wtgodbe wtgodbe merged commit 69f1ace into release/6.0 Sep 15, 2021
@wtgodbe wtgodbe deleted the safia/bp-param-desc branch September 15, 2021 23:03
@ghost ghost added this to the 6.0-rc2 milestone Sep 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

old-area-web-frameworks-do-not-use *DEPRECATED* This label is deprecated in favor of the area-mvc and area-minimal labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants