Skip to content

fix(Replay): Use Sentry Prefix for Project_id#104296

Merged
cliffordxing merged 1 commit intomasterfrom
project-id-column
Dec 3, 2025
Merged

fix(Replay): Use Sentry Prefix for Project_id#104296
cliffordxing merged 1 commit intomasterfrom
project-id-column

Conversation

@cliffordxing
Copy link
Copy Markdown
Contributor

As title says, this PR uses sentry prefix to query for project_id (sentry.project_id), as it was causing the following issue:
image

The reason this is causing an issue is because project_id itself is not queryable as an attribute, since it is a top level protobuf field, which causes the following line to be null on the frontend:

return projects.projects.find(p => p.id === replayRecord.project_id)?.slug ?? null;

Thus, we change our query to use the sentry prefix to receive the proper data. The reason this was not caught in a unit test previously is because we were manually adding the project_id in the attributes dummy data, allowing it to be queried. To test this new change, we remove the attribute and see that it still works as expected.

@cliffordxing cliffordxing requested a review from a team as a code owner December 2, 2025 23:34
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 2, 2025
@cliffordxing cliffordxing merged commit 3aa274e into master Dec 3, 2025
68 checks passed
@cliffordxing cliffordxing deleted the project-id-column branch December 3, 2025 00:11
@github-actions github-actions bot locked and limited conversation to collaborators Dec 18, 2025
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.

2 participants