Skip to content

ref(investigations): Share the comments page size and stop polling archived comments - #127709

Open
wedamija wants to merge 1 commit into
danf/investigations-block-comments-entryfrom
danf/investigations-comments-followups
Open

wedamija wants to merge 1 commit into
danf/investigations-block-comments-entryfrom
danf/investigations-comments-followups

Conversation

@wedamija

Copy link
Copy Markdown
Member

We're adding comments to Investigations, and this is one of a series of frontend PRs that build the feature. Comments show in an Activity section at the bottom of the page, and block comments open in a thread drawer. Everything is behind the investigations flag.

Follow-ups from review on #127498:

  • One constant sets the comments page size and the "Showing the latest N comments." note, in the Activity list and the drawer.
  • Comment lists don't poll on archived investigations, where comments can't change.

Stacked on #127593.

@wedamija
wedamija requested a review from a team as a code owner October 15, 2026 22:59
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 15, 2026
@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Story previews

Preview the stories changed in this PR on the Vercel deployment:

Preview deployment: https://sentry-cakw5ff4w.sentry.dev

@wedamija
wedamija force-pushed the danf/investigations-comments-followups branch from c8e2e0a to 8d8ca6e Compare October 15, 2026 23:06
@wedamija
wedamija force-pushed the danf/investigations-block-comments-entry branch from 3316dd6 to 8c0b401 Compare October 15, 2026 23:07
@sentry

sentry Bot commented Oct 15, 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 630 0 ✅ Unchanged

⚙️ sentry-frontend Snapshot Settings

…chived comments

One constant sets the comments page size and the "latest N comments" note. Comment lists don't poll on archived investigations, where comments can't change.
@wedamija
wedamija force-pushed the danf/investigations-block-comments-entry branch from 8c0b401 to 9cab10b Compare October 16, 2026 03:28
@wedamija
wedamija force-pushed the danf/investigations-comments-followups branch from 8d8ca6e to 6075146 Compare October 16, 2026 03:28

This branch was successfully deployed

1 active deployment
Preview — 6075146c Deployed Oct 16, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant