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

Improve checks determining the JSCallResultType for JS interop calls. #25624

Closed
MackinnonBuck opened this issue Sep 4, 2020 · 0 comments
Closed

Improve checks determining the JSCallResultType for JS interop calls. #25624

MackinnonBuck opened this issue Sep 4, 2020 · 0 comments
Assignees
@MackinnonBuck
Labels
area-blazor Includes: Blazor, Razor Components Done This issue has been fixed enhancement This issue represents an ask for new feature or an enhancement to an existing one
Milestone
5.0.0-rc2

Comments

@MackinnonBuck
Copy link
Member

Summary

Currently, trying to create an IJSObjectReference instance by calling IJSRuntime.InvokeAsync<JSObjectReference>(...) instead of IJSRuntime.InvokeAsync<IJSObjectReference>(...) results in a JSON deserialization error. This is by design, but the error could be caught earlier and a more useful error message could be provided.

Additional context

@SteveSandersonMS had a suggestion for this here.
@MackinnonBuck MackinnonBuck added enhancement This issue represents an ask for new feature or an enhancement to an existing one area-blazor Includes: Blazor, Razor Components labels Sep 4, 2020
@MackinnonBuck MackinnonBuck added this to the 5.0.0-rc2 milestone Sep 4, 2020
@MackinnonBuck MackinnonBuck self-assigned this Sep 4, 2020
@MackinnonBuck MackinnonBuck mentioned this issue Sep 4, 2020
Merged
@ghost ghost added Done This issue has been fixed and removed Working labels Sep 9, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Oct 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@MackinnonBuck MackinnonBuck

Labels
area-blazor Includes: Blazor, Razor Components 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
Milestone
5.0.0-rc2
Development

No branches or pull requests
2 participants
@MackinnonBuck @mkArtakMSFT