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

Migrate from Newtonsoft.Json to System.Text.Json #14609

Closed
wants to merge 27 commits into from
Closed

Migrate from Newtonsoft.Json to System.Text.Json #14609

wants to merge 27 commits into from

Conversation

sarahelsaig
Copy link
Contributor

Followup to #14457

hishamco and others added 26 commits October 8, 2023 20:47
@hishamco
Convert Newtonsoft converters to STJ converters
b78cdb4
@hishamco
Fix the build
b2ce4be
@hishamco
ContentItemConverter
cf8b11e
@sarahelsaig
Merge branch 'main' into daud-ici/system-text-json
0a9b3e6
@sarahelsaig
Replace JObject properties.
d862aac
@sarahelsaig
Replace JArray properties.
25d4e6c
@sarahelsaig
Store ContentElement.Data and IEntity.Properties as JsonObject.
d450344
@sarahelsaig
Convert and simplify template steps.
e778347
@sarahelsaig
Methods for getting and setting properties without doing Json stuff.
41c2acf
@sarahelsaig
Add ContentItem.Parse(object) helper.
06cda41
@sarahelsaig
Update ContentDefinition.
cda757c
@sarahelsaig
Use ContentDefinition.GetSettings.
2241024
@sarahelsaig
Convert .ToObject() to .Deserialize().
19b7a4a
@sarahelsaig
Convert JObject.FromObject().
3b5f884
@sarahelsaig
Convert all deployment steps.
eb525d4
@sarahelsaig
unusing
22f55be
@sarahelsaig
Use IEntity.Put wherever possible.
41136cf
@sarahelsaig
Use Entity.TryAs wherever possible.
975257c
@sarahelsaig
Fix IShape.
8d268d2
@sarahelsaig
Content management builders.
1959c41
@sarahelsaig
Fix RecipeReader.
682f7f3
@sarahelsaig
Fix ElasticIndexStep.
f341de2
@sarahelsaig
Fix Activity.
452917a
@sarahelsaig
Fix OrchardCore.Apis.GraphQL.Client.ContentResource.
e5d4a3d
@sarahelsaig
Fix compile time errors in OrchardCore.Tests.
1c3eb4e
@sarahelsaig
Fix some "new JObject" occurrences.
e1884a3
@sarahelsaig
Merge remote-tracking branch 'upstream/main' into daud-ici/system-tex…
09eb28d 
…t-json

# Conflicts:
#	src/OrchardCore.Modules/OrchardCore.Spatial/Views/GeoPointField.Edit.cshtml
#	src/OrchardCore.Modules/OrchardCore.Workflows/Services/WorkflowManager.cs
#	src/OrchardCore/OrchardCore/Shell/ShellSettingsManager.cs
@sarahelsaig sarahelsaig marked this pull request as draft October 30, 2023 12:33
@sarahelsaig
Copy link
Contributor Author

Sorry, this is still a draft and didn't mean to (automatically) request review

@sarahelsaig sarahelsaig mentioned this pull request Oct 30, 2023
Merged
@sarahelsaig sarahelsaig deleted the daud-ici/system-text-json branch August 27, 2024 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sfmskywalker sfmskywalker Awaiting requested review from sfmskywalker

@deanmarcussen deanmarcussen Awaiting requested review from deanmarcussen

@kevinchalet kevinchalet Awaiting requested review from kevinchalet

@jtkech jtkech Awaiting requested review from jtkech

@agriffard agriffard Awaiting requested review from agriffard

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sarahelsaig @hishamco