From 14ed80fe4bc93d25da9f44cd4b5a2777098aa9f4 Mon Sep 17 00:00:00 2001
From: "Troll (Hermes Agent)" <ai@hallsworth.ca>
Date: Sun, 2 Aug 2026 00:02:05 +0000
Subject: [PATCH] Fix dashboard filters: Delivered instead of Done, correct
 Needs Upload status mapping

---
 models.py                      | 2 ++
 templates/admin/dashboard.html | 4 ++--
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/models.py b/models.py
index baa8c07..c1bcf95 100644
--- a/models.py
+++ b/models.py
@@ -118,6 +118,8 @@ def list_requests(status=None):
     """List all requests, optionally filtered by status, newest first."""
     db = get_db()
     if status:
+        # Translate human filter names to stored status values.
+        status = status.lower().replace(' ', '_')
         rows = db.execute('SELECT * FROM requests WHERE status = ? ORDER BY created_at DESC', (status,)).fetchall()
     else:
         rows = db.execute('SELECT * FROM requests ORDER BY created_at DESC').fetchall()
diff --git a/templates/admin/dashboard.html b/templates/admin/dashboard.html
index 7b07808..3782aa8 100644
--- a/templates/admin/dashboard.html
+++ b/templates/admin/dashboard.html
@@ -127,9 +127,9 @@
     <!-- Status filter links -->
     <div class="filters">
       <a href="{{ url_for('admin_dashboard') }}" class="{% if current_status in (None, '') %}active{% endif %}">All</a>
-      <a href="{{ url_for('admin_dashboard', status='needs_upload') }}" class="{% if current_status == 'needs_upload' %}active{% endif %}">Needs Upload</a>
+      <a href="{{ url_for('admin_dashboard', status='songs_uploaded') }}" class="{% if current_status == 'songs_uploaded' %}active{% endif %}">Needs Upload</a>
       <a href="{{ url_for('admin_dashboard', status='awaiting_payment') }}" class="{% if current_status == 'awaiting_payment' %}active{% endif %}">Awaiting Payment</a>
-      <a href="{{ url_for('admin_dashboard', status='done') }}" class="{% if current_status == 'done' %}active{% endif %}">Done</a>
+      <a href="{{ url_for('admin_dashboard', status='delivered') }}" class="{% if current_status == 'delivered' %}active{% endif %}">Delivered</a>
       <span class="refresh-hint">
         {% if refresh_seconds and refresh_seconds > 0 %}
         Auto-refresh every {{ refresh_seconds }}s