-
Notifications
You must be signed in to change notification settings - Fork 83
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
Link to code.jquery.com #217
Comments
I'm sorry for that. The next move I will do or accept about JQuery is just dropping it. |
Would you accept to inline jquery in the 2 html files as an easy workaround? Our staging environment has to have a specific CSP policy just because of this |
Instead of inlining it I prefer using a local snapshot of jquery (.min.js) in order to keep the integrity check ( |
I guess <script src="https://code.jquery.com/jquery-3.2.1.min.js"
integrity="sha256-hwg4gsxgFZhOsEEamdOYGBf13FyQuiTwlAQgxVSNgt4=" crossorigin="anonymous"></script> would be replaced by <script src="js/jquery-3.2.1.min.js"
integrity="sha256-hwg4gsxgFZhOsEEamdOYGBf13FyQuiTwlAQgxVSNgt4="></script> |
Ok, I did a PR to move the external JS to assets: #248 |
Move jQuery and jscolor to assets (#217)
Web UI don't work without internet connection (if browser cache is clear)
The text was updated successfully, but these errors were encountered: