Skip to content

Conversation

RickDMyers
Copy link
Contributor

Refine IncludeFields remarks to highlight that only public fields are serialized and that private fields are not.

Summary

Revised the documentation for JsonSerializerOptions.IncludeFields property clarify that setting the property
to true will only result in 'public' fields being serialized, private fields are not serialized.

Refine IncludeFields remarks to highlight that only public
fields are serialized and that private fields are not.
@RickDMyers RickDMyers requested a review from a team as a code owner August 28, 2024 13:43
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Aug 28, 2024
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-text-json

Copy link

Learn Build status updates of commit 238352d:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Text.Json/JsonSerializerOptions.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@gewarren gewarren enabled auto-merge (squash) October 30, 2024 15:11
Copy link

Learn Build status updates of commit 4a5ec8d:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Text.Json/JsonSerializerOptions.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@gewarren gewarren merged commit 5e37f39 into dotnet:main Oct 30, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.Text.Json community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants