-
Notifications
You must be signed in to change notification settings - Fork 9k
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
Missing documentation on simple static distribution #4976
Comments
Hey @maral! Fair point 😄 Though I'm not a big fan of direct copying, since it locks users into specific versions... you can do it, so we should document it as an option. |
I think it's worth comparing the difference to ReDoc's documentation... https://github.com/Rebilly/ReDoc#tldr Their TDLR is just a single HTML page that points at a CDN'd version of redoc. It'd be great if Swagger UI did something similar. Either just being able to copy the /dist files, or being able to immediately point at CDN hosted versions is really helpful for allowing developers to just jump in, vs. "Here's an NPM package" |
@tomchristie, we do document this 😄 https://github.com/swagger-api/swagger-ui/blob/master/docs/usage/installation.md#unpkg It is a bit tucked away - I'd be happy to accept a PR that adds a TL;DR to our README! |
fixed in #5095! |
In the install section, there is no mention of the possibility to just copy the /dist directory to my project. It was in documentation in the older version, so I knew how to do it, however for others it might be confusing, as from the webpage the natural direction is to the Installation page, which is quite difficult to grasp for non-js programmers.
The text was updated successfully, but these errors were encountered: