Skip to content

CLOSE_ISSUE.md

Latest commit

 

History

History
29 lines (22 loc) · 1.02 KB

CLOSE_ISSUE.md

File metadata and controls

29 lines (22 loc) · 1.02 KB
Raw

Issue #19797 Already Resolved

Summary

All destructive actions mentioned in issue #19797 already have confirmation dialogs properly implemented.

Evidence

UnifiedDashboard Reset (line 336 reference)

  • Button at line 490: onClick={handleResetRequest}
  • Handler shows confirmation: line 312-314
  • Confirmation dialog: lines 607-616
  • Only calls destructive handleResetConfirmed() after user confirms

Card Removal

  • Handler: handleRemoveCard() line 234-238
  • Confirmation dialog: lines 618-632
  • Only removes card after user confirms in handleRemoveCardConfirmed()

Logout (auth.tsx lines 217,220,etc)

  • User-facing logout button in UserProfileDropdown.tsx
  • Confirmation dialog: lines 531-533
  • The auth.tsx lines are cleanup code INSIDE the logout handler (not user-facing buttons)

Previous Fixes

  • Commit #19383: Added confirmation dialog for card removal
  • Commit #19388: Updated tests for card removal confirmation dialog

Conclusion

This PR closes #19797 as the issue has been resolved in previous commits.