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
DOC-1008: Configuration tool #2176
Conversation
<div id="configurator"> | ||
</div> | ||
<script src="https://cdn.tiny.cloud/1/qagffr3pkuv17a8on1afax661irst1hbr4e6tbv888sz91jc/tinymce/5/tinymce.min.js" referrerpolicy="origin"></script> | ||
<script src="../js/configurator/configurator.js" referrerpolicy="origin"></script> |
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.
Why is this getting embedded on the docs site? That is a really bad idea and this should be hosted by actual infrastructure similar to the skin tool.
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.
Marking as needs changes as this tool shouldn't be deployed on the docs, it should be handled by the website team or at least deployed on a separate domain/path as I mentioned in a previous comment.
Edit: To clarify, I don't mind embedding it via an iframe, but the tool itself cannot be under the /docs/
path.
7b55d43
to
76902d8
Compare
Related Ticket: DOC-1008
Description of Changes:
Add Skin tool pagePre-checks:
feature/
orhotfix/
_data/nav.yml
has been updated (if applicable)Review: