-
Notifications
You must be signed in to change notification settings - Fork 10.7k
Add support for setting ApiDescriptor.GroupName for minimal APIs #34443
Copy link
Copy link
Closed
Labels
Priority:1Work that is critical for the release, but we could probably ship withoutWork that is critical for the release, but we could probably ship withoutapi-suggestionEarly API idea and discussion, it is NOT ready for implementationEarly API idea and discussion, it is NOT ready for implementationarea-minimalIncludes minimal APIs, endpoint filters, parameter binding, request delegate generator etcIncludes minimal APIs, endpoint filters, parameter binding, request delegate generator etcfeature-minimal-actionsController-like actions for endpoint routingController-like actions for endpoint routing
Milestone
Metadata
Metadata
Assignees
Labels
Priority:1Work that is critical for the release, but we could probably ship withoutWork that is critical for the release, but we could probably ship withoutapi-suggestionEarly API idea and discussion, it is NOT ready for implementationEarly API idea and discussion, it is NOT ready for implementationarea-minimalIncludes minimal APIs, endpoint filters, parameter binding, request delegate generator etcIncludes minimal APIs, endpoint filters, parameter binding, request delegate generator etcfeature-minimal-actionsController-like actions for endpoint routingController-like actions for endpoint routing
Type
Fields
Give feedbackNo fields configured for issues without a type.
Minimal APIs currently don't populate any value for the
ApiDescriptor.GroupNameproperty. We should add support for setting the group name via endpoint metadata and flowing that through to theApiDescriptor, e.g.:Setting it by default
Swashbuckle currently uses
ApiDescriptor.GroupNamein its default API inclusion predicate in such a way that if we were to set a default value that doesn't match the configured Swagger document name, the APIs would not be included by default. For that reason it seems like we shouldn't set a default value forApiDescriptor.GroupNameunless we also update the Swashbuckle logic.@bradygaster