-
Notifications
You must be signed in to change notification settings - Fork 10.3k
Remove Newtonsoft.Json dependency from JsonPatch #52366
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.
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
Comments
@davidfowl what was the reason for reopening this issue rather than keeping it closed as a dupe of #24333? @stevendarby, @chenzuo I see you guys have not liked the previous comment about closing this as a dupe. What was the reason? Because removing Newtonsoft.Json dependency will be in favor of the framework-provided System.Text.Json instead, which is exactly what the other issue is tracking. |
@mkArtakMSFT I wasn't pleased that MS are recommending other libraries instead of updating their own (see also my post on the linked issue.) |
@mkArtakMSFT I think reopend it because it was closed as completed. We can close this as a dupe now |
Is there an existing issue for this?
Is your feature request related to a problem? Please describe the problem.
I just moved to .NET 8.0, removed Newtonsoft.Json dependency and moved to System.Text.Json.
BUT, when I create an API PATCH method with JsonPatch ( refer to link below ) I have to install Microsoft.AspNetCore.Mvc.NewtonsoftJson:
https://learn.microsoft.com/en-us/aspnet/core/web-api/jsonpatch?view=aspnetcore-8.0
Describe the solution you'd like
Any plans to remove Newtonsoft.Json dependency from JsonPatch and adopt System.Text.Json?
Additional context
No response
The text was updated successfully, but these errors were encountered: