Frankie Dintino
Frankie Dintino
I'll look into merging these pull requests sometime this week.
I've tried several different variations on your example code and am unable to reproduce this. Could you provide a test repo (or even just a functional models.py and admin.py) that...
I'd welcome a PR to add support for django-tabbed-admin, but I won't be working on this any time in the near future—it's hard enough keeping up with new django admin...
This also seems like a feature that would exist outside of django-nested-admin. If it were added to django-grappelli, for instance, in all likelihood it would work as-is with this project.
This is a brilliant idea. I think I'll probably put it behind a setting and disable it by default, but this will help us as well with performance.
This turns out to be not as straightforward as I would have hoped. There are more variables involved than just `formset_class_name` that would need to vary the cache.
@learning-91 are those the built-in django admin select2 fields, or are you using another package for those?
Sorry, to clarify my question: which dal package are you using? `dal_select2`?
To be on the safe side, if you're going to change that, it ought to be in the `DjangoFormset .remove()` method itself, before this line: https://github.com/theatlantic/django-nested-admin/blob/de440b1333eeb9bd85ca412d717d0e89652207e7/nested_admin/static/nested_admin/src/nested-admin/jquery.djangoformset.js#L166
@jwilder any feedback?