-
Notifications
You must be signed in to change notification settings - Fork 13
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
Nested inlines not displaying #41
Comments
Hi @christianpbrink, sorry to hear about your issue. Could you tell me what versions of Django and Client Admin you are using? |
Thanks for replying so fast. Django 1.6.5, django-client-admin 1.1.0 On Thu, Jun 12, 2014 at 2:24 PM, Kyle Rimkus [email protected]
|
Thanks for the info. I suspect that this bug is caused by some changes in Django 1.6+ inlines.js code. The missing management form probably means that the javascript is not properly copying and replacing ids when it duplicates the hidden inline fieldsets. I will go through the overridden file to see if anything needs to be updated, but I won't be able to get to it today. The relevant javascript logic is here in case you are curious: |
I've just installed django-client-admin per the README instructions. It works on a basic level -- it's taken over rendering my admin pages, with proper styling and everything. However, it won't display nested inline menus at all.
Not only can I not see the nested inline, but I can't submit the admin form without getting a
ValidationError
([u'ManagementForm data is missing or has been tampered with']
). Some excerpts from the stacktrace:Has anyone else run into similar behavior?
Here's my
settings.INSTALLED_APPS
:The relevant slice of
urls.urlpatterns
:The relevant bits of
myapp.admin
:I've tried all combinations of
StackedInline
andTabularInline
; all fail the same way.I've tried manually creating an instance of the nested inline's model, hoping that would force it to render the nested formset -- I could more easily start debugging this if I could see it work properly in one scenario. But no luck.
The text was updated successfully, but these errors were encountered: