Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Controllers Option #1360

Merged
merged 4 commits into from
Nov 12, 2021
Merged

Add Controllers Option #1360

merged 4 commits into from
Nov 12, 2021

Conversation

RehanSaeed
Copy link
Member

No description provided.

@RehanSaeed RehanSaeed added template/API The Swagger API project template minor Pull requests requiring a minor version update according to semantic versioning. labels Nov 12, 2021
@RehanSaeed RehanSaeed force-pushed the add-controllers-option branch from a2b86da to 1c74fe7 Compare November 12, 2021 11:14
@RehanSaeed RehanSaeed force-pushed the add-controllers-option branch from 8b0ee38 to 829ba83 Compare November 12, 2021 14:21
@RehanSaeed RehanSaeed merged commit 231c648 into main Nov 12, 2021
@RehanSaeed RehanSaeed deleted the add-controllers-option branch November 12, 2021 14:49
@VictorioBerra
Copy link
Collaborator

If controllers are unchecked, I see that it says it will use minimal APIs.

If we use minimal APIs, is the resulting API exactly the same? IE all Cars endpoints, all Swagger metadata, attributes, comments, all mapping and commands?

Or does the option essentially just strip out all normal API functionality for a small minimal api sample?

@RehanSaeed
Copy link
Member Author

If we use minimal APIs, is the resulting API exactly the same? IE all Cars endpoints, all Swagger metadata, attributes, comments, all mapping and commands?

No. You get a completely empty app and the only endpoints are health checks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Pull requests requiring a minor version update according to semantic versioning. template/API The Swagger API project template
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants