Skip to content

Update ContentSerializer Settings to allow data POST back to AutoTask API#3

Open
MacgyverH wants to merge 7 commits into
panoramicdata:mainfrom
MacgyverH:main
Open

Update ContentSerializer Settings to allow data POST back to AutoTask API#3
MacgyverH wants to merge 7 commits into
panoramicdata:mainfrom
MacgyverH:main

Conversation

@MacgyverH

@MacgyverH MacgyverH commented Feb 23, 2026

Copy link
Copy Markdown

While attempting to implement this API Library, I ran into some issues when attempting to submit POST requests back to AutoTask, after some further investigation, I found that null fields were being sent to AutoTask, which was resulting in a 500 Server error.

This change seems to fix this issue.

MacgyverH added 2 commits July 9, 2026 09:12
- Replace System.Text.Json with Newtonsoft.Json for serialization
- Update RefitSettings to use NewtonsoftJsonContentSerializer
- Adjust JSON settings for null value handling and property naming
@MacgyverH MacgyverH reopened this Jul 9, 2026
- Added Refit.Newtonsoft.Json package to AutoTask.Psa.Api.csproj
- Added Refit.Newtonsoft.Json package version to Directory.Packages.props
@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 1 medium

Alerts:
⚠ 1 issue (≤ 0 issues of at least minor severity)

Results:
1 new issue

Category Results
Complexity 1 medium

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

AI Reviewer: run a review on demand. To trigger the first review automatically, go to your organization or repository integration settings. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

MacgyverH added 3 commits July 9, 2026 09:24
- Removed the using directive for Refit.Newtonsoft.Json
- Replace System.Text.Json with Newtonsoft.Json for serialization
- Update RefitSettings to use NewtonsoftJsonContentSerializer
- Adjust JSON settings for null value handling and property naming
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.

1 participant