Skip to content

chore(issues): Remove old reference to performance category when building alert message #103580

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.

Sign up for GitHub

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
+0 −5
Open

chore(issues): Remove old reference to performance category when building alert message #103580

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions src/sentry/integrations/messaging/message_builder.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
from sentry import features
from sentry.integrations.messaging.types import LEVEL_TO_COLOR
from sentry.integrations.types import EXTERNAL_PROVIDERS, ExternalProviders
from sentry.issues.grouptype import GroupCategory
from sentry.models.environment import Environment
from sentry.models.group import Group
from sentry.models.organization import Organization
Expand Down Expand Up @@ -333,9 +332,5 @@ def get_color(
color = event_for_tags.occurrence.level
if color and color in LEVEL_TO_COLOR.keys():
return color
if group.issue_category == GroupCategory.PERFORMANCE:
# XXX(CEO): this shouldn't be needed long term, but due to a race condition
# the group's latest event is not found and we end up with no event_for_tags here for perf issues
return "info"

return "error"
Loading