-
Notifications
You must be signed in to change notification settings - Fork 25.1k
Description
Description
There are a few fixes and improvements we should make to the JSON Patch documentation for .NET 10.
-
Move JSON Patch article under APIs (out from under Controllers), after the "Middleware" topic.
-
Remove JsonPatch support from "Consider controller-based APIs if you need:" here:
In the "Enable JSON Patch support with System.Text.Json" section
-
Remove "--prerelease" from the dotnet add command to add Microsoft.AspNetCore.JsonPatch.SystemTextJson
-
References to "T" should be "TModel"
The "Action method code applying JSON Patch" section
- Replace this with a section that describes the Patch method in the CustomerApi.cs file (the Minimal APIs PATCH endpoint)
- Need to update the section title
- Note the example error response should be updated
Not sure we need this section: Apply a JSON Patch document to an object. Can we remove it?
- Or maybe this is an opportunity to use a polyglot notebook?
Not sure we need this section: Example: Apply a JsonPatchDocument with error handling. Can we remove it?
In the "Get the code" section:
-
Update the sample project with code in the main branch of this repo:
-
The "View or download sample code" should link to
-
Replace the instructions for testing the sample (I think they are unclear or broken -- what JSON project folder?) with something like this:
- To test the sample, run the app and send HTTP requests using the included http file.
Under "Additional resources"
- Source code link should link to:
Page URL
https://learn.microsoft.com/en-us/aspnet/core/web-api/jsonpatch?view=aspnetcore-10.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/web-api/jsonpatch.md
Document ID
6a88d237-1a03-b865-e3c5-20ceedef2a1f
Platform Id
54f6475a-b72c-2ccc-3ab8-5326d9d05c50
Article author
Metadata
- ID: 593f528c-61b9-7021-335d-c1c0ec1814d1
- PlatformId: 54f6475a-b72c-2ccc-3ab8-5326d9d05c50
- Service: aspnet-core
- Sub-service: web-api
