-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
docs: add API page to docs using Swagger UI #11154
Conversation
Codecov Report
@@ Coverage Diff @@
## master #11154 +/- ##
==========================================
- Coverage 65.84% 61.22% -4.63%
==========================================
Files 827 827
Lines 39051 39037 -14
Branches 3673 3673
==========================================
- Hits 25715 23901 -1814
- Misses 13229 14955 +1726
- Partials 107 181 +74
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
fbc538b
to
6942127
Compare
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.
Minor non-blocking nit, apart from that LOVE IT! 🚀
<strong>NOTE! </strong> | ||
You can find an interactive version of this documentation | ||
on your local Superset instance | ||
at <strong>/swagger/v1</strong> (if enabled) |
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.
nit: FAB_API_SWAGGER_UI
is enabled by default:
at <strong>/swagger/v1</strong> (if enabled) | |
at <strong>/swagger/v1</strong> |
673366d
to
be8eace
Compare
@mistercrunch what is the process for generating |
* docs: add API page to docs using Swagger UI * silence fossa
SUMMARY