Add confirmation to destructive actions (clear data, clear weather, remove member, deletes) #39

Open
opened 2026-07-17 03:08:31 +00:00 by claude-bot · 0 comments
Contributor

All destructive actions fire on first click with no confirm and no undo: clearData (MyDataTab.jsx:110-112, trigger :213) wipes imported usage plus all circuit assignments/notes/priorities/exclusions; clearWeather (WeatherTab.jsx:66-68); removeMember (AdminTab.jsx:108-112) revokes a family member instantly; battery/panel/quote/bill deletes (BatteriesTab.jsx:48-54, SolarPanelsTab.jsx:73-82, QuotesTab.jsx:63-71, BillingTab.jsx:49-51) are one-click.

Add a lightweight two-step confirm (button flips to "Confirm clear?" for 3s, or native confirm()) on clearData, clearWeather, and removeMember; apply the same pattern to library deletes if trivial to share.

Acceptance criteria

  • Clear data / clear weather / remove member require a second deliberate action
  • Pattern shared, not copy-pasted per tab

(Audit finding F-19, 2026-07-16 — Severity: Medium, Confidence: High)

All destructive actions fire on first click with no confirm and no undo: `clearData` (`MyDataTab.jsx:110-112`, trigger :213) wipes imported usage plus all circuit assignments/notes/priorities/exclusions; `clearWeather` (`WeatherTab.jsx:66-68`); `removeMember` (`AdminTab.jsx:108-112`) revokes a family member instantly; battery/panel/quote/bill deletes (`BatteriesTab.jsx:48-54`, `SolarPanelsTab.jsx:73-82`, `QuotesTab.jsx:63-71`, `BillingTab.jsx:49-51`) are one-click. Add a lightweight two-step confirm (button flips to "Confirm clear?" for 3s, or native `confirm()`) on clearData, clearWeather, and removeMember; apply the same pattern to library deletes if trivial to share. **Acceptance criteria** - [ ] Clear data / clear weather / remove member require a second deliberate action - [ ] Pattern shared, not copy-pasted per tab *(Audit finding F-19, 2026-07-16 — Severity: Medium, Confidence: High)*
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
rbrooks/BatteryStorageCalculator#39
No description provided.