Fix searching a BooleanField and if we have choice activated

morlandi / django-ajax-datatable

A Django app which provides the integration of a Django project with the jQuery Javascript library DataTables.net

MIT License

204 stars 64 forks source link

Closed etiennepouliot closed 2 years ago

etiennepouliot commented 2 years ago

Does not make sense to use __icontains by default when we have a drop box :

Make more sense

Also fix searching a BooleanField :

Same without a dropbox :

morlandi commented 2 years ago

Thank you @etiennepouliot for your detailed analisys. I do agree on your comments. Will review and merge soon

etiennepouliot commented 2 years ago

Will redo this PR without the unnecessary commit of me trying to merge the upstream correctly.