From c9423d43c1c5f9d292fbae8a05ff45115a926008 Mon Sep 17 00:00:00 2001 From: pac7 <47831526-pac7@users.noreply.replit.com> Date: Mon, 13 Oct 2025 14:43:51 +0000 Subject: [PATCH] Improve sorting controls for moderation queues Update the `QueueSortControls` component to validate sort fields and improve type safety for sort direction toggling. Replit-Commit-Author: Agent Replit-Commit-Session-Id: ef7037e7-a631-48a2-94d1-9a4b52d7c35a Replit-Commit-Checkpoint-Type: intermediate_checkpoint Replit-Commit-Screenshot-Url: https://storage.googleapis.com/screenshot-production-us-central1/7cdf4e95-3f41-4180-b8e3-8ef56d032c0e/ef7037e7-a631-48a2-94d1-9a4b52d7c35a/kcnWjk6 --- .replit | 4 ++++ src/components/moderation/QueueSortControls.tsx | 15 +++++++++++++-- 2 files changed, 17 insertions(+), 2 deletions(-) diff --git a/.replit b/.replit index cf054042..c654948f 100644 --- a/.replit +++ b/.replit @@ -41,3 +41,7 @@ externalPort = 3000 [[ports]] localPort = 37143 externalPort = 3001 + +[[ports]] +localPort = 46865 +externalPort = 3002 diff --git a/src/components/moderation/QueueSortControls.tsx b/src/components/moderation/QueueSortControls.tsx index 9fdfb5f5..b0ae3c8f 100644 --- a/src/components/moderation/QueueSortControls.tsx +++ b/src/components/moderation/QueueSortControls.tsx @@ -21,14 +21,25 @@ export const QueueSortControls = ({ onSortChange, isMobile }: QueueSortControlsProps) => { - const handleFieldChange = (field: SortField) => { + const handleFieldChange = (value: string) => { + const validFields: SortField[] = ['created_at', 'submission_type', 'status']; + + if (!validFields.includes(value as SortField)) { + console.warn('⚠️ [SORT] Invalid sort field:', value); + return; + } + + const field = value as SortField; + console.log('🔄 [SORT] Field change:', { from: sortConfig.field, to: field }); onSortChange({ ...sortConfig, field }); }; const handleDirectionToggle = () => { + const newDirection = sortConfig.direction === 'asc' ? 'desc' : 'asc'; + console.log('🔄 [SORT] Direction toggle:', { from: sortConfig.direction, to: newDirection }); onSortChange({ ...sortConfig, - direction: sortConfig.direction === 'asc' ? 'desc' : 'asc' + direction: newDirection }); };