chore(react): cleanup outstanding TODOs on @sentry/react #2661
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Following up #2659
Motivation
As I'm getting ready to test with getsentry, I went ahead and made a PR to cleanup and review all the react integration components.
Didn't think these small changes required a CHANGELOG entry, but please let me know if they do.
The commits
feat(react): Add Event Processor for React
0e2bf38addGlobalEventProcessor
that changes the sdk name tosentry.javascript.react
chore(react): Update comments and test for Profiler
aeb266ffix(react): Correctly give eventID to dialog
3fca5b0sentry.captureException
is given to the report dialogfix(react): Prevent state updates from creating activities
23a96b5useState
as otherwise we create new transactions every render. Previously if we updated a child component (like clicking a button to increment a counter), we get this:Future
Test with getsentry. Also need to add a proper README, but will do that after getsentry testing.