Ensure Stable Serialization in both Newtonsoft and System.Text

Hochfrequenz / BO4E-dotnet

C# / .NET Core implementation of Business Objects for Energy (BO4E). Full Docs: https://hochfrequenz.github.io/bo4e-livedocs/api/BO4E.BO.html

https://hochfrequenz.de

MIT License

5 stars 4 forks source link

Ensure Stable Serialization in both Newtonsoft and System.Text #164

Open hf-kklein opened 2 years ago

hf-kklein commented 2 years ago

https://github.com/Hochfrequenz/BO4E-dotnet/blob/a71f590f46ff4aa693208283206ae092260309e5/BO4E/BO/Energiemenge.cs#L58

[ ] adding JsonPropertyOrder to all BOs and COM Properties
[ ] adding the newtonsoft Order=... attribute arguemtn to those BOs (the new ones) that miss it.

hf-kklein commented 2 years ago

strafarbeit.

hf-kklein commented 2 years ago

There is a draft with a (failing) unittest here: https://github.com/Hochfrequenz/BO4E-dotnet/pull/176

hf-kklein commented 2 years ago

I added a test in #176 . Now we can slowly, step by step, add the attributes and remove those BOs and COMs that have been resolved from the ignore list.