Skip to content

Ensured that sparse fieldset support formatted field names#1286

Merged
n2ygk merged 2 commits intodjango-json-api:mainfrom
sliverc:sparse_format
Jul 18, 2025
Merged

Ensured that sparse fieldset support formatted field names#1286
n2ygk merged 2 commits intodjango-json-api:mainfrom
sliverc:sparse_format

Conversation

@sliverc
Copy link
Copy Markdown
Member

@sliverc sliverc commented Jul 17, 2025

Fixes #1053

Description of the Change

Added missed undoing of formatting when interpreting sparse field set field query parameter.

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

@n2ygk
Copy link
Copy Markdown
Contributor

n2ygk commented Jul 17, 2025

@sliverc is this a request to review?

@sliverc sliverc requested a review from n2ygk July 18, 2025 03:09
@sliverc
Copy link
Copy Markdown
Member Author

sliverc commented Jul 18, 2025

yes 😄

@n2ygk n2ygk merged commit 2dc5571 into django-json-api:main Jul 18, 2025
9 checks passed
@sliverc sliverc deleted the sparse_format branch July 18, 2025 12:57
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.

SparseFieldsetsMixin does not use the correct configured JSON_API_FORMAT_FIELD_NAMES

2 participants