Skip to content

Update delete session dialog for local sessions #24

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 26, 2025
Merged

Update delete session dialog for local sessions #24

merged 1 commit into from
Oct 26, 2025

Conversation

@built-by-as
Copy link
Owner

@built-by-as built-by-as commented Oct 25, 2025

Summary

  • Show different confirmation messages when deleting sessions based on session type
  • Worktree sessions: "Delete {name}? This will remove the git worktree."
  • Local sessions: "Delete {name}? This will remove the session."

Test plan

  • Create a local session and verify the delete dialog shows "This will remove the session."
  • Create a worktree session and verify the delete dialog shows "This will remove the git worktree."

🤖 Generated with Claude Code

@built-by-as @claude
Update delete session dialog to differentiate local and worktree sess…
c1e370b 
…ions

Show different confirmation messages when deleting sessions based on their type:
- Worktree sessions: mentions removing the git worktree
- Local sessions: only mentions removing the session

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@built-by-as built-by-as force-pushed the delete-session-dialog branch from 80b8f4f to c1e370b Compare October 25, 2025 23:57
@built-by-as built-by-as merged commit e0e64d8 into main Oct 26, 2025
1 check passed
@built-by-as built-by-as deleted the delete-session-dialog branch October 26, 2025 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@built-by-as