runtime icon indicating copy to clipboard operation
runtime copied to clipboard

Add custom JsonConverter for BinaryData that converts to Base64 string

Open ovidiucosteanet opened this issue 3 years ago • 3 comments

  • Created custom converter as per described in the issue
  • Added the converter also to the ref BinaryData class
  • Added appropriate tests for serialization and deserialization of a class that has a member of BinaryData type

Fixes #70267

ovidiucosteanet avatar Aug 05 '22 09:08 ovidiucosteanet

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.

CLA assistant check
All CLA requirements met.

dnfadmin avatar Aug 05 '22 09:08 dnfadmin

Tagging subscribers to this area: @dotnet/area-system-memory See info in area-owners.md if you want to be subscribed.

Issue Details
  • Created custom converter as per described in the issue
  • Added the converter also to the ref BinaryData class
  • Added appropriate tests for serialization and deserialization of a class that has a member of BinaryData type

Fixes #70267

Author: ovidiucosteanet
Assignees: -
Labels:

area-System.Memory, new-api-needs-documentation

Milestone: -

msftbot[bot] avatar Aug 05 '22 09:08 msftbot[bot]

Thank you @adamsitnik!

ovidiucosteanet avatar Aug 09 '22 08:08 ovidiucosteanet