Skip to content

Commit 3604e43

Browse files
authored
ref(app-platform): Use event_id (#13437)
1 parent 2178fc2 commit 3604e43

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/sentry/tasks/sentry_apps.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,21 +69,21 @@ def send_alert_event(event, rule, sentry_app_id):
6969
event_context['url'] = absolute_uri(reverse('sentry-api-0-project-event-details', args=[
7070
project.organization.slug,
7171
project.slug,
72-
event.id,
72+
event.event_id,
7373
]))
7474

7575
if features.has('organizations:sentry10', organization):
7676
event_context['web_url'] = absolute_uri(reverse('sentry-organization-event-detail', args=[
7777
organization.slug,
7878
group.id,
79-
event.id,
79+
event.event_id,
8080
]))
8181
else:
8282
event_context['web_url'] = absolute_uri(reverse('sentry-group-event', args=[
8383
organization.slug,
8484
project.slug,
8585
group.id,
86-
event.id,
86+
event.event_id,
8787
]))
8888

8989
# The URL has a regex OR in it ("|") which means `reverse` cannot generate

tests/sentry/tasks/test_sentry_apps.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,12 +128,12 @@ def test_send_alert_event(self, safe_urlopen):
128128
url=absolute_uri(reverse('sentry-api-0-project-event-details', args=[
129129
self.organization.slug,
130130
self.project.slug,
131-
event.id,
131+
event.event_id,
132132
])),
133133
web_url=absolute_uri(reverse('sentry-organization-event-detail', args=[
134134
self.organization.slug,
135135
group.id,
136-
event.id,
136+
event.event_id,
137137
])),
138138
issue_url=absolute_uri(
139139
'/api/0/issues/{}/'.format(group.id),

0 commit comments

Comments
 (0)