## Summary (#43349)

Changed the code samples with System.Text.Json Converters (Now readonly property). Fixes #43178, makes #43179 obsolete.
dotnet · Nov 15, 2024 · 4e51802 · 4e51802
1 parent 043b3c1
commit 4e51802
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/...tion/system-text-json/snippets/how-to/csharp/RegisterConverterWithConvertersCollection.cs b/...tion/system-text-json/snippets/how-to/csharp/RegisterConverterWithConvertersCollection.cs
@@ -13,12 +13,9 @@ public static void Run()
             // <Serialize>
             var serializeOptions = new JsonSerializerOptions
             {
-                WriteIndented = true,
-                Converters =
-                {
-                    new DateTimeOffsetJsonConverter()
-                }
+                WriteIndented = true
             };
+			serializeOptions.Converters.Add(new DateTimeOffsetJsonConverter());
 
             jsonString = JsonSerializer.Serialize(weatherForecast, serializeOptions);
             // </Serialize>

diff --git a/docs/standard/serialization/system-text-json/snippets/how-to/csharp/RoundtripStackOfT.cs b/docs/standard/serialization/system-text-json/snippets/how-to/csharp/RoundtripStackOfT.cs
@@ -13,10 +13,8 @@ public static void Run()
 
             Console.WriteLine("Deserialize JSON string [1, 2, 3] with custom converter, then serialize it back to JSON.");
             // <Register>
-            var options = new JsonSerializerOptions
-            {
-                Converters = { new JsonConverterFactoryForStackOfT() },
-            };
+            var options = new JsonSerializerOptions();
+            options.Converters.Add(new JsonConverterFactoryForStackOfT());
             // </Register>
             stack = JsonSerializer.Deserialize<Stack<int>>("[1, 2, 3]", options)!;
             serialized = JsonSerializer.Serialize(stack, options);