Skip to content

Add confirmation dialogs for start and rotate actions#75

Merged
huseyinbabal merged 1 commit intomasterfrom
feature/start-instance-confirmation
Jan 14, 2026
Merged

Add confirmation dialogs for start and rotate actions#75
huseyinbabal merged 1 commit intomasterfrom
feature/start-instance-confirmation

Conversation

@huseyinbabal
Copy link
Owner

Summary

  • Add confirmation modal to EC2 start instance action (s key)
  • Add confirmation modal to RDS start instance action (s key)
  • Add confirmation modal to Secrets Manager rotate secret action (R key)

Related Issue

Closes #74

Rationale

The user raised a valid concern: pressing s (start) accidentally instead of d (details) could start instances and incur unexpected AWS charges. Other state-changing actions like stop (S) and reboot (r) already require confirmation.

This PR adds confirmation dialogs to all remaining write/modify operations that were missing them:

Service Action Key Before After
EC2 Start Instance s No confirmation ✅ Confirmation
RDS Start Instance s No confirmation ✅ Confirmation
Secrets Manager Rotate Secret R No confirmation ✅ Confirmation

Type of Change

  • Bug fix (non-breaking change that fixes an issue)

Checklist

  • I have linked this PR to an existing issue
  • My code follows the project's code style
  • I have tested my changes locally
  • My changes don't introduce new warnings

- Add confirmation modal to EC2 start instance action (s key)
- Add confirmation modal to RDS start instance action (s key)
- Add confirmation modal to Secrets Manager rotate secret action (R key)

These actions can incur AWS charges or modify state, so they should
require user confirmation like other similar actions (stop, reboot, etc.)

Closes #74
@huseyinbabal huseyinbabal force-pushed the feature/start-instance-confirmation branch from e657608 to ff0e07c Compare January 14, 2026 22:35
@huseyinbabal huseyinbabal merged commit 8ea5e93 into master Jan 14, 2026
@huseyinbabal huseyinbabal deleted the feature/start-instance-confirmation branch January 14, 2026 22:42
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.

\<s> should require confirmation

1 participant