Skip to content

Update Razor Components and JSInterop to use System.Text.Json #8205

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.

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

Closed
rynowak opened this issue Mar 5, 2019 · 3 comments
Closed

Update Razor Components and JSInterop to use System.Text.Json #8205

rynowak opened this issue Mar 5, 2019 · 3 comments
Assignees
Labels
area-blazor Includes: Blazor, Razor Components Components Big Rock This issue tracks a big effort which can span multiple issues Done This issue has been fixed enhancement This issue represents an ask for new feature or an enhancement to an existing one

Comments

@rynowak
Copy link
Member

rynowak commented Mar 5, 2019

We need to integrate with System.Text.Json and replace our use of SimpleJSON. We don't have to expose the System.Text.Json APIs, but we need to have a default serializer to use with JS Interop.

Since we don't want to use JSON.NET in Blazor (code size), and we don't want to have breaking changes, using System.Text.Json is the most logical choice.

@rynowak rynowak added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Mar 5, 2019
@rynowak rynowak mentioned this issue Mar 5, 2019
56 tasks
@rynowak rynowak added the Components Big Rock This issue tracks a big effort which can span multiple issues label Mar 5, 2019
@mkArtakMSFT mkArtakMSFT added this to the 3.0.0-preview5 milestone Mar 5, 2019
@pranavkm
Copy link
Contributor

pranavkm commented Mar 5, 2019

@rynowak
Copy link
Member Author

rynowak commented Mar 5, 2019

external > internal

@mkArtakMSFT mkArtakMSFT added the area-blazor Includes: Blazor, Razor Components label Mar 6, 2019
@pranavkm pranavkm added the blocked The work on this issue is blocked due to some dependency label Mar 26, 2019
@mkArtakMSFT
Copy link
Member

Moving this to preview5 as the required .Net Core feature is going to land then.

@danroth27 danroth27 added enhancement This issue represents an ask for new feature or an enhancement to an existing one and removed task labels Apr 25, 2019
@mkArtakMSFT mkArtakMSFT removed area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates labels May 9, 2019
@mkArtakMSFT mkArtakMSFT added Working Done This issue has been fixed and removed 1 - Ready blocked The work on this issue is blocked due to some dependency labels May 29, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-blazor Includes: Blazor, Razor Components Components Big Rock This issue tracks a big effort which can span multiple issues Done This issue has been fixed enhancement This issue represents an ask for new feature or an enhancement to an existing one
Projects
None yet
Development

No branches or pull requests

4 participants