JamesRandall / FunctionMonkey

Write more elegant Azure Functions with less boilerplate, more consistency, and support for REST APIs. Docs can be found at https://functionmonkey.azurefromthetrenches.com
MIT License
294 stars 50 forks source link

Replace Newtonsoft.Json with new System.Text.Json Api's #104

Closed MrBliz closed 4 years ago

MrBliz commented 4 years ago

As per this tweet.

https://twitter.com/AzureTrenches/status/1206627061273649153?s=20

I Should have a PR for you by the weekend.

mdg215199 commented 4 years ago

I would hold on doing this (or at least give the option of continue using newtonsoft). Main reason I say this, this was a huge issue for my development: https://github.com/dotnet/corefx/issues/38579

I was excited to use System.text.json, but I ended up having to pull out due to the issues in that ticket.

JamesRandall commented 4 years ago

I've also run into issues with not being able to instruct it to ignore properties, will close this for now and can look at it again when System.Text.Json is more mature.