feat(scan): add bulk delete functionality for saved scans - #120
feat(scan): add bulk delete functionality for saved scans#120DevOlabode wants to merge 1 commit into
Conversation
Implement a new API endpoint `DELETE /api/scans` to remove all saved scan reports while retaining the account storage. Update the ScanController and storageService to handle the deletion logic, including necessary authentication checks and error handling. Enhance the frontend with a confirmation dialog for users to delete all scans, ensuring a smooth user experience. Update documentation to reflect the new functionality.
ieliofficial
left a comment
There was a problem hiding this comment.
Backend is right, DELETE /api/scans, the storageService half, the tests. Keep all of it.
The frontend I don't. A Delete all button in Account deletes things you can't see from where you're standing. #119 is the shape: checkboxes on the Dashboard list, select all at the top, same confirm. That also gives partial delete, which people ask for more often.
Move the UI to DashboardView and point this at #119. I'll close #110 as covered.
|
Just to confirm based on the recent discussion: are we planning to completely remove the "Delete all scans" functionality on the account page and replace it with the checkbox-based bulk deletion from #119? I want to make sure I understand the intended direction before making any further changes. |
Implement a new API endpoint
DELETE /api/scansto remove all saved scan reports while retaining the account storage. Update the ScanController and storageService to handle the deletion logic, including necessary authentication checks and error handling. Enhance the frontend with a confirmation dialog for users to delete all scans, ensuring a smooth user experience. Update documentation to reflect the new functionality.closes #110