Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Port OSLC4Net.Core.JsonProvider to use System.Text.Json #108

Open
berezovskyi opened this issue Oct 24, 2023 · 1 comment
Open

Port OSLC4Net.Core.JsonProvider to use System.Text.Json #108

berezovskyi opened this issue Oct 24, 2023 · 1 comment
Labels
prio:low

Comments

@berezovskyi
Copy link
Member

berezovskyi commented Oct 24, 2023 •
edited
Loading

OSLC4Net.Core.JsonProvider currently relies on System.Json as its main API (despite including the NuGet dependency on Newtonsoft.Json, which doesn't seem to have been used). System.Json was replaced with System.Text.Json as the recommended JSON processing approach.

The legacy System.Json namespace is not available in netstandard2.0. It should be possible to drag the legacy dependency via https://www.nuget.org/packages/System.Json#supportedframeworks-body-tab into netstandard2.0, but I was unable to get it working in #98 and removed OSLC4Net.Core.JsonProvider from the build (#109 re-adds it to the SLN).
@berezovskyi
Copy link
Member Author

Most likely the package will be deprecated and removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
prio:low
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@berezovskyi