[change] Migrate to openwisp-users REST API filters

openwisp / openwisp-radius

Administration web interface and REST API for freeradius 3 build in django & python. Supports captive portal authentication, WPA Enerprise (802.1x), freeradius rlm_rest, social login, Hotspot 2.0 / 802.11u, importing users from CSV, registration of new users and more.

https://openwisp.io/docs/dev/radius/

GNU General Public License v3.0

360 stars 176 forks source link

[change] Migrate to openwisp-users REST API filters #473

Closed nemesifier closed 1 year ago

nemesifier commented 1 year ago

Replace plain django-filters with the multi-tenant capable filters being added in https://github.com/openwisp/openwisp-users/pull/347 (once this is merged).

pandafy commented 1 year ago

We use django-filters on only two API views in openwisp-radius

AccountingView https://github.com/openwisp/openwisp-radius/blob/265f88a90366823ceb03e60d530ba122f7c054a7/openwisp_radius/api/freeradius_views.py#L392-L410

Screenshot from 2023-06-26 23-09-59

UserAccountingView https://github.com/openwisp/openwisp-radius/blob/265f88a90366823ceb03e60d530ba122f7c054a7/openwisp_radius/api/views.py#L417-L424

Screenshot from 2023-06-26 23-10-46

None of filters provides choices that should be filtered by the user's organization.

nemesifier commented 1 year ago

Thanks @pandafy, I am closing this.