crucialfelix/django-ajax-selects — comment on commit 2190a31a1f6a471737e8fa9fb574026ef93a3374
File:  line 0
By sjrd:

Hum. IMHO this default is too for-admin directed. I would have seen this `check_auth` method in a mixin/subclass named `StaffLookupChannel` (or any other name). The other defaults in `LookupChannel` are general-purpose, but that one is too related to a usage of this app in an admin context.

It will add a burden to people using django-ajax-selects in a front-end. Having a separate mixin/subclass for `check_auth` won't add that burden, and for people using it in an admin it's only a matter of choosing another superclass for their lookup.