-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
chore(api): clean up API spec #24838
Conversation
Codecov Report
@@ Coverage Diff @@
## master #24838 +/- ##
==========================================
+ Coverage 68.98% 69.00% +0.01%
==========================================
Files 1903 1903
Lines 74007 74085 +78
Branches 8193 8193
==========================================
+ Hits 51052 51120 +68
- Misses 20834 20844 +10
Partials 2121 2121
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
Thanks @sebastianliebscher for exposing the summary and cleaning up/standardizing the API documentation.
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.
This is an amazing piece of cleanup, thanks so much @sebastianliebscher! ❤️
SUMMARY
This PR makes use of the OpenAPI specs
summary
to display relevant information in the API doc overview. This way, you don't have to expand every API endpoint to see what it does, improving usability by a lot.I mostly replaced
description
withsummary
where the text was small enough. In some places with more explanatory text, description has been split intodescription
andsummary
.If this PR is too big, I'm happy to split it up.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
After:
Before:
TESTING INSTRUCTIONS
/swagger/v1
and have a lookADDITIONAL INFORMATION