-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 plugin apidocs details #2371
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2371 +/- ##
=======================================
Coverage 72.74% 72.74%
=======================================
Files 110 110
Lines 7948 7948
=======================================
Hits 5782 5782
Misses 2166 2166
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! This is a bit redundant, but just for consistency with the other endpoints, can you add an aiohttp_apispec.response
description
for these? It could just be something like "Returns a list of all Plugin objects dumped in PluginSchema format."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Makes sense, added a description field there! |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Description
Adds more information to the swagger documentation for plugin endpoints.
Type of change
How Has This Been Tested?
Swagger documentation is updated with new information.
Checklist: