Add JsonIgnoreCondition & per-property ignore logic #34049

layomia · 2020-03-25T00:36:28Z

Dotnet-GitSync-Bot · 2020-03-25T00:36:31Z

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

ahsonkhan · 2020-03-25T01:05:49Z

cc @CodeBlanch

src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonParameterInfoOfT.cs

src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonIgnoreCondition.cs

src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonParameterInfo.cs

src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonParameterInfoOfT.cs

src/libraries/System.Text.Json/tests/Serialization/PropertyVisibilityTests.cs

jozkee

Just a few questions but overall; looks good.

src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonParameterInfoOfT.cs

src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonClassInfo.cs

src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonPropertyInfo.cs

src/libraries/System.Text.Json/tests/Serialization/PropertyVisibilityTests.cs

Add JsonIgnoreCondition & per-property ignore logic

f01f115

layomia added the area-System.Text.Json label Mar 25, 2020

layomia added this to the 5.0 milestone Mar 25, 2020

layomia requested review from steveharter and jozkee March 25, 2020 00:36

layomia requested a review from ahsonkhan as a code owner March 25, 2020 00:36

layomia self-assigned this Mar 25, 2020

Dotnet-GitSync-Bot added the new-api-needs-documentation label Mar 25, 2020

layomia commented Mar 25, 2020

View reviewed changes

src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonParameterInfoOfT.cs Show resolved Hide resolved