-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support async KeyValuePair (de)serialization #36607
Conversation
...ystem.Text.Json/src/System/Text/Json/Serialization/Converters/Value/KeyValuePairConverter.cs
Show resolved
Hide resolved
...tem/Text/Json/Serialization/Converters/Object/ObjectWithParameterizedConstructorConverter.cs
Show resolved
Hide resolved
...tem/Text/Json/Serialization/Converters/Object/ObjectWithParameterizedConstructorConverter.cs
Show resolved
Hide resolved
...ystem.Text.Json/src/System/Text/Json/Serialization/Converters/Value/KeyValuePairConverter.cs
Outdated
Show resolved
Hide resolved
Do we have stream-based tests with KeyValuePair? |
...braries/System.Text.Json/tests/Serialization/CollectionTests/CollectionTests.KeyValuePair.cs
Show resolved
Hide resolved
...braries/System.Text.Json/tests/Serialization/CollectionTests/CollectionTests.KeyValuePair.cs
Outdated
Show resolved
Hide resolved
Yes, see the diff in |
{ | ||
// Used for value converters that need to re-enter the serializer since it will support JsonPath | ||
// and reference handling. | ||
internal abstract class JsonValueConverter<T> : JsonConverter<T> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This converter type may come back depending on work in #36785.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks
Refactors the KeyValuePair converter to use logic for deserializing with parameterized ctors and leverage the async support (for reading and writing).
Fixes #32148 (no longer applicable following this change).