-
-
Notifications
You must be signed in to change notification settings - Fork 316
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
Has stopped working after upgrade 2.0.4 → 2.0.5 #132
Comments
Oh can you try to import tinymce after staticfiles? |
Yes I use contrib.staticfiles. Here's what my settings.INSTALLED_APPS looks like:
Moving import statement from line 3 to line 17 didn't help (guess it shouldn't in my case). |
I've got the same problem. |
Same problem with django-tinymce 2.0.5. Rollback to version 2.0.4 helped in my case. pip install django-tinymce==2.0.4 |
Is there still an issue with that or are it reproduced on more recent version - 2.2.0. |
I'm trying to use it for a project and I'm seeing this error with 2.2.0. It happens after I do
Django==1.8.4 and python 3.4 Downgrading to 2.0.4 makes TinyMCE work correctly. |
These are the changes that occured between 2.0.4 and 2.0.5. The fact that we use finder at the app loading rather than latter when the function is used can be the problem here. |
@gandalfar Can you somehow give a minimal reproducable code example so we can debug it? |
@gandalfar I have provide some basic app for specified version. Can you check it or add what other package do you use? |
Hi, thanks for being so proactive about this issue. I was swamped with organising conference, but I'm now looking through the 100+ packages in requirements.txt to see if I can reproduce this problem. |
I can't reproduce this on a simple app. Will try again later on primary app, to see if I can find cause by removing things instead of adding them. |
Ok, managed to somehow reproduce this. If I comment out 'haystack' from django-haystack==2.4.1 in my project, then it doesn't crash with django-tinymce==2.3.0 I have to do some more work, to get it to crash in your sample app. |
Same problem here, is there a hotfix for this? |
@wyx2000 can you update to django-tinymce 2.3.0? |
Updated. Still same issue. I have to change the code to remove the line 3 and hardcode the root path Sent from my Samsung device -------- Original message -------- @wyx2000 can you update to django-tinymce 2.3.0 — |
I had the same problem. Moving my import Model into function fixed the issue! |
Does anyone have a solution for this issue? |
There is definitely an issue here. The fix would be to create a lazy finder function. If anyone is interested in filling a pull-request |
Can anyone review #243 and tell me if it works properly for them? |
Just tested it, works well 👍 |
Thanks I am going to make a release then. |
Does anyone needs a 2.0.6 release for this? |
I do! That would be great if you could do a release. |
Please meet 2.0.7 |
Thanks! I appreciate it. |
Is it possible to get this fix on top of 2.6.0? |
Is there anything blocking you to use 2.8.0 directly instead? |
We're trying to upgrade from Django 1.8. |
Would you like me to get a 2.6.1 branch going? |
It's fine, I just somehow missed your previous message. Working on it. |
Released 2.6.1 |
Thank you! |
Here’s a traceback.
The text was updated successfully, but these errors were encountered: