Skip to content

System.Text.Json support via new Activout.RestClient.Json package#67

Merged
twogood merged 19 commits intomainfrom
System.Text.Json
May 29, 2025
Merged

System.Text.Json support via new Activout.RestClient.Json package#67
twogood merged 19 commits intomainfrom
System.Text.Json

Conversation

@twogood
Copy link
Owner

@twogood twogood commented May 28, 2025

Resolves #46

@twogood twogood requested a review from Copilot May 28, 2025 23:12
@twogood twogood self-assigned this May 28, 2025

This comment was marked as outdated.

PropertyNameCaseInsensitive = true
@twogood twogood requested a review from Copilot May 29, 2025 07:45

This comment was marked as outdated.

@twogood twogood requested a review from Copilot May 29, 2025 08:25

This comment was marked as outdated.

@twogood twogood requested a review from Copilot May 29, 2025 19:57

This comment was marked as outdated.

@twogood twogood requested a review from Copilot May 29, 2025 20:27

This comment was marked as outdated.

twogood and others added 2 commits May 29, 2025 22:30
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: David Eriksson <david@activout.se>
@twogood twogood changed the title System.text.json System.Text.Json support via new Activout.RestClient.Json package May 29, 2025
@twogood twogood marked this pull request as ready for review May 29, 2025 20:38
@twogood twogood requested a review from Copilot May 29, 2025 20:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for System.Text.Json serialization and deserialization by introducing a new package, Activout.RestClient.Json. Key changes include:

  • New serializer and deserializer classes using System.Text.Json with support for custom options.
  • A default configuration class along with a JSON converter for simple value objects.
  • Extension methods and tests ensuring proper integration and behavior.

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
ActivoutRestClient.sln Added new projects for Activout.RestClient.Json and its tests.
SystemTextJsonSerializer.cs Implements JSON serialization using System.Text.Json.
SystemTextJsonDeserializer.cs Implements JSON deserialization using System.Text.Json.
SystemTextJsonDefaults.cs Provides default serializer options and supported media types.
SimpleValueObjectConverter.cs Adds a JSON converter for simple value objects.
RestClientBuilderJsonExtensions.cs Registers the serializer and deserializer via extension method.
README.md Documents the usage of Activout.RestClient.Json.
Activout.RestClient.Json.csproj Project configuration for the new JSON package.
SystemTextJsonDeserializerTest.cs Tests for verifying JSON deserialization behavior.
SimpleValueObjectTest.cs Tests for verifying serialization and deserialization of value objects.
Activout.RestClient.Json.Test.csproj Project configuration for the test project.

@twogood twogood merged commit a53c044 into main May 29, 2025
8 checks passed
@twogood twogood deleted the System.Text.Json branch May 29, 2025 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create separate packages for JSON support

2 participants