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.

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

Small fix that fixes map drawing on Server side Blazor. #23

Merged
merged 4 commits into from
Dec 18, 2023

Conversation

vrenken
Copy link
Contributor

@vrenken vrenken commented Nov 24, 2023

Hi,

please find in this PR a small fix that enables the drawing capabilities on Blazor Server hosting model.
Nothing too fancy, just ensuring that the JS interop from the client to the .NET side is done asynchronously, as synchronous calls are not permitted for Blazor Server.

Thanks in advance for creating and maintaining this library & greetings from Germany,

Peter Vrenken

@lolochristen
Copy link
Owner

Thanks! You're right about the async Blazor Server behavior. I'll test it and merge it asap.

@lolochristen
Copy link
Owner

Please fix the call to invokeMethodAsync. With this fix we are good to go.
Thanks, lolo

@vrenken
Copy link
Contributor Author

vrenken commented Dec 11, 2023

Fixed call to use InvokeMethodAsync.

@vrenken vrenken closed this Dec 11, 2023
@vrenken vrenken reopened this Dec 11, 2023
@lolochristen lolochristen merged commit ea13071 into lolochristen:main Dec 18, 2023
2 of 4 checks passed
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.

2 participants