Skip to content

Conversation

@scttcper
Copy link
Member

@scttcper scttcper commented Nov 7, 2024

Combines the components from the two performance issue layouts into the main layout. Our goal is to have fewer custom layouts and turn on and off components based on the issue's properties.

config changes:

  • rename "tags" to "tagsTab"? It seems we hide the tags tab sometimes but not the tags for the event. kinda odd. seen in crons
  • new "tags" that disables highlights and tags
  • add spanEvidence option that can be disabled for these
  • add performanceDurationRegression and profilingDurationRegression which are kinda gross but right now they seem very specific together

example issues:
performance duration regression - https://sentry.sentry.io/issues/5954692777/
profiling duration regression - https://sentry.sentry.io/issues/6023271097/

Combines the two performance issue layouts into the main layout. Our goal is to have fewer layouts specific to issue types and turn on and off components based on the issue's properties.

config changes:
- rename "tags" to "tagsTab"
- new "tags" that disables highlights and tags
- add spanEvidence option that can be disabled for these
- add performanceDurationRegression and profilingDurationRegression which are kinda gross but right now they seem very specific together
@scttcper scttcper requested a review from a team as a code owner November 7, 2024 23:50
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 7, 2024
);
}
}
return hasStreamlinedUI ? (
Copy link
Member

Choose a reason for hiding this comment

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

we love to see it

@codecov
Copy link

codecov bot commented Nov 7, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ils/groupEventDetails/groupEventDetailsContent.tsx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master   #80433   +/-   ##
=======================================
  Coverage   78.33%   78.33%           
=======================================
  Files        7204     7204           
  Lines      318616   318600   -16     
  Branches    43914    43912    -2     
=======================================
- Hits       249589   249584    -5     
+ Misses      62665    62654   -11     
  Partials     6362     6362           

@scttcper scttcper merged commit b9a9d6b into master Nov 8, 2024
44 of 45 checks passed
@scttcper scttcper deleted the scttcper/combine-performance-issue-layouts branch November 8, 2024 18:16
@github-actions github-actions bot locked and limited conversation to collaborators Nov 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants