From 8b95c0d77b6db4e06a364838b3c09a0d9701efd4 Mon Sep 17 00:00:00 2001
From: Mohan Ki <mohanki@digifox.live>
Date: Mon, 27 Jul 2026 18:27:35 +0530
Subject: [PATCH] fix: resolve Untitled QR issue and make destination links
 clickable in Admin modal

---
 api/routes/admin.js                        | 2 +-
 client/src/components/UserDetailsModal.tsx | 8 ++++++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/api/routes/admin.js b/api/routes/admin.js
index 82a7c46..d741846 100644
--- a/api/routes/admin.js
+++ b/api/routes/admin.js
@@ -36,7 +36,7 @@ router.get('/users/:id/details', async (req, res) => {
         
         // Fetch QR codes and scan counts
         const qrRes = await pgPool.query(`
-            SELECT q.id, q.type, q.data_type, q.destination_url, q.short_url_id, q.created_at, q.design_data->>'name' as name,
+            SELECT q.id, q.type, q.data_type, q.destination_url, q.short_url_id, q.created_at, q.name,
                    (SELECT COUNT(*) FROM qr_scans s WHERE s.qr_code_id = q.id) as scan_count
             FROM qr_codes q
             WHERE q.user_id = $1
diff --git a/client/src/components/UserDetailsModal.tsx b/client/src/components/UserDetailsModal.tsx
index c6fd8da..04057aa 100644
--- a/client/src/components/UserDetailsModal.tsx
+++ b/client/src/components/UserDetailsModal.tsx
@@ -98,11 +98,15 @@ export default function UserDetailsModal({ userId, onClose }: UserDetailsModalPr
                             </span>
                           </div>
                           <div className="text-xs text-gray-500 mt-1 truncate max-w-md" title={qr.destination_url}>
-                            {qr.destination_url}
+                            <a href={qr.destination_url} target="_blank" rel="noopener noreferrer" className="hover:text-indigo-600 hover:underline">
+                              {qr.destination_url}
+                            </a>
                           </div>
                           {qr.type === 'dynamic' && qr.short_url_id && (
                             <div className="text-xs text-indigo-500 mt-1">
-                              /{qr.short_url_id}
+                              <a href={`${process.env.NEXT_PUBLIC_APP_URL || ''}/l/${qr.short_url_id}`} target="_blank" rel="noopener noreferrer" className="hover:text-indigo-700 hover:underline">
+                                /{qr.short_url_id}
+                              </a>
                             </div>
                           )}
                         </div>