Skip to content

Further filter fixes.#547

Merged
oyvindhagberg merged 19 commits intounioslo:masterfrom
terjekv:filter_fixes
Aug 22, 2024
Merged

Further filter fixes.#547
oyvindhagberg merged 19 commits intounioslo:masterfrom
terjekv:filter_fixes

Conversation

@terjekv
Copy link
Collaborator

@terjekv terjekv commented Aug 6, 2024

Manually sets the appropriate filters for most filter sets. A number of attempts at automating the code led nowhere good, so this is where we are.

A lot of this may be cleaned up when DRF 3.15.3 is released as we can bump DRF to 3.15 that officially supports Django5. We are stuck on 3.14.0 due to encode/django-rest-framework#9358 but encode/django-rest-framework#9483 has been merged an hopefully makes 3.15.3.

@terjekv terjekv self-assigned this Aug 6, 2024
@coveralls
Copy link
Collaborator

coveralls commented Aug 6, 2024

Coverage Status

coverage: 98.721% (-0.1%) from 98.842%
when pulling 0c442c1 on terjekv:filter_fixes
into 8952b79 on unioslo:master.

  - Also reformat as per ruff.
  - This should catch the generic issue of filtering on IDs. Ideally we'd do this for every model that supports ID...
@terjekv terjekv requested a review from pederhan August 12, 2024 05:16
Copy link
Member

@pederhan pederhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

  - Match exact CIDR or IP within a CIDR.
@oyvindhagberg oyvindhagberg merged commit 9df966e into unioslo:master Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants