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

Added System.Text.Json serialization-based JsonPatch implementation #61313

Draft
wants to merge 27 commits into
base: main
Choose a base branch
from
Draft

Added System.Text.Json serialization-based JsonPatch implementation #61313

wants to merge 27 commits into from

Conversation

mkArtakMSFT
Copy link
Member

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Apr 4, 2025
danmoseley and others added 25 commits April 4, 2025 17:41
@danmoseley
update community-triager list (#61309)
0bf1cac 
* update triager list

* Remove extra newline in area-owners.md
@dependabot
[main] (deps): Update dotnet/arcade requirement to 0c0caeb605f7b2b994…
fca94dd 
…08bfa6b18b13526d747727 (#61321)

Updates the requirements on [dotnet/arcade](https://github.com/dotnet/arcade) to permit the latest version.
- [Commits](https://github.com/dotnet/arcade/commits/0c0caeb605f7b2b99408bfa6b18b13526d747727)

---
updated-dependencies:
- dependency-name: dotnet/arcade
  dependency-version: 0c0caeb605f7b2b99408bfa6b18b13526d747727
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dotnet-maestro
Update dependencies from https://github.com/dotnet/source-build-refer…
258234a 
…ence-packages build 20250404.2 (#61331)

Microsoft.SourceBuild.Intermediate.source-build-reference-packages
 From Version 10.0.620202 -> To Version 10.0.620402

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
@dotnet-maestro
[main] Update dependencies from dotnet/efcore, dotnet/runtime (#61314)
5f174c0 
* Update dependencies from https://github.com/dotnet/runtime build 20250403.14

Microsoft.Bcl.AsyncInterfaces , Microsoft.Bcl.TimeProvider , Microsoft.Extensions.Caching.Abstractions , Microsoft.Extensions.Caching.Memory , Microsoft.Extensions.Configuration , Microsoft.Extensions.Configuration.Abstractions , Microsoft.Extensions.Configuration.Binder , Microsoft.Extensions.Configuration.CommandLine , Microsoft.Extensions.Configuration.EnvironmentVariables , Microsoft.Extensions.Configuration.FileExtensions , Microsoft.Extensions.Configuration.Ini , Microsoft.Extensions.Configuration.Json , Microsoft.Extensions.Configuration.UserSecrets , Microsoft.Extensions.Configuration.Xml , Microsoft.Extensions.DependencyInjection , Microsoft.Extensions.DependencyInjection.Abstractions , Microsoft.Extensions.DependencyModel , Microsoft.Extensions.Diagnostics , Microsoft.Extensions.Diagnostics.Abstractions , Microsoft.Extensions.FileProviders.Abstractions , Microsoft.Extensions.FileProviders.Composite , Microsoft.Extensions.FileProviders.Physical , Microsoft.Extensions.FileSystemGlobbing , Microsoft.Extensions.HostFactoryResolver.Sources , Microsoft.Extensions.Hosting , Microsoft.Extensions.Hosting.Abstractions , Microsoft.Extensions.Http , Microsoft.Extensions.Logging , Microsoft.Extensions.Logging.Abstractions , Microsoft.Extensions.Logging.Configuration , Microsoft.Extensions.Logging.Console , Microsoft.Extensions.Logging.Debug , Microsoft.Extensions.Logging.EventLog , Microsoft.Extensions.Logging.EventSource , Microsoft.Extensions.Logging.TraceSource , Microsoft.Extensions.Options , Microsoft.Extensions.Options.ConfigurationExtensions , Microsoft.Extensions.Options.DataAnnotations , Microsoft.Extensions.Primitives , Microsoft.Internal.Runtime.AspNetCore.Transport , Microsoft.NET.Runtime.MonoAOTCompiler.Task , Microsoft.NET.Runtime.WebAssembly.Sdk , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.AOT.win-x64.Cross.browser-wasm , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.BrowserDebugHost.Transport , Microsoft.NETCore.Platforms , System.Collections.Immutable , System.Composition , System.Configuration.ConfigurationManager , System.Diagnostics.DiagnosticSource , System.Diagnostics.EventLog , System.Diagnostics.PerformanceCounter , System.DirectoryServices.Protocols , System.Formats.Asn1 , System.IO.Hashing , System.IO.Pipelines , System.Memory.Data , System.Net.Http.Json , System.Net.Http.WinHttpHandler , System.Net.ServerSentEvents , System.Numerics.Tensors , System.Reflection.Metadata , System.Resources.Extensions , System.Runtime.Caching , System.Security.Cryptography.Pkcs , System.Security.Cryptography.Xml , System.Security.Permissions , System.ServiceProcess.ServiceController , System.Text.Encodings.Web , System.Text.Json , System.Threading.AccessControl , System.Threading.Channels , System.Threading.RateLimiting , Microsoft.SourceBuild.Intermediate.runtime.linux-x64
 From Version 10.0.0-preview.4.25202.14 -> To Version 10.0.0-preview.4.25203.14

* Update dependencies from https://github.com/dotnet/efcore build 20250403.5

dotnet-ef , Microsoft.EntityFrameworkCore , Microsoft.EntityFrameworkCore.Design , Microsoft.EntityFrameworkCore.InMemory , Microsoft.EntityFrameworkCore.Relational , Microsoft.EntityFrameworkCore.Sqlite , Microsoft.EntityFrameworkCore.SqlServer , Microsoft.EntityFrameworkCore.Tools
 From Version 10.0.0-preview.4.25201.3 -> To Version 10.0.0-preview.4.25203.5

---------

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
@carlossanlop
Update maintenance-packages versions (#61326)
4a906a0
@dotnet-maestro
[main] Update dependencies from dotnet/runtime (#61332)
eb67ecc 
[main] Update dependencies from dotnet/runtime
@mkArtakMSFT
Migrating to the new solution file format - .slnx (#61273)
b4c1921 
* Converted to the SLNX solution format

* Added the default solution setting to the devcontainer settings.
@dotnet-maestro
Update dependencies from https://github.com/dotnet/runtime build 2025…
ca513a1 
…0406.1 (#61356)

[main] Update dependencies from dotnet/runtime
@dotnet-maestro
Update dependencies from https://github.com/dotnet/extensions build 2…
55a1ae7 
…0250404.8 (#61353)

[main] Update dependencies from dotnet/extensions
@dotnet-maestro
Update dependencies from https://github.com/dotnet/efcore build 20250…
614e753 
…406.1 (#61359)

dotnet-ef , Microsoft.EntityFrameworkCore , Microsoft.EntityFrameworkCore.Design , Microsoft.EntityFrameworkCore.InMemory , Microsoft.EntityFrameworkCore.Relational , Microsoft.EntityFrameworkCore.Sqlite , Microsoft.EntityFrameworkCore.SqlServer , Microsoft.EntityFrameworkCore.Tools
 From Version 10.0.0-preview.4.25203.5 -> To Version 10.0.0-preview.4.25206.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
@Rick-Anderson @halter73
Typo in AuthenticationMetrics.cs description (#61312)
81db799 
* Typo in AuthenticationMetrics.cs description

Copy/paste error?

* Update src/Http/Authentication.Core/src/AuthenticationMetrics.cs

Co-authored-by: Stephen Halter <[email protected]>

* Update sign-in counter description

---------

Co-authored-by: Stephen Halter <[email protected]>
@github-actions
Update .NET SDK to 10.0.100-preview.4.25207.6 (#61362)
2a723be 
* Update .NET SDK
@mkArtakMSFT
Consistent parameter naming and removed a leftover dynamic object test
c90bec5
@mkArtakMSFT
Shrink the public API of the library.
93b5158
@mkArtakMSFT
Removed removed type information
d1b3bcb
@mkArtakMSFT
Added System.Text.Json-based implementation for JsonPatch.
ad726ed
@mkArtakMSFT
Added a polimorphic list validation scenario
346859f
@mkArtakMSFT
Consistent parameter naming and removed a leftover dynamic object test
5259dd3
@mkArtakMSFT
Shrink the public API of the library.
433e508
@mkArtakMSFT
Removed removed type information
4827156
@mkArtakMSFT
merging
77a2393
@mkArtakMSFT
Removed solution file
50b9ab2
@mkArtakMSFT
Added the JsonPatch.SystemTextJson project to the solution
305136c
@mkArtakMSFT
Update the public API
6e12414
@mkArtakMSFT
Removed unused type
0a18aaf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eiriktsarpalis eiriktsarpalis Awaiting requested review from eiriktsarpalis

@Rick-Anderson Rick-Anderson Awaiting requested review from Rick-Anderson Rick-Anderson will be requested when the pull request is marked ready for review Rick-Anderson is a code owner

@tdykstra tdykstra Awaiting requested review from tdykstra tdykstra will be requested when the pull request is marked ready for review tdykstra is a code owner

@wtgodbe wtgodbe Awaiting requested review from wtgodbe wtgodbe will be requested when the pull request is marked ready for review wtgodbe is a code owner

@BrennanConroy BrennanConroy Awaiting requested review from BrennanConroy BrennanConroy will be requested when the pull request is marked ready for review BrennanConroy is a code owner

@halter73 halter73 Awaiting requested review from halter73 halter73 will be requested when the pull request is marked ready for review halter73 is a code owner

@JamesNK JamesNK Awaiting requested review from JamesNK JamesNK will be requested when the pull request is marked ready for review JamesNK is a code owner

@mgravell mgravell Awaiting requested review from mgravell mgravell will be requested when the pull request is marked ready for review mgravell is a code owner

@captainsafia captainsafia Awaiting requested review from captainsafia captainsafia will be requested when the pull request is marked ready for review captainsafia is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

JsonPatchDocument should use System.Text.Json
4 participants
@mkArtakMSFT @danmoseley @carlossanlop @Rick-Anderson