From 2c7cb0074507f221282f9b54c51cc9a8792a9a3d Mon Sep 17 00:00:00 2001
From: aarbit <aarbit@gmail.com>
Date: Tue, 19 May 2026 10:02:04 -0500
Subject: [PATCH] Lists all unreviewed photos for a hunt

---
 src/lib/api/index.ts                |   3 +
 src/routes/admin/PhotoReview.svelte | 106 ++++++++++++++++++++++------
 2 files changed, 86 insertions(+), 23 deletions(-)

diff --git a/src/lib/api/index.ts b/src/lib/api/index.ts
index 31fa33f..2c544b9 100644
--- a/src/lib/api/index.ts
+++ b/src/lib/api/index.ts
@@ -104,6 +104,9 @@ export function photoUrl(photoId: string, version: 'ORIGINAL' | 'LARGE' | 'MEDIU
 export const apiReviewPhoto = (photoId: string, status: 'SUBMITTED' | 'APPROVED' | 'REJECTED' | 'REMOVED') =>
   client.patch<void>(`/admin/photo/${photoId}`, { status })
 
+export const apiGetUnreviewedPhotos = (huntId: string) =>
+  client.get<PhotoResponse[]>(`/admin/hunt/${huntId}/photo/unreviewed`)
+
 // ── Stats ─────────────────────────────────────────────────────────────────────
 
 export const apiGetTeamLeaderboard = (huntId: string) =>
diff --git a/src/routes/admin/PhotoReview.svelte b/src/routes/admin/PhotoReview.svelte
index 522a5ed..d94df8e 100644
--- a/src/routes/admin/PhotoReview.svelte
+++ b/src/routes/admin/PhotoReview.svelte
@@ -1,7 +1,7 @@
 <script lang="ts">
     import {push} from 'svelte-spa-router'
     import {auth} from '../../lib/stores/auth.svelte'
-    import {apiGetHunt, apiGetItemPhotos, apiGetItems, apiListTeams, apiReviewPhoto} from '../../lib/api/index'
+    import {apiGetHunt, apiGetItemPhotos, apiGetItems, apiGetUnreviewedPhotos, apiListTeams, apiReviewPhoto} from '../../lib/api/index'
     import {parseUTC} from '../../lib/utils'
     import type {HuntResponse, ItemResponse, PhotoResponse, TeamResponse} from '../../lib/api/types'
     import StatusBadge from '../../lib/components/StatusBadge.svelte'
@@ -15,26 +15,42 @@
     if (!auth.isAdmin) push('/')
   })
 
-  type PhotoWithTeam = PhotoResponse & { teamName: string }
+  type PhotoWithContext = PhotoResponse & { teamName: string; itemName: string }
 
   let hunt = $state<HuntResponse | null>(null)
   let teams = $state<TeamResponse[]>([])
   let items = $state<ItemResponse[]>([])
   let selectedItem = $state<ItemResponse | null>(null)
-  let photos = $state<PhotoWithTeam[]>([])
+  let photos = $state<PhotoWithContext[]>([])
+  let unreviewedPhotos = $state<PhotoWithContext[]>([])
   let loading = $state(true)
   let photosLoading = $state(false)
   let error = $state('')
   let reviewing = $state<string | null>(null)
-  let expandedPhoto = $state<PhotoWithTeam | null>(null)
+  let expandedPhoto = $state<PhotoWithContext | null>(null)
 
   $effect(() => {
     const { huntId } = params
-    Promise.all([apiGetHunt(huntId), apiListTeams(huntId), apiGetItems(huntId)])
-      .then(([h, t, i]) => {
+    Promise.all([apiGetHunt(huntId), apiListTeams(huntId), apiGetItems(huntId), apiGetUnreviewedPhotos(huntId)])
+      .then(async ([h, t, i, unreviewed]) => {
         hunt = h
         teams = t
         items = i
+        // Build photoId → context map from all item+team combos in parallel
+        const rows = await Promise.all(
+          i.flatMap(item =>
+            t.map(team =>
+              apiGetItemPhotos(huntId, team.id, item.id)
+                .then(ps => ps.map(p => [p.id, { itemName: item.name, teamName: team.name }] as [string, { itemName: string; teamName: string }]))
+                .catch(() => [] as [string, { itemName: string; teamName: string }][])
+            )
+          )
+        )
+        const contextMap = new Map(rows.flat())
+        unreviewedPhotos = unreviewed.map(p => {
+          const ctx = contextMap.get(p.id) ?? { itemName: 'Unknown', teamName: 'Unknown' }
+          return { ...p, ...ctx }
+        })
       })
       .catch(e => { error = e instanceof Error ? e.message : 'Failed to load' })
       .finally(() => { loading = false })
@@ -47,7 +63,7 @@
       Promise.all(
         teams.map(team =>
           apiGetItemPhotos(params.huntId, team.id, selectedItem!.id)
-            .then(ps => ps.map(p => ({ ...p, teamName: team.name })))
+            .then(ps => ps.map(p => ({ ...p, teamName: team.name, itemName: selectedItem!.name })))
         )
       )
         .then(results => { photos = results.flat() })
@@ -56,12 +72,13 @@
     }
   })
 
-  async function review(photo: PhotoWithTeam, status: 'APPROVED' | 'REJECTED' | 'REMOVED') {
+  async function review(photo: PhotoWithContext, status: 'APPROVED' | 'REJECTED' | 'REMOVED') {
     reviewing = photo.id
     try {
       await apiReviewPhoto(photo.id, status)
-      const updated = { ...photo, photoStatus: status } as PhotoWithTeam
+      const updated = { ...photo, photoStatus: status } as PhotoWithContext
       photos = photos.map(p => p.id === photo.id ? updated : p)
+      unreviewedPhotos = unreviewedPhotos.map(p => p.id === photo.id ? updated : p)
       if (expandedPhoto?.id === photo.id) expandedPhoto = updated
     } catch (e: unknown) {
       error = e instanceof Error ? e.message : 'Failed to review photo'
@@ -70,7 +87,8 @@
     }
   }
 
-  const activePhotos = $derived(photos.filter(p => p.photoStatus !== 'REMOVED'))
+  const activePhotos    = $derived(photos.filter(p => p.photoStatus !== 'REMOVED'))
+  const activeUnreviewed = $derived(unreviewedPhotos.filter(p => p.photoStatus === 'SUBMITTED'))
 </script>
 
 <!-- Lightbox -->
@@ -83,6 +101,7 @@
   >
     <div class="flex items-center justify-between px-4 py-3 shrink-0">
       <div>
+        <p class="text-xs font-semibold uppercase tracking-wide text-white/40">{photo.itemName}</p>
         <p class="font-semibold text-white">{photo.hunterName} <span class="text-white/50 font-normal">· {photo.teamName}</span></p>
         <p class="text-xs text-white/40">{parseUTC(photo.photoUploadDateTime).toLocaleString()}</p>
       </div>
@@ -148,14 +167,20 @@
   {:else}
     <!-- Item selector -->
     <div class="mb-4">
-      <p class="text-sm font-medium text-base-content/60 mb-2">Select Item</p>
+      <p class="text-sm font-medium text-base-content/60 mb-2">
+        {#if selectedItem}
+          Filtering by item — tap again to see all unreviewed
+        {:else}
+          Unreviewed ({activeUnreviewed.length})
+        {/if}
+      </p>
       <div class="flex flex-wrap gap-2">
         {#each items as item}
           <button
             class="btn btn-sm"
             class:btn-secondary={selectedItem?.id === item.id}
             class:btn-outline={selectedItem?.id !== item.id}
-            onclick={() => { selectedItem = item }}
+            onclick={() => { selectedItem = selectedItem?.id === item.id ? null : item }}
           >
             {item.name}
           </button>
@@ -164,6 +189,7 @@
     </div>
 
     {#if selectedItem}
+      <!-- Per-item photo view -->
       {#if photosLoading}
         <LoadingSpinner />
       {:else if activePhotos.length === 0}
@@ -172,7 +198,7 @@
         </div>
       {:else}
         <div class="flex flex-col gap-4">
-          {#each activePhotos as photo}
+          {#each activePhotos as photo (photo.id)}
             <div class="card bg-base-100 shadow-sm border border-base-200 overflow-hidden">
               <button class="w-full cursor-zoom-in" onclick={() => expandedPhoto = photo}>
                 <AuthImage photoId={photo.id} version="SMALL" alt={photo.hunterName} class="w-full h-56 object-cover" />
@@ -189,19 +215,11 @@
                 </div>
                 {#if photo.photoStatus === 'SUBMITTED'}
                   <div class="flex gap-2 mb-2">
-                    <button
-                      class="btn btn-success btn-sm flex-1"
-                      onclick={() => review(photo, 'APPROVED')}
-                      disabled={reviewing === photo.id}
-                    >
+                    <button class="btn btn-success btn-sm flex-1" onclick={() => review(photo, 'APPROVED')} disabled={reviewing === photo.id}>
                       {#if reviewing === photo.id}<span class="loading loading-spinner loading-xs"></span>{/if}
                       Approve
                     </button>
-                    <button
-                      class="btn btn-error btn-sm flex-1"
-                      onclick={() => review(photo, 'REJECTED')}
-                      disabled={reviewing === photo.id}
-                    >
+                    <button class="btn btn-error btn-sm flex-1" onclick={() => review(photo, 'REJECTED')} disabled={reviewing === photo.id}>
                       Reject
                     </button>
                   </div>
@@ -232,6 +250,48 @@
           {/each}
         </div>
       {/if}
+
+    {:else}
+      <!-- All-unreviewed view -->
+      {#if activeUnreviewed.length === 0}
+        <div class="text-center py-10 text-base-content/50">
+          <p>All caught up — no unreviewed photos!</p>
+        </div>
+      {:else}
+        <div class="flex flex-col gap-4">
+          {#each activeUnreviewed as photo (photo.id)}
+            <div class="card bg-base-100 shadow-sm border border-base-200 overflow-hidden">
+              <button class="w-full cursor-zoom-in" onclick={() => expandedPhoto = photo}>
+                <AuthImage photoId={photo.id} version="SMALL" alt={photo.hunterName} class="w-full h-56 object-cover" />
+              </button>
+              <div class="p-3">
+                <p class="text-xs font-semibold uppercase tracking-wide text-primary mb-1">{photo.itemName}</p>
+                <div class="flex items-center justify-between mb-3">
+                  <div>
+                    <p class="font-semibold">{photo.hunterName} <span class="text-base-content/50 font-normal">· {photo.teamName}</span></p>
+                    <p class="text-xs text-base-content/50">
+                      {parseUTC(photo.photoUploadDateTime).toLocaleString()}
+                    </p>
+                  </div>
+                  <StatusBadge status={photo.photoStatus} />
+                </div>
+                <div class="flex gap-2 mb-2">
+                  <button class="btn btn-success btn-sm flex-1" onclick={() => review(photo, 'APPROVED')} disabled={reviewing === photo.id}>
+                    {#if reviewing === photo.id}<span class="loading loading-spinner loading-xs"></span>{/if}
+                    Approve
+                  </button>
+                  <button class="btn btn-error btn-sm flex-1" onclick={() => review(photo, 'REJECTED')} disabled={reviewing === photo.id}>
+                    Reject
+                  </button>
+                </div>
+                <button class="btn btn-ghost btn-sm w-full text-base-content/50" onclick={() => review(photo, 'REMOVED')} disabled={reviewing === photo.id}>
+                  Remove
+                </button>
+              </div>
+            </div>
+          {/each}
+        </div>
+      {/if}
     {/if}
   {/if}
 </div>