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: check browser profiling organization flag on react render #58781

Merged
merged 3 commits into from
Oct 26, 2023

Conversation

JonasBa
Copy link
Member

@JonasBa JonasBa commented Oct 25, 2023

No description provided.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 25, 2023
@JonasBa
Copy link
Member Author

JonasBa commented Oct 25, 2023

Good eye @markstory

@codecov
Copy link

codecov bot commented Oct 25, 2023

Codecov Report

Merging #58781 (d682142) into master (aa0b334) will increase coverage by 0.00%.
Report is 31 commits behind head on master.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##           master   #58781   +/-   ##
=======================================
  Coverage   80.60%   80.60%           
=======================================
  Files        5150     5151    +1     
  Lines      225444   225477   +33     
  Branches    37983    37990    +7     
=======================================
+ Hits       181717   181751   +34     
  Misses      38175    38175           
+ Partials     5552     5551    -1     
Files Coverage Δ
src/sentry/web/frontend/react_page.py 93.24% <66.66%> (+0.38%) ⬆️

... and 17 files with indirect coverage changes

@JonasBa JonasBa merged commit a553297 into master Oct 26, 2023
48 of 49 checks passed
@JonasBa JonasBa deleted the jb/profiling/browser-flag-handler-check branch October 26, 2023 13:21
@github-actions github-actions bot locked and limited conversation to collaborators Nov 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants