-
Notifications
You must be signed in to change notification settings - Fork 16
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
Make it easy to test locally #83
Conversation
Looks like sphinx 3.5 broke the extension :) taking a look |
Looks like they updated underscore.js, I'll check the incompatible changes, also see if we can drop that dep. |
Additionally, disabling the CORS check should be done. I'm using https://addons.mozilla.org/en-US/firefox/addon/cors-everywhere/ for that.
Additionally, disabling the CORS check should be done. I'm using https://addons.mozilla.org/en-US/firefox/addon/cors-everywhere/ for that.
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.
Looks like a good change 👍
@@ -697,7 +697,7 @@ window.addEventListener("DOMContentLoaded", () => { | |||
const project = READTHEDOCS_DATA.project; | |||
const version = READTHEDOCS_DATA.version; | |||
const language = READTHEDOCS_DATA.language || "en"; | |||
const api_host = '/_'; | |||
const api_host = READTHEDOCS_DATA.proxied_api_host || '/_'; |
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.
I'd love to standardize this a bit more in some kind of JS library, at some point in the future.
|
||
cd docs | ||
pip install sphinx-autobuild | ||
pip install -r requirement.txt |
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.
Isn't it the name of this file requirements.txt (plural)?
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.
Good catch @nienn!
Additionally, disabling the CORS check should be done.
I'm using https://addons.mozilla.org/en-US/firefox/addon/cors-everywhere/ for that.