Closed heerup closed 2 years ago
Could you add an example to the sample app to show how you see having this interface put in providing value or maybe document it in the PR, whatever you think is easier for you :)
On Mon, 13 Jan 2020 at 07:11, heerup notifications@github.com wrote:
I changed to referring an interface IRouteMetadata instead of the abstract class (the abstract base class is still there) to allow for more composition scenarios. Then I Added the option to pass an instance of IRouteMetadata instead of using the type parameter.
You can view, comment on, or merge this pull request online at:
https://github.com/CarterCommunity/Carter/pull/234 Commit Summary
- Move to using a route metadata interface.
- Accept Routemetadata instances
File Changes
- M Carter.sln https://github.com/CarterCommunity/Carter/pull/234/files#diff-0 (34)
- M src/Carter/CarterExtensions.cs https://github.com/CarterCommunity/Carter/pull/234/files#diff-1 (2)
- M src/Carter/CarterModule.cs https://github.com/CarterCommunity/Carter/pull/234/files#diff-2 (142)
- M src/Carter/OpenApi/CarterOpenApi.cs https://github.com/CarterCommunity/Carter/pull/234/files#diff-3 (12)
- A src/Carter/OpenApi/IRouteMetaData.cs https://github.com/CarterCommunity/Carter/pull/234/files#diff-4 (51)
- M src/Carter/OpenApi/RouteMetaData.cs https://github.com/CarterCommunity/Carter/pull/234/files#diff-5 (2)
Patch Links:
- https://github.com/CarterCommunity/Carter/pull/234.patch
- https://github.com/CarterCommunity/Carter/pull/234.diff
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/CarterCommunity/Carter/pull/234?email_source=notifications&email_token=AAAZVJQKDF7BQKBBM7CPHPTQ5QHZ7A5CNFSM4KF5OJO2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IFVFVSQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAZVJVM42EWIULKNL7CNZ3Q5QHZ7ANCNFSM4KF5OJOQ .
I changed to referring an interface
IRouteMetadata
instead of the abstract class (the abstract base class is still there) to allow for more composition scenarios. Then I Added the option to pass an instance ofIRouteMetadata
instead of using the type parameter.