Skip to content

add send now button#11880

Merged
harryalbert merged 2 commits into
masterfrom
harry/app-4625-add-a-send-now-action-to-queued-prompts
Jun 2, 2026
Merged

add send now button#11880
harryalbert merged 2 commits into
masterfrom
harry/app-4625-add-a-send-now-action-to-queued-prompts

Conversation

@harryalbert

@harryalbert harryalbert commented May 29, 2026

Copy link
Copy Markdown
Contributor

Description

This PR adds a "send now" button to the list of action buttons for queued prompts so that users can immediately send a queued prompt if they want to. This button is disabled for all queued prompts during cloud environment setup, as no prompts can be sent at that time.

Testing

  • I have manually tested my changes locally with ./script/run

Screenshots / Videos

https://www.loom.com/share/cdc86b2ea7d447bfb0275b3cbc55c8bb

Agent Mode

  • Warp Agent Mode - This PR was created via Warp's AI Agent Mode

@cla-bot cla-bot Bot added the cla-signed label May 29, 2026

harryalbert commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

@oz-for-oss

oz-for-oss Bot commented May 29, 2026

Copy link
Copy Markdown
Contributor

@harryalbert

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@oz-for-oss oz-for-oss 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.

Overview

This PR adds a send-now action to queued prompt rows, routes the new panel event through the input submission path, disables send-now while the initial cloud setup prompt is blocking submissions, and adds focused unit coverage for immediate submission and disabled-state behavior.

Concerns

  • No blocking correctness, security, or spec-alignment concerns found in the annotated diff.
  • Visual evidence is present in the PR description via the Loom link.
  • No approved or repository spec context was available, so there were no spec commitments to compare beyond the PR description.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@harryalbert harryalbert changed the base branch from harry/app-4594-replace-queued-prompt-block-with-new-queued-ui-for-compact to graphite-base/11880 May 29, 2026 20:38
@harryalbert harryalbert force-pushed the graphite-base/11880 branch from c653f13 to fb8d00b Compare June 1, 2026 22:07
@harryalbert harryalbert force-pushed the harry/app-4625-add-a-send-now-action-to-queued-prompts branch from 21a8a65 to c8873ea Compare June 1, 2026 22:07
@harryalbert harryalbert changed the base branch from graphite-base/11880 to master June 1, 2026 22:07
@harryalbert harryalbert force-pushed the harry/app-4625-add-a-send-now-action-to-queued-prompts branch from c8873ea to 39157ea Compare June 2, 2026 01:47
@harryalbert harryalbert force-pushed the harry/app-4625-add-a-send-now-action-to-queued-prompts branch from 39157ea to e18ec6d Compare June 2, 2026 02:01
@harryalbert harryalbert merged commit 1aa03f9 into master Jun 2, 2026
26 checks passed
@harryalbert harryalbert deleted the harry/app-4625-add-a-send-now-action-to-queued-prompts branch June 2, 2026 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants