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

Fixed #25519 -- Added support for SCRIPT_NAME for admin view site #5426

Closed

Conversation

DheerendraRathor
Copy link
Contributor

Used request.META['SCRIPT_NAME'] to determine if SCRIPT_NAME is set.
If it is indeed there and site_url is '/' then used this as site_url

@timgraham
Copy link
Member

Could you add a mention about this behavior in docs/ref/contrib/index.txt where AdminSite.site_url is mentioned, as well as in the 1.10 release notes?

@DheerendraRathor
Copy link
Contributor Author

Should I close this PR and create new with docs or a new PR just for doc will be fine?

@timgraham
Copy link
Member

You can amend this one.

Used request.META['SCRIPT_NAME'] to determine if SCRIPT_NAME is set.
If it is indeed there and site_url is '/' then used this as site_url

Added documentation in docs/ref/contrib/admin/index.rst
Modified 1.10 release notes to reflect the same
@DheerendraRathor
Copy link
Contributor Author

I've updated the PR with docs and also added the same in 1.10 release notes

@timgraham
Copy link
Member

merged in 59e85f0, thanks!

@timgraham timgraham closed this Oct 22, 2015
@DheerendraRathor DheerendraRathor deleted the ticket_25519 branch October 29, 2015 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants