Skip to content
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

Site URLs should be configurable #62

Closed
msiebuhr opened this issue May 30, 2011 · 4 comments
Closed

Site URLs should be configurable #62

msiebuhr opened this issue May 30, 2011 · 4 comments

Comments

@msiebuhr
Copy link

I'm trying to set up a RTFD-site internally in my company to host our documentation, but I'm struggling a bit with readthedocs.org (36) and, to a lesser extent, rtfd.org (3) being hard-coded all over the place.

IMHO, this should be configurable, somehow.

(I tried hacking it by adding a variable in readthedocs/settings/base.py, and accessing that in the templates, expecting it to "just work" like SITE_ROOT and friends, but to no avail. Any pointers to where I'm going wrong?)

@msiebuhr
Copy link
Author

This has apparently been fixed by e7f9991, but I haven't had the time to verify this yet.

@dcolish
Copy link
Contributor

dcolish commented Jun 18, 2011

I've begun to build urls using the {% url %} tag. That will ensure that the mentioned site settings are preserved.

@ericholscher
Copy link
Member

Yea. Things originally used the url tags, but it caused some oddities with 404 pages on subdomains and CNAME'd pages. Basically, you would get .readthedocs.org/search or /login, which is another broken link, so it made it really hard to get back to the main site without manipulating the URLs.

I haven't thought of a great way to handle this, but keeping it working for local dev is probably more important ATM than having 404 pages work correctly, especially now that a lot of the 404s will come from nginx directly.

@ericholscher
Copy link
Member

Closing this since it's done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants