Skip to content

fix(dashboard): Remove queued messages individually - #1554

Merged
dcramer merged 3 commits into
mainfrom
fix/per-message-queue-cancel
Aug 14, 2026
Merged

fix(dashboard): Remove queued messages individually#1554
dcramer merged 3 commits into
mainfrom
fix/per-message-queue-cancel

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Replace the queue-wide cancel action with an X control on each accepted message. Each control removes only its message on mobile and desktop. Keep other remove controls disabled while one request is pending, and keep the composer available.

Tests cover accepted, sending, failed, and collapsed queue states.

Requested by David Cramer.

--

View Junior Session [Sentry]

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

vercel Bot commented Aug 14, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
junior-docs Ready Ready Preview Aug 14, 2026 3:06am

Request Review

@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown

Dashboard visual evidence

Mode: path-selected
Selected: conversations, conversation-detail

Triggered by:

  • packages/junior-dashboard/e2e/conversations.spec.ts
  • packages/junior-dashboard/src/client/conversations/ConversationPage.tsx
  • packages/junior-dashboard/src/client/conversations/PendingMailboxStack.tsx
  • packages/junior-dashboard/tests/pending-mailbox-stack.test.tsx

Conversations · desktop

Conversations · desktop

Conversations · mobile

Conversations · mobile

Conversation detail · desktop

Conversation detail · desktop

Conversation detail · mobile

Conversation detail · mobile

Full-page screenshots from the mock dashboard. Not a pixel-diff gate.

@blacksmith-sh

This comment has been minimized.

Match the collapsed expand control to the total queue count after
removing the cancel-queue bar.
@dcramer
dcramer marked this pull request as ready for review August 14, 2026 02:59
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Aug 14, 2026

@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 2 potential issues.

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 ce896a1. Configure here.

Show an inline remove failure on the target row so mobile can see it
without tooltips, and keep in-flight remove state when send starts.
@dcramer
dcramer merged commit 3cc7056 into main Aug 14, 2026
34 checks passed
@dcramer
dcramer deleted the fix/per-message-queue-cancel branch August 14, 2026 03:18
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