Add functionality to delete diagnostic reports

Resolved 💬 1 comment Opened Jul 5, 2025 by jimjones26 Closed Jul 5, 2025

Feature Request

Description

Add the ability to delete diagnostic reports from the system. Currently, there is no way to remove reports once they are created.

Requirements

  1. Backend API Endpoint
  • Add DELETE endpoint at /api/v1/diagnostic-reports/{report_id}
  • Should delete the report and all associated scraped data
  • Should check user permissions (admin only or owner of the prospect)
  • Should handle cascade deletion of related entities
  1. Frontend UI
  • Add delete button on diagnostic report detail page
  • Add confirmation dialog before deletion
  • Show appropriate success/error messages
  • Redirect to reports list after successful deletion
  1. Database Considerations
  • Ensure proper cleanup of relationships in Neo4j
  • Delete associated ScrapedData nodes
  • Update prospect status if needed
  • Consider soft delete vs hard delete

Implementation Notes

  • Consider adding a "trash" feature for soft deletes
  • May want to add bulk delete functionality for multiple reports
  • Should log deletion events for audit trail
  • Consider data retention policies

Related Files

  • Backend: /backend/app/api/v1/endpoints/diagnostic_reports.py
  • Frontend: /frontend/src/routes/(app)/diagnostic-reports/[id]/+page.svelte
  • Repository: /backend/app/repositories/diagnostic_report.py

Priority

Medium - Nice to have for data management and cleanup

View original on GitHub ↗

This issue has 1 comment on GitHub. Read the full discussion on GitHub ↗