feat(hackathons): Add delete functionality with confirmation dialog and enhanced logging

[codeunia-dev]

• Add delete button to hackathon management table with trash icon
• Implement confirmation dialog for hackathon deletion using AlertDialog component
• Add comprehensive logging throughout DELETE endpoint for debugging
• Enhance error handling with auth error checking and proper HTTP status codes (403 for unauthorized)
• Update DELETE response format to include success flag
• Add error message details in 500 response for better debugging
• Include tooltips on action buttons (Edit, View, Delete) for better UX
• Add delete state management (deleteDialogOpen, hackathonToDelete, isDeleting)
• Implement handleDeleteClick and handleDeleteConfirm functions for deletion workflow
• Use hackathon slug instead of id for API endpoint consistency
• Add success and error toast notifications for user feedback
• Refresh hackathon list after successful deletion

Authored by: @akshay0611

Summary by CodeRabbit

Release Notes

New Features

• Added the ability to delete hackathons with a confirmation dialog before removal
• Enhanced error handling and validation to provide clearer feedback during deletion operations

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
codeunia	Building	Preview	Comment	Nov 17, 2025 9:52am

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

The pull request enhances the hackathon deletion functionality across the API route, dashboard UI, and service layer. It introduces comprehensive logging, explicit authentication and authorization validation with appropriate HTTP status codes (401/403), a delete confirmation dialog in the UI, and improved error handling with detailed logging at each step of the deletion flow.

Changes

Cohort / File(s)	Summary
API Route & Service Layer app/api/hackathons/[id]/route.ts, lib/services/hackathons.ts	Added debug logging throughout DELETE flow. Enhanced authentication validation with authError checking, returning 401 if missing. Changed authorization failure to 403 with logging. Added hackathon existence check returning 404 if not found. Modified response payload to include { success: true, message: ... } structure. Service layer now retrieves deleted row data, logs success/warning states, and throws detailed error messages.
Dashboard UI app/dashboard/company/[slug]/hackathons/page.tsx	Introduced delete state management (deleteDialogOpen, hackathonToDelete, isDeleting). Added Trash2 delete button per row with titles on edit/view actions. Implemented handleDeleteClick for dialog opening and handleDeleteConfirm for DELETE API call with error handling and toast notifications. Added AlertDialog confirmation component with dynamic content and deletion-state-controlled disablement.

Sequence Diagram

sequenceDiagram
    actor User
    participant Dashboard
    participant API Route
    participant Service
    participant Database

    User->>Dashboard: Click delete button
    Dashboard->>Dashboard: Show confirmation dialog
    User->>Dashboard: Confirm deletion
    Dashboard->>API Route: DELETE /api/hackathons/[id]
    
    rect rgb(200, 220, 255)
    Note over API Route: Authentication & Authorization
    API Route->>API Route: Check authentication
    alt No user/authError
        API Route-->>Dashboard: 401 Unauthorized
    else Check authorization (admin/member)
        alt Not authorized
            API Route-->>Dashboard: 403 Forbidden
        else Authorized
            rect rgb(220, 255, 220)
            Note over API Route, Service: Delete Flow
            API Route->>Service: deleteHackathon(slug)
            Service->>Database: DELETE from hackathons WHERE slug=?
            Database-->>Service: Deleted row(s)
            
            alt No rows deleted
                Service-->>API Route: Error (not found/forbidden)
            else Success
                Service->>Service: Log deletion confirmation
                Service-->>API Route: Return deleted data
            end
            end
            
            API Route-->>Dashboard: 200 { success: true, message: ... }
        end
    end
    end
    
    Dashboard->>Dashboard: Close dialog, refresh list, show toast

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• API route changes: Review authentication/authorization validation logic, status code assignments (401/403/404), and response payload structure
• Service layer changes: Verify database query changes with .select(), error handling logic for "no rows deleted" guard, and logging placement
• Dashboard component: Validate delete state management, async handling during deletion, error/success toast flows, and dialog state synchronization

Possibly related PRs

• feat(hackathons): Add comprehensive hackathon management and moderation system #324: Directly modifies the same files (app/api/hackathons/[id]/route.ts and lib/services/hackathons.ts) with related hackathon delete/route logic changes.

Poem

🐰 A hackathon vanished with care,

Logging each step of the affair,

Dialogs confirm with a click,

Auth checks are thorough and thick,

Deletion done right, debug laid bare! ✨

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/companydashboard

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 81c610f and 4d64cec.

📒 Files selected for processing (3)

• app/api/hackathons/[id]/route.ts (3 hunks)
• app/dashboard/company/[slug]/hackathons/page.tsx (5 hunks)
• lib/services/hackathons.ts (1 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}