Skip to content

Sample Blazor Wasm - #574

Merged
bruno-garcia merged 6 commits into
mainfrom
sample/blazor-wasm
Nov 24, 2020
Merged

bruno-garcia merged 6 commits into
mainfrom
sample/blazor-wasm

Conversation

@bruno-garcia

Copy link
Copy Markdown
Member

image

@bruno-garcia
bruno-garcia requested a review from Tyrrrz November 5, 2020 03:03
@bruno-garcia

Copy link
Copy Markdown
Member Author

Fails on Windows only, odd

@Tyrrrz

Tyrrrz commented Nov 5, 2020

Copy link
Copy Markdown
Contributor

No changes required after all?

@Tyrrrz

Tyrrrz commented Nov 6, 2020

Copy link
Copy Markdown
Contributor

No idea what fails the build

@DominikPf

Copy link
Copy Markdown

@bruno-garcia Hi, i just tried to run your sample project.
But unfortunately it doesn't report the Events to Sentry.
The unhandled Exception ist displayed in the Browser Console. (Blazor Default)
But no Network request is sent.

If i try to send a Message manually it works (SentrySdk.CaptureMessage)

Is additional configuration needed?

Tested with Sentry (2.1.8, 3.0.0-alpha.5 & your codebase from branch "sample/blazor-wasm")

@bruno-garcia

Copy link
Copy Markdown
Member Author

@DominikPf what browser did you tried? I'm afraid Threads in Wasm is still not generally supported

@bruno-garcia

Copy link
Copy Markdown
Member Author

@DominikPf I just tested it (bumped to net5.0) and it worked: https://twitter.com/brungarc/status/1331056131624591362

The MEL integration isnt' capturing breadcrumbs, so can be improved though

@bruno-garcia
bruno-garcia merged commit 7e6d797 into main Nov 24, 2020
@bruno-garcia
bruno-garcia deleted the sample/blazor-wasm branch November 24, 2020 02:43
@DominikPf

Copy link
Copy Markdown

@bruno-garcia The updated Version with net5 works like a charm.
Thank you 👍

@bruno-garcia

Copy link
Copy Markdown
Member Author

@DominikPf curious about your experience with it. Let us know how it goes. Thanks

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.

3 participants