Audit FilterSet definitions application-wide #9283

Closed
opened 2025-12-29 20:47:50 +01:00 by adam · 0 comments
Owner

Originally created by @jeremystretch on GitHub (Feb 23, 2024).

Originally assigned to: @jeremystretch on GitHub.

Proposed Changes

Review all currently defined FilterSet classes and check for:

  • Missing filters
  • Inconsistent names or behavior
  • Suboptimal performance
  • Opportunities for code reduction

Justification

The FilterSet classes defined in NetBox have evolved organically over the past several years and we are overdue for a structured audit of the existing filter sets. This will help us identify inconsistent or suboptimal implementations, and may yield additional work to help standardize and shrink the code base.

Originally created by @jeremystretch on GitHub (Feb 23, 2024). Originally assigned to: @jeremystretch on GitHub. ### Proposed Changes Review all currently defined FilterSet classes and check for: - Missing filters - Inconsistent names or behavior - Suboptimal performance - Opportunities for code reduction ### Justification The FilterSet classes defined in NetBox have evolved organically over the past several years and we are overdue for a structured audit of the existing filter sets. This will help us identify inconsistent or suboptimal implementations, and may yield additional work to help standardize and shrink the code base.
adam added the status: acceptedtype: housekeeping labels 2025-12-29 20:47:50 +01:00
adam closed this issue 2025-12-29 20:47:50 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#9283