#9025: Nested Inline Support in Admin
-------------------------------+------------------------------------
     Reporter:  pixelcort      |                    Owner:  Gargamel
         Type:  New feature    |                   Status:  assigned
    Component:  contrib.admin  |                  Version:  master
     Severity:  Normal         |               Resolution:
     Keywords:                 |             Triage Stage:  Accepted
    Has patch:  1              |      Needs documentation:  0
  Needs tests:  1              |  Patch needs improvement:  1
Easy pickings:  0              |                    UI/UX:  1
-------------------------------+------------------------------------

Comment (by kmike):

 Replying to [comment:121 Gargamel]:
 > I fixed the second bug reported by rugbyjock. As of this patch, every
 reported bug should have been fixed. I'll monitor this page until friday,
 and if no new bug reports have come up, I'll make a pull request to the
 master branch. Happy testing, I know I will :D

 Hey,
 thanks for a work on this!

 I didn't try the patch (only read it) and it seems there are some small
 issues:

 
https://code.djangoproject.com/attachment/ticket/9025/nested_inlines_finished.diff#L98
 - this is incorrect in Python3 (remove "u");
 
https://code.djangoproject.com/attachment/ticket/9025/nested_inlines_finished.diff#L519
 - it seems 'var' is missed and this variable is defined as global;
 
https://code.djangoproject.com/attachment/ticket/9025/nested_inlines_finished.diff#L525
 (and further) - the same applies to border_class variable.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/9025#comment:122>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to