[RTY-260025]: feat(ui): update links to open in new tab

Author: harshmishra2701

app/routes.py

@@ -56,7 +56,6 @@ async def index(request: Request):
 
     new_short_url = session.pop("new_short_url", None)
     qr_enabled = session.pop("qr_enabled", False)
-    qr_type = session.pop("qr_type", "short")
     original_url = session.pop("original_url", None)
     short_code = session.pop("short_code", None)
     info_message = session.pop("info_message", None)
@@ -66,7 +65,7 @@ async def index(request: Request):
     qr_data = None
 
     if qr_enabled and new_short_url and short_code:
-        qr_data = new_short_url if qr_type == "short" else original_url
+        qr_data = new_short_url
         qr_filename = f"{short_code}.png"
         qr_dir = BASE_DIR / "static" / "qr"
         qr_dir.mkdir(parents=True, exist_ok=True)
@@ -231,44 +230,30 @@ def delete_recent_api(short_code: str):
     recent = get_recent_from_cache(MAX_RECENT_URLS) or []
     removed_from_cache = False
 
-    # Try removing from cache (memory only)
     for i, item in enumerate(recent):
         code = item.get("short_code") or item.get("code")
         if code == short_code:
-            recent.pop(i)
             removed_from_cache = True
             break
 
     db_available = db.is_connected()
     db_deleted = False
 
-    # If DB available → rely ONLY on DB
     if db_available:
         db_deleted = db.delete_by_short_code(short_code)
 
-        if not db_deleted:
-            raise HTTPException(
-                status_code=404, detail=f"short_code '{short_code}' not found"
-            )
-
-        return {
-            "status": "deleted",
-            "short_code": short_code,
-            "db_deleted": True,
-            "db_available": True,
-        }
-
-    # If DB NOT available → rely on cache only
-    if not removed_from_cache:
+    # ✅ If nothing was deleted anywhere → 404
+    if not removed_from_cache and not db_deleted:
         raise HTTPException(
             status_code=404, detail=f"short_code '{short_code}' not found"
         )
 
     return {
+        "success": True,
         "status": "deleted",
         "short_code": short_code,
-        "db_deleted": False,
-        "db_available": False,
+        "db_deleted": db_deleted,
+        "db_available": db_available,
     }
 
 

app/templates/index.html

@@ -28,9 +28,11 @@ <h1>tiny URL</h1>
       <div>
         <span class="ready-label">Ready</span>
         <div class="short-url">
-          <a href="{{ new_short_url }}" id="shortUrlDisplay" style="text-decoration: underline;">
+          <a href="{{ new_short_url }}" id="shortUrlDisplay" style="text-decoration: underline;" target="_blank"
+            rel="noopener noreferrer">
             {{ new_short_url }}
           </a>
+          </a>
         </div>
       </div>
     </div>
@@ -46,7 +48,7 @@ <h1>tiny URL</h1>
   <div class="recent-tray">
     <div class="recent-header">
       <h3>Recently Shortened</h3>
-      <a href="/recent" class="history-link">View History →</a>
+      <a href="/recent" class="history-link" target="_blank">View History →</a>
     </div>
     <div class="scroll-container">
       {% for url in urls %}
@@ -70,20 +72,20 @@ <h3>🔗 tiny URL</h3>
     <div class="footer-col">
       <h4>Product</h4>
       <ul>
-        <li><a href="cache/list">CACHE URLs</a></li>
-        <li><a href="#">API Documentation</a></li>
-        <li><a href="#">Custom Branded Links</a></li>
-        <li><a href="#">QR Code Engine</a></li>
+        <li><a href="cache/list" target="_blank">CACHE URLs</a></li>
+        <li><a href="#" target="_blank">API Documentation</a></li>
+        <li><a href="#" target="_blank">Custom Branded Links</a></li>
+        <li><a href="#" target="_blank">QR Code Engine</a></li>
       </ul>
     </div>
 
     <div class="footer-col">
       <h4>Support</h4>
       <ul>
-        <li><a href="/docs"> UI docs</a></li>
-        <li><a href="#">Help Center</a></li>
-        <li><a href="#">System Status</a></li>
-        <li><a href="#">Contact Us</a></li>
+        <li><a href="/docs" target="_blank"> UI docs</a></li>
+        <li><a href="#" target="_blank">Help Center</a></li>
+        <li><a href="#" target="_blank">System Status</a></li>
+        <li><a href="#" target="_blank">Contact Us</a></li>
       </ul>
     </div>