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

[react] Fix grouping by SE "supertag" #417

Merged
merged 4 commits into from
Mar 20, 2024
Merged

Conversation

realkosty
Copy link
Collaborator

@realkosty realkosty commented Mar 12, 2024

After TDA started using the new SE "supertag" format (#332) we're having multiple issues for same flagship error, 1 for each browser/platform combination. This fixes it.

Testing

Note

Tested together with 2 other PRs: #422 and #423

./deploy.sh --env=local react
example event

./deploy.sh --env=staging react
https://staging-application-monitoring-react-dot-sales-engineering-sf.appspot.com/products?se=kosty_node_18_webpack_se&backend=rails
Test error
Test transaction
Test error with 'prod-tda-direct-*' se tag to test grouping signature

@realkosty realkosty requested a review from cstavitsky March 12, 2024 17:11
@realkosty realkosty requested a review from ndmanvar March 19, 2024 18:12
@realkosty realkosty force-pushed the react-fix-se-grouping branch from dc365fa to 4fa52ea Compare March 19, 2024 18:13
@realkosty realkosty changed the base branch from master to react-fix-sourcemap-double-upload March 19, 2024 18:13
@realkosty realkosty force-pushed the react-fix-sourcemap-double-upload branch from 0e2961e to e71a936 Compare March 19, 2024 23:01
@realkosty realkosty force-pushed the react-fix-se-grouping branch from 4fa52ea to 252535f Compare March 19, 2024 23:02
react/src/index.js Outdated Show resolved Hide resolved
Copy link
Contributor

@cstavitsky cstavitsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved but see minor suggestion

Base automatically changed from react-fix-sourcemap-double-upload to master March 20, 2024 00:08
@realkosty realkosty merged commit 03afb75 into master Mar 20, 2024
1 check failed
@realkosty realkosty deleted the react-fix-se-grouping branch March 20, 2024 00:10
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