Skip to content

Improved documentation and tests of ordering filter - #1334

Merged
n2ygk merged 2 commits into
django-json-api:mainfrom
sliverc:ordering_tests
Oct 8, 2026
Merged

n2ygk merged 2 commits into
django-json-api:mainfrom
sliverc:ordering_tests

Conversation

@sliverc

@sliverc sliverc commented Oct 8, 2026

Copy link
Copy Markdown
Member

Fixes #1317
Addresses #468

Description of the Change

I have moved the ordering filter tests into the tests module. As of issue #1317 it was unclear how it is allowed to sort by relationships. Adjusted documentation how this can be done. This already worked, so nothing has changed in DJA functionality therefore no changelog entry is needed.

Checklist

  • PR only contains one change (considered splitting up PR)
  • unit-test added
  • documentation updated
  • CHANGELOG.md updated (only for user relevant changes)
  • author name in AUTHORS

@sliverc
sliverc requested a review from n2ygk October 8, 2026 06:44
@n2ygk
n2ygk merged commit 0ba9539 into django-json-api:main Oct 8, 2026
9 checks passed
@sliverc
sliverc deleted the ordering_tests branch October 8, 2026 12:54
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.

OrderingFilter does not accept related fields (relation.field / relation__field) in sort?

2 participants