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

Consider using a JsonConverter to parse "FieldValue" #11517

Closed
pranavkm opened this issue Jun 24, 2019 · 2 comments
Closed

Consider using a JsonConverter to parse "FieldValue" #11517

pranavkm opened this issue Jun 24, 2019 · 2 comments
Labels
area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one
Milestone
Backlog

Comments

@pranavkm
Copy link
Contributor

FieldValue is a discriminated union type currently represented as a object \ JsonObject. We could use a JsonConverter to read the value.
@mkArtakMSFT mkArtakMSFT added the area-blazor Includes: Blazor, Razor Components label Jun 24, 2019
@pranavkm pranavkm mentioned this issue Jun 24, 2019
Merged
@mkArtakMSFT mkArtakMSFT added this to the Backlog milestone Jun 25, 2019
@mkArtakMSFT mkArtakMSFT added the enhancement This issue represents an ask for new feature or an enhancement to an existing one label Jun 25, 2019
@pranavkm
Copy link
Contributor Author

We couldn't do this post 3.0 since it would be an API breaking change. Should just close in that case.

@pranavkm
Copy link
Contributor Author

Closing this as it's not clear if there's any value in doing this.

@ghost ghost locked as resolved and limited conversation to collaborators Jan 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
2 participants
@pranavkm @mkArtakMSFT