Skip to content

feat(dashboard): mark newly finished conversations - #1854

Merged
dcramer merged 4 commits into
mainfrom
feat/conversation-finished-indicator
Sep 11, 2026
Merged

feat(dashboard): mark newly finished conversations#1854
dcramer merged 4 commits into
mainfrom
feat/conversation-finished-indicator

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

summary

  • remember the status of conversations visible in the dashboard with local storage
  • show an orange indicator when a visible conversation moves from active to completed
  • clear the indicator when the conversation is opened and prune records outside the current list

checks

  • pnpm --filter @sentry/junior-dashboard exec vitest run tests/conversation-finished-indicators.test.ts
  • pnpm --filter @sentry/junior-dashboard typecheck
  • pnpm --filter @sentry/junior-dashboard lint
  • git diff --check

via David Cramer.

--

View Junior Session [Sentry]

Co-Authored-By: David Cramer <david@sentry.io>
@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
junior-docs Ready Ready Preview Sep 11, 2026 10:26pm UTC

Request Review

Co-Authored-By: David Cramer <david@sentry.io>
@dcramer
dcramer marked this pull request as ready for review September 11, 2026 22:09
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Sep 11, 2026
@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown

5 screenshot changes — 5 changed · 0 added · 0 removed

Review screenshots in Frameshift

Conversation Create Focused · Desktop
Conversation Create Focused · Desktop
Changed
Conversation Create Focused · Mobile
Conversation Create Focused · Mobile
Changed
Conversations · Desktop
Conversations · Desktop
Changed
Conversations · Mobile
Conversations · Mobile
Changed
Memories · Desktop
Memories · Desktop
Changed

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread packages/junior-dashboard/src/client/conversations/ConversationWorkspace.tsx Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 135c7ce. Configure here.

@dcramer
dcramer merged commit ae9284c into main Sep 11, 2026
38 checks passed
@dcramer
dcramer deleted the feat/conversation-finished-indicator branch September 11, 2026 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant