Skip to content

Update JSON serializer options for case insensitivity and null handling#32

Merged
IvanMurzak merged 2 commits intomainfrom
fix/json-format
Dec 5, 2025
Merged

Update JSON serializer options for case insensitivity and null handling#32
IvanMurzak merged 2 commits intomainfrom
fix/json-format

Conversation

@IvanMurzak
Copy link
Copy Markdown
Owner

Enhance JSON serialization by allowing case-insensitive property names and ignoring null properties.

@IvanMurzak IvanMurzak self-assigned this Dec 5, 2025
@IvanMurzak IvanMurzak added the bug Something isn't working label Dec 5, 2025
@IvanMurzak IvanMurzak requested a review from Copilot December 5, 2025 02:16
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 5, 2025

Test Results

    3 files  ±0      3 suites  ±0   3m 19s ⏱️ +13s
  578 tests ±0    578 ✅ ±0  0 💤 ±0  0 ❌ ±0 
1 696 runs  ±0  1 696 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 881e172. ± Comparison against base commit 27d3505.

Copy link
Copy Markdown
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 modifies the JSON serialization configuration to change the property naming strategy and add case-insensitive deserialization support.

Key changes:

  • Removes camelCase property naming policy (sets PropertyNamingPolicy = null)
  • Enables case-insensitive property name matching during deserialization (PropertyNameCaseInsensitive = true)
  • Reorganizes comment placement for better readability

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ReflectorNet/src/Utils/Json/JsonSerializer.cs
@IvanMurzak IvanMurzak merged commit dcf0904 into main Dec 5, 2025
1 check passed
@IvanMurzak IvanMurzak deleted the fix/json-format branch December 5, 2025 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants