coracle/caravel
1
1
Fork 2
Code Issues 4 Pull Requests Actions Packages Projects Releases Wiki Activity

feat: add confirmation dialog for relay deactivate/reactivate with explicit warnings #41

Merged
hodlbod merged 1 commits from userAdityaa/caravel:confirmation-dialog into master 2026-04-20 21:25:24 +00:00
Conversation 0 Commits 1 Files Changed 2
+213 -8
userAdityaa commented 2026-04-20 21:21:49 +00:00
Contributor
Copy Link
Copy Source

Problem

Clicking "Deactivate" or "Reactivate" in the relay detail menu immediately triggered the API call with no confirmation step, creating a risk of accidental relay state changes. Users had no visibility into the consequences of deactivation.

Solution

Implemented a reusable confirmation dialog that intercepts relay state-change actions and requires explicit user confirmation before execution. The deactivation flow now displays detailed warnings about the operational impact.

Deactivation Warning Details

When a user clicks "Deactivate", they now see:

Primary message: "{Relay Name} will be taken offline immediately."

Warning details (displayed in an amber warning box):

  • All client connections will be dropped immediately
  • Members will be unable to read from or publish to the relay
  • Scheduled and automated tasks (billing, syncing) will be paused
  • All relay data, settings, and members are preserved, nothing is deleted
  • You can reactivate at any time from this page
Screenshot 2026-04-21 at 3.02.38 AM.png Screenshot 2026-04-21 at 3.04.38 AM.png

closes #30

### Problem Clicking "Deactivate" or "Reactivate" in the relay detail menu immediately triggered the API call with no confirmation step, creating a risk of accidental relay state changes. Users had no visibility into the consequences of deactivation. ### Solution Implemented a reusable confirmation dialog that intercepts relay state-change actions and requires explicit user confirmation before execution. The deactivation flow now displays detailed warnings about the operational impact. ### Deactivation Warning Details When a user clicks "Deactivate", they now see: **Primary message:** "{Relay Name} will be taken offline immediately." **Warning details** (displayed in an amber warning box): - All client connections will be dropped immediately - Members will be unable to read from or publish to the relay - Scheduled and automated tasks (billing, syncing) will be paused - All relay data, settings, and members are preserved, nothing is deleted - You can reactivate at any time from this page <img width="1439" alt="Screenshot 2026-04-21 at 3.02.38 AM.png" src="attachments/a98448e5-d860-4060-922f-2527d6027829"> <img width="1439" alt="Screenshot 2026-04-21 at 3.04.38 AM.png" src="attachments/7e765e18-e81a-485b-a58b-230de397a8e9"> closes #30
Screenshot 2026-04-21 at 3.02.38 AM.png
390 KiB
Screenshot 2026-04-21 at 3.04.38 AM.png
311 KiB
userAdityaa added 1 commit 2026-04-20 21:23:00 +00:00
feat: add confirmation dialog for relay deactivate/reactivate with explicit warnings c60139965b
userAdityaa force-pushed confirmation-dialog from 2c223fed65 to c60139965b 2026-04-20 21:23:00 +00:00 Compare
hodlbod merged commit 38e3a64312 into master 2026-04-20 21:25:24 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
hodlbod (hodlbod)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coracle/caravel#41
Delete Branch "userAdityaa/caravel:confirmation-dialog"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?