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.

Sign up for GitHub

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

Jump to bottom

New option for JSON serialization (#1482) #1485

Merged

cartermp merged 2 commits into fsprojects:main from bongjunj:json-save-options

Apr 30, 2023

Contributor

bongjunj commented Apr 29, 2023

This PR resolves #1482.

Add a new case in JsonSaveOptions
Updated serialization method WriteTo
Passed the tests

https://github.com/bonjune/FSharp.Data/blob/39da6c2d6a841baee019cd03439ad9240b5b91c7/src/FSharp.Data.Json.Core/JsonValue.fs#L20-L32

bongjunj added 2 commits

April 29, 2023 10:20


          Add JsonSaveOptions case (fsprojects#1482)

fix

39da6c2

cartermp approved these changes

View reviewed changes

cartermp merged commit 13afb37 into fsprojects:main

bongjunj deleted the json-save-options branch

May 1, 2023 00:31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet