Skip to content
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

Serialization/Deserialization net8 with Text.Json doesn't work #98

Open
MaximKitsenko opened this issue Apr 10, 2024 · 3 comments
Open

Serialization/Deserialization net8 with Text.Json doesn't work #98

MaximKitsenko opened this issue Apr 10, 2024 · 3 comments

Comments

@MaximKitsenko
Copy link

I saw there are some converters but there is no for Text Json.

After serialization / deserialization i receive Amount=0, Currency=null
RemyDuijkeren added a commit that referenced this issue Jun 4, 2024
@RemyDuijkeren
Added test cases for System.Text.Json serialization and deserialization
8326659 
#98 (still failing)

Added multiple test cases to test serialization and deserialization of the 'Money' object using the System.Text.Json library. The test cases cover both valid and invalid scenarios and also include testing the serialization and deserialization of nested objects.
RemyDuijkeren added a commit that referenced this issue Jun 11, 2024
@RemyDuijkeren
Add JSON support for NodaMoney #98
045e475 
This commit introduces a new dependency on System.Text.Json. It also adds new code to convert NodaMoney objects to and from JSON. This includes modifications to the NodaMoney.csproj file and the creation of MoneyJsonConverter.cs, which contains the converter implementation.
@markphillips100
Copy link

Hi @RemyDuijkeren will there be a release soon that doesn't depend on Newtonsoft and includes this issues fix?

@RemyDuijkeren
Copy link
Owner

Hi @markphillips100,

End October is the target to release this as v2.0.

Unfortunately not earlier because of being very busy in Aug and enjoying holiday in September.

@markphillips100
Copy link

All good @RemyDuijkeren . Thanks for the update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@MaximKitsenko @markphillips100 @RemyDuijkeren