Migrate from Newtonsoft.Json to System.Text.Json

[sarahelsaig]

I've noticed there isn't a public issue to track what needs to be done to move over from Json.NET to STJ. I'd like to collect all information into one issue to see if/when this move will be possible and what it will entail, so please chime in. This is also research for OrchardCMS/OrchardCore.Commerce#362.

Blockers

Can we collect any dependencies outside of OC that block this move? Here is what I know of so far:

• YesSql: Any plan to support System.Text.Json? sebastienros/yessql#227

Things to Change

• OrchardCore.Entities.IEntity
• Lots of Newtonsoft.Json.Linq (JToken, JObject, JArray) usages that can be trivially converted to their System.Text.Json.Nodes counterparts.
• Converters:
  • OrchardCore.ContentManagement.ContentItemConverter
  • OrchardCore.DisplayManagement.Notify.NotifyEntryConverter
  • OrchardCore.Workflows.Abstractions.Converters.LocalizedStringConverter

[MikeAlhayek]

We have been slowly migrating to System.Text.Json in OC. Feel free to submit PRs that would convert.

[hishamco]

@MikeAlhayek I will start a PR on sebastienros/yessql#227 then we can work here too to avoid any consequences

[sarahelsaig]

Feel free to submit PRs that would convert.

Sure, that's my plan. I just opened this issue to check if there are any known blockers I might have missed, and to know if someone may already be working on this.

[hishamco]

@DAud-IcI do you want work on this or we might work together to make this done

[sarahelsaig]

We can work together too, whatever is fastest.

[hishamco]

I started working on the Converters

[sebastienros]

Really looking forward to it, thanks to all for working on this.

[Piedone]

This was done in #14572.

[MikeAlhayek]

Also, in the 1.9 release notes we tried to document all the required steps under the breaking changes section ti help users migrate successfully.

[MikeAlhayek]

Here is the reference to the 1.9.0 release notes
https://docs.orchardcore.net/en/latest/docs/releases/1.9.0/