Skip to content

JsonPatchDocument should use System.Text.Json in ASP.NET 6 #30635

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
Closed
coderdnewbie opened this issue Mar 4, 2021 · 4 comments
Closed

JsonPatchDocument should use System.Text.Json in ASP.NET 6 #30635

coderdnewbie opened this issue Mar 4, 2021 · 4 comments
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-json-patch old-area-web-frameworks-do-not-use *DEPRECATED* This label is deprecated in favor of the area-mvc and area-minimal labels
Milestone
Backlog

Comments

@coderdnewbie
Copy link

The request in #16968 was rejected for ASP.NET 5. Please can it be considered for ASP.NET 6?

The training videos from Cecil are recommending that Newtonsoft be used. It would be better if System.Text.Json did this as it would mean that Newtonsoft would not be required.

Thanks.
@javiercn javiercn added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Mar 4, 2021
@mkArtakMSFT mkArtakMSFT added the enhancement This issue represents an ask for new feature or an enhancement to an existing one label Mar 4, 2021
@mkArtakMSFT mkArtakMSFT added this to the Next sprint planning milestone Mar 4, 2021
@ghost
Copy link

ghost commented Mar 4, 2021

Thanks for contacting us.
We're moving this issue to the Next sprint planning milestone for future evaluation / consideration. We will evaluate the request when we are planning the work for the next milestone. To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@ghost
Copy link

ghost commented Aug 10, 2021

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@pranavkm pranavkm added old-area-web-frameworks-do-not-use *DEPRECATED* This label is deprecated in favor of the area-mvc and area-minimal labels and removed area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates labels Oct 19, 2021
@pranavkm
Copy link
Contributor

Closing as a dupe of #24333.

@p10tyr
Copy link

p10tyr commented Nov 25, 2021

Its been on the backlog for years now.. why is this such an issue ?

@ghost ghost locked as resolved and limited conversation to collaborators Dec 25, 2021
@amcasey amcasey added the area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions label Jun 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-json-patch old-area-web-frameworks-do-not-use *DEPRECATED* This label is deprecated in favor of the area-mvc and area-minimal labels
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
6 participants
@pranavkm @p10tyr @javiercn @amcasey @mkArtakMSFT @coderdnewbie