Skip to content

ref(action-log): Restore GALE row IDs in activity responses - #124835

Open
shashjar wants to merge 1 commit into
shashjar/use-comment-references-for-activity-mutationsfrom
shashjar/restore-GALE-row-IDs-in-activity-responses
Open

shashjar wants to merge 1 commit into
shashjar/use-comment-references-for-activity-mutationsfrom
shashjar/restore-GALE-row-IDs-in-activity-responses

Conversation

@shashjar

@shashjar shashjar commented Sep 17, 2026

Copy link
Copy Markdown
Member

Refs https://linear.app/getsentry/issue/ISWF-3427/serialize-a-dedicated-comment-reference-field-instead-of-overloading.

PR 3/4.

GALE-backed activity responses now serialize each entry's own id, including comments. Comment edit/delete operations continue using the separate commentId reference introduced in PR 1 and adopted by the frontend in PR 2.

Removes the ID compatibility logic, updates stale comments, and consolidates overlapping tests.

@shashjar
shashjar added this pull request to stack #124829 September 17, 2026 21:14
@github-actions github-actions Bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Sep 17, 2026
@shashjar
shashjar force-pushed the shashjar/restore-GALE-row-IDs-in-activity-responses branch from 05908ed to ad8ab76 Compare September 17, 2026 21:15
@github-actions github-actions Bot removed the Scope: Frontend Automatically applied to PRs that change frontend components label Sep 17, 2026
@shashjar shashjar changed the title Expose a dedicated comment reference field on GALEs ref(action-log): Restore GALE row IDs in activity responses Sep 17, 2026
@linear-code

linear-code Bot commented Sep 17, 2026

Copy link
Copy Markdown

ISWF-3427

@sentry

sentry Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Sentry Snapshot Testing

Name Added Removed Changed Renamed Unchanged Skipped Status
sentry-frontend
sentry-frontend
0 0 0 0 503 0 ✅ Unchanged

⚙️ sentry-frontend Snapshot Settings

@shashjar

Copy link
Copy Markdown
Member Author

bugbot review

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit ad8ab76. Configure here.

@shashjar
shashjar requested review from cvxluo and kcons September 17, 2026 21:21
@shashjar
shashjar marked this pull request as ready for review September 17, 2026 21:21
@shashjar
shashjar requested review from a team as code owners September 17, 2026 21:21
@shashjar
shashjar removed request for a team September 17, 2026 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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