Consolidate FilterSet classes #4849

Closed
opened 2025-12-29 19:21:15 +01:00 by adam · 0 comments
Owner

Originally created by @jeremystretch on GitHub (Apr 29, 2021).

Originally assigned to: @jeremystretch on GitHub.

Proposed Changes

We have various FilterSet base classes, including:

  • BaseFilterSet
  • CreatedUpdatedFilterSet
  • CustomFieldModelFilterSet
  • NameSlugSearchFilterSet

We can consolidate these into fewer classes aligned with NetBox base models (e.g. PrimaryModel, OrganizationalModel, etc.).

Justification

  • Cleans up the code base a bit
  • Provides clearer intent as to the purpose of each FilterSet class
Originally created by @jeremystretch on GitHub (Apr 29, 2021). Originally assigned to: @jeremystretch on GitHub. ### Proposed Changes We have various FilterSet base classes, including: * BaseFilterSet * CreatedUpdatedFilterSet * CustomFieldModelFilterSet * NameSlugSearchFilterSet We can consolidate these into fewer classes aligned with NetBox base models (e.g. PrimaryModel, OrganizationalModel, etc.). ### Justification * Cleans up the code base a bit * Provides clearer intent as to the purpose of each FilterSet class
adam added the status: acceptedtype: housekeeping labels 2025-12-29 19:21:15 +01:00
adam closed this issue 2025-12-29 19:21:15 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#4849