From c836e184a6398262b99a818dfa3fc930e04e2b54 Mon Sep 17 00:00:00 2001
From: AbelOsaretin <contact.abel321@gmail.com>
Date: Thu, 22 Oct 2026 21:11:00 +0100
Subject: [PATCH 1/4] fix(seo): remove /connect from sitemap to match
 robots.txt disallow

Crawlers were seeing /connect in the sitemap while robots.txt disallowed
it, wasting crawl budget. Remove the entry so all sitemap URLs are
indexable.

Fixes #559
---
 src/app/sitemap.ts | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/src/app/sitemap.ts b/src/app/sitemap.ts
index 16f18c7..a09ad51 100644
--- a/src/app/sitemap.ts
+++ b/src/app/sitemap.ts
@@ -18,12 +18,6 @@ export default async function sitemap(): Promise<MetadataRoute.Sitemap> {
       changeFrequency: "daily",
       priority: 0.9,
     },
-    {
-      url: `${SITE_URL}/connect`,
-      lastModified: new Date(),
-      changeFrequency: "monthly",
-      priority: 0.5,
-    },
   ];
 
   let courseRoutes: MetadataRoute.Sitemap = [];

From b9b0af6177d2cde1ea8ca7eca260d1acf02056ee Mon Sep 17 00:00:00 2001
From: AbelOsaretin <contact.abel321@gmail.com>
Date: Thu, 22 Oct 2026 21:11:05 +0100
Subject: [PATCH 2/4] fix(rewards): stop re-throwing claim errors after toast

handleClaim already surfaces the failure via addToast, but the re-throw
was also caught by the route error boundary and rendered a second error
UI on top of the toast. Drop the re-throw so the toast is the only error
indication.

Fixes #560
---
 src/app/rewards/page.tsx | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/app/rewards/page.tsx b/src/app/rewards/page.tsx
index 4f1d780..cea7c8f 100644
--- a/src/app/rewards/page.tsx
+++ b/src/app/rewards/page.tsx
@@ -29,9 +29,8 @@ export default function RewardsPage() {
     try {
       await claim(claimableId);
       addToast("Reward claimed successfully!", "success");
-    } catch (err) {
+    } catch {
       addToast("Failed to claim reward. Please try again.", "error");
-      throw err;
     }
   };
 

From da8e24627eca84da6380c418cb99de7d97b1edb2 Mon Sep 17 00:00:00 2001
From: AbelOsaretin <contact.abel321@gmail.com>
Date: Thu, 22 Oct 2026 21:11:10 +0100
Subject: [PATCH 3/4] fix(a11y): unwrap course card Link to avoid nested
 interactive elements

The card was wrapped in <Link> which contained buttons and a dropdown,
producing invalid <a><button> nesting and trapping keyboard users.
Replace the wrapper with role="link" + onClick/router.push + Enter/Space
handler, and stop propagation on inner controls so the dropdown and
action buttons stay independently operable.

Fixes #561
---
 src/components/course/course-card.tsx | 30 ++++++++++++++++++++-------
 1 file changed, 23 insertions(+), 7 deletions(-)

diff --git a/src/components/course/course-card.tsx b/src/components/course/course-card.tsx
index 391353e..f9e61e2 100644
--- a/src/components/course/course-card.tsx
+++ b/src/components/course/course-card.tsx
@@ -1,7 +1,7 @@
 "use client";
 
 import { memo } from "react";
-import Link from "next/link";
+import { useRouter } from "next/navigation";
 import Image from "next/image";
 import { cn } from "@/lib/utils/cn";
 import { Card, CardContent, CardFooter } from "@/components/ui/card";
@@ -44,19 +44,36 @@ export const CourseCard = memo(function CourseCard({
   onContinue,
   className,
 }: CourseCardProps) {
+  const router = useRouter();
+
+  const handleNavigate = () => {
+    router.push(`/courses/${course.id}`);
+  };
+
+  const handleCardKeyDown = (e: React.KeyboardEvent) => {
+    if (e.key === "Enter" || e.key === " ") {
+      e.preventDefault();
+      handleNavigate();
+    }
+  };
+
   const handleEnroll = (e: React.MouseEvent) => {
-    e.preventDefault();
+    e.stopPropagation();
     onEnroll?.(course.id);
   };
 
   const handleContinue = (e: React.MouseEvent) => {
-    e.preventDefault();
+    e.stopPropagation();
     onContinue?.(course.id);
   };
 
   return (
-    <Link href={`/courses/${course.id}`}>
       <Card
+        role="link"
+        tabIndex={0}
+        aria-label={course.title}
+        onClick={handleNavigate}
+        onKeyDown={handleCardKeyDown}
         className={cn(
           "group cursor-pointer transition-all hover:shadow-lg hover:-translate-y-1",
           className,
@@ -107,9 +124,9 @@ export const CourseCard = memo(function CourseCard({
 
           <div
             className="absolute top-2 right-2"
-            onClick={(e) => e.preventDefault()}
+            onClick={(e) => e.stopPropagation()}
             onKeyDown={(e) => {
-              if (e.key === "Enter" || e.key === " ") e.preventDefault();
+              if (e.key === "Enter" || e.key === " ") e.stopPropagation();
             }}
           >
             <DropdownMenu>
@@ -192,6 +209,5 @@ export const CourseCard = memo(function CourseCard({
           )}
         </CardFooter>
       </Card>
-    </Link>
   );
 });

From 8d6f69843eb4a3de4d1b514ea589e18570101943 Mon Sep 17 00:00:00 2001
From: AbelOsaretin <contact.abel321@gmail.com>
Date: Thu, 22 Oct 2026 21:11:15 +0100
Subject: [PATCH 4/4] fix(a11y): add arrow-key and Home/End navigation to
 connect-button menu

The wallet dropdown used role="menu"/"menuitem" but only handled Escape
and click-outside. WCAG requires arrow-key navigation for menu widgets.
Move focus between menu items with ArrowUp/ArrowDown and jump to first/
last with Home/End; Escape already closes and restores focus to the
trigger.

Fixes #562
---
 src/components/wallet/connect-button.tsx | 38 ++++++++++++++++++++++++
 1 file changed, 38 insertions(+)

diff --git a/src/components/wallet/connect-button.tsx b/src/components/wallet/connect-button.tsx
index 3ff76ad..8c99f2d 100644
--- a/src/components/wallet/connect-button.tsx
+++ b/src/components/wallet/connect-button.tsx
@@ -29,12 +29,48 @@ export function ConnectButton() {
   const [showMainnetConfirm, setShowMainnetConfirm] = useState(false);
   const dropdownRef = useRef<HTMLDivElement>(null);
   const triggerRef = useRef<HTMLButtonElement>(null);
+  const menuRef = useRef<HTMLDivElement>(null);
 
   const closeDropdown = useCallback(() => {
     setShowDropdown(false);
     triggerRef.current?.focus();
   }, []);
 
+  const getMenuItems = useCallback(() => {
+    if (!menuRef.current) return [];
+    return Array.from(
+      menuRef.current.querySelectorAll<HTMLElement>('[role="menuitem"]')
+    );
+  }, []);
+
+  const handleMenuKeyDown = (e: React.KeyboardEvent) => {
+    const items = getMenuItems();
+    if (items.length === 0) return;
+
+    const currentIndex = items.findIndex(
+      (item) => item === document.activeElement
+    );
+
+    switch (e.key) {
+      case "ArrowDown":
+        e.preventDefault();
+        items[(currentIndex + 1 + items.length) % items.length].focus();
+        break;
+      case "ArrowUp":
+        e.preventDefault();
+        items[(currentIndex - 1 + items.length) % items.length].focus();
+        break;
+      case "Home":
+        e.preventDefault();
+        items[0].focus();
+        break;
+      case "End":
+        e.preventDefault();
+        items[items.length - 1].focus();
+        break;
+    }
+  };
+
   useEffect(() => {
     if (!showDropdown) return;
 
@@ -116,8 +152,10 @@ export function ConnectButton() {
 
         {showDropdown && (
           <div
+            ref={menuRef}
             className="absolute right-0 top-full mt-2 w-56 rounded-xl border border-gray-200 bg-white p-1.5 shadow-xl z-50 dark:border-gray-800 dark:bg-gray-900"
             role="menu"
+            onKeyDown={handleMenuKeyDown}
           >
             <div className="px-3 py-2 border-b border-gray-100 dark:border-gray-800">
               <div className="text-xs text-gray-400 font-medium">Connected Wallet</div>