Skip to content

Simplified method to append django-debug-toolbar urlpatterns #441

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

Closed
wants to merge 4 commits into from

Conversation

nuklea
Copy link
Contributor

@nuklea nuklea commented Nov 6, 2013

Reason for creating this pull request is need to inspect object returned from get_urlconf(). I cannot inspect and get module __file__ when object created dynamically by imp.new_module.

@dkrnl
Copy link

dkrnl commented Nov 7, 2013

good request

@aaugustin
Copy link
Contributor

Your pull request disable two tests that — I assume — were introduced for a reason.

Can you check why these tests were introduced and why you think they aren't relevant any more?

@nuklea
Copy link
Contributor Author

nuklea commented Nov 9, 2013

@aaugustin tests reverted and fixed.

@aaugustin
Copy link
Contributor

I made even more drastic choices in #456. The toolbar no longer uses imp.new_module, and therefore this issue is fixed.

@aaugustin aaugustin closed this Nov 11, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants