-
Notifications
You must be signed in to change notification settings - Fork 126
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
Embed swagger UI #314
Embed swagger UI #314
Conversation
Just tested, works fine! |
@caco3 Looks good so far!
We had this before and #293 fixed the issue. |
Good point, I changed it to make sure the mtime does not change. BTW, this PR adds 3544 bytes to the flash usage. If this is a problem, we could create a define in |
Tested against Lolin D32 (without PSRAM) & additional flash usage is OK |
This PR embeds the Swagger UI into the Web Interface. To keep the additionally required flash usage low, most files get pulled from a CDN.
I also renamed the last tab
Forum
toHelp
and extended the tab with a section for the REST API documentation.The great advantage for an embedded Swagger UI is to be able to directly send the cmds to the device.
Updated UI
Swagger UI
Direct calls to the device