django-extra-views icon indicating copy to clipboard operation
django-extra-views copied to clipboard

SearchableListMixin should manage form

Open typonaut opened this issue 2 years ago • 1 comments

It seems to me that if you are going to have a SearchableListMixin, then that Mixin should also deal with any form input required for the search. ie there should be a form_class property.

typonaut avatar Feb 16 '23 14:02 typonaut

contrib.SearchableListMixin hasn't received a lot of maintenance over the years... probably django-filter deals with this much better now. Are you using the mixin? Feel free to submit a pull request if that change would be useful, it sounds reasonable.

sdolemelipone avatar Jun 19 '23 16:06 sdolemelipone