Skip to content

Add support for SortFieldsByName-setting in FilterInputType and SortInputType #8515

Merged
michaelstaib merged 9 commits intoChilliCream:mainfrom
N-Olbert:4685
Mar 9, 2026
Merged

Add support for SortFieldsByName-setting in FilterInputType and SortInputType #8515
michaelstaib merged 9 commits intoChilliCream:mainfrom
N-Olbert:4685

Conversation

@N-Olbert
Copy link
Copy Markdown
Contributor

@N-Olbert N-Olbert commented Aug 9, 2025

Summary of the changes (Less than 80 chars)

  • Use the "full" CompleteFields-Method
  • Note: The removal of the no longer used public method is a breaking change in theory

Closes #4685

@michaelstaib
Copy link
Copy Markdown
Member

I have moved this one to 16 as we have more work planned in this area.

@michaelstaib michaelstaib marked this pull request as draft August 28, 2025 06:34
# Conflicts:
#	src/HotChocolate/Data/src/Data/Sorting/SortInputType.cs
#	src/HotChocolate/Data/test/Data.Filters.Tests/FilterInputTypeTest.cs
#	src/HotChocolate/Data/test/Data.Sorting.Tests/SortInputTypeTests.cs
@github-actions github-actions Bot added the 📚 documentation This issue is about working on our documentation. label Mar 9, 2026
@michaelstaib michaelstaib marked this pull request as ready for review March 9, 2026 10:47
@michaelstaib michaelstaib changed the title Filter- and SortInputType now honor the SortFieldsByName-setting Add support for SortFieldsByName-setting in FilterInputType and SortInputType Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📚 documentation This issue is about working on our documentation. 🌶️ hot chocolate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SortFieldsByName does not appear to work for filter/sort input types.

2 participants