Skip to content

fix: PromptManager stage save/create/delete give no visible confirmation - #6108

Merged
atomantic merged 1 commit into
atomantic:mainfrom
Bryandero98:fix/promptmanager-stage-toasts-and-select
Sep 3, 2026
Merged

fix: PromptManager stage save/create/delete give no visible confirmation#6108
atomantic merged 1 commit into
atomantic:mainfrom
Bryandero98:fix/promptmanager-stage-toasts-and-select

Conversation

@Bryandero98

Copy link
Copy Markdown
Contributor

Summary

  • saveStage, createStage, and confirmDeleteStage in PromptManager.jsx mutated the stage and silently reloaded — no toast ever confirmed the save/create/delete happened.
  • createStage also never selected the stage it just created, so the user was left looking at whatever stage was selected before, with no visible confirmation their new stage exists.
  • All three now call toast.success(...); createStage additionally calls setSelectedStage(payload.stageName) so the URL and editor deep-link to the stage just made.

Test plan

  • client/src/pages/PromptManager.test.jsx — 53/53, including a new describe block covering all three actions' feedback

🤖 Generated with Claude Code

…ion (atomantic#6022)

saveStage, createStage, and confirmDeleteStage all mutated the stage and
silently reloaded, with no toast telling the user the save/create/delete
actually happened. createStage also never deep-linked to the stage it just
created, leaving the user looking at whatever stage happened to be selected
before. All three now toast success, and createStage selects the new stage
by name so the URL and the visible editor point at what was just made.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

@atomantic atomantic left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Reviewed — change is correct, focused, and covered by tests. Approving.

@atomantic
atomantic merged commit 1fd0ab3 into atomantic:main Sep 3, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants