-
Notifications
You must be signed in to change notification settings - Fork 1.1k
ImportError: cannot import name 'get_show_toolbar' #938
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
Comments
Hmm. Looks very much like a circular import error. Does it work if you move the |
I can confirm we got this error at my company when we tried to use the development version of djdt on one of our dev servers, which uses uwsgi. (It was this error or a very similar one that also looked like a circular import error trying to import middleware from a view that was indirectly imported from middleware.) The error did not occur for us using manage.py runserver, but it did through uwsgi. @timb07 We got around this temporarily by using the 1.7-tagged commit, with the #910 commit git cherry-pick'd on top of it. (Plus an additional commit of our own to ignore certain widget templates.) Not sure if that is a valid workaround for your case. |
@matthiask Yes, that fix works. Thanks for suggesting it. |
Great! Do you want to submit a pull request?
…On May 4, 2017 00:01, "Tim Bell" ***@***.***> wrote:
@matthiask <https://github.com/matthiask> Yes, that fix works. Thanks for
suggesting it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#938 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAKQ0xnQB6gXVphOXBq2PR6CCW6FtcVks5r2PkngaJpZM4NO7Tm>
.
|
Done! |
Fix circular import issue; django-commons#938
Since commit 68c4410, requests fail with
ImportError: cannot import name 'get_show_toolbar'
. (It was working fine with the previous commit.) Here's the relevant part of the traceback:I'm using Django 1.11, so I'm keen to use a recent commit that includes the fix for #910. (I'm using Python 3.5.2.) The test suite (run with
tox -e py35-dj111
) passes.The text was updated successfully, but these errors were encountered: