Skip to content

Commit e84ab23

Browse files
aliu39shashjar
authored andcommitted
ref(seer-explorer): add user id to initial context (#102170)
Needed for getsentry/seer#3812 TODO: figure out how to handle user id = None for anonymous users
1 parent 0831f29 commit e84ab23

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

src/sentry/seer/endpoints/organization_seer_explorer_chat.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ def _collect_user_org_context(request: Request, organization: Organization) -> d
4040
if isinstance(user, AnonymousUser):
4141
return {
4242
"org_slug": organization.slug,
43+
"user_id": None,
4344
"user_name": None,
4445
"user_email": None,
4546
"user_teams": [],
@@ -62,6 +63,7 @@ def _collect_user_org_context(request: Request, organization: Organization) -> d
6263

6364
return {
6465
"org_slug": organization.slug,
66+
"user_id": user.id,
6567
"user_name": user.name,
6668
"user_email": user.email,
6769
"user_teams": user_teams,

tests/sentry/seer/endpoints/test_organization_seer_explorer_chat.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,7 @@ def test_post_with_query_calls_seer(
9191
):
9292
mock_context = {
9393
"org_slug": self.organization.slug,
94+
"user_id": self.user.id,
9495
"user_name": self.user.name,
9596
"user_email": self.user.email,
9697
"user_teams": [],
@@ -286,6 +287,7 @@ def test_post_with_both_feature_flags_succeeds(
286287
):
287288
mock_context = {
288289
"org_slug": self.organization.slug,
290+
"user_id": self.user.id,
289291
"user_name": self.user.name,
290292
"user_email": self.user.email,
291293
"user_teams": [],
@@ -336,6 +338,7 @@ def test_collect_context_with_member(self):
336338

337339
assert context is not None
338340
assert context["org_slug"] == self.organization.slug
341+
assert context["user_id"] == self.user.id
339342
assert context["user_name"] == self.user.name
340343
assert context["user_email"] == self.user.email
341344

0 commit comments

Comments
 (0)