From 86b1414ae03b08001d4c577419cf3ae73742a468 Mon Sep 17 00:00:00 2001
From: Guillermo Pages <mail@guillermo.at>
Date: Mon, 24 Nov 2025 10:43:51 +0100
Subject: [PATCH] refactor: simplify navigation for admin portal

Remove player-facing menu items:
- Remove 'Booking' (player feature)
- Remove 'Player Lookup' (player feature)
- Remove 'Assessment' (player feature)

Update remaining navigation:
- Replace 'Dashboard' with direct link to '/admin/clubs'
- Rename to 'Venue Management' for clarity
- Simplified menu now shows only admin features
- Updated animation indices after removing items

Result: Clean, focused admin portal navigation.
---
 src/components/Navigation.tsx | 105 +++-------------------------------
 1 file changed, 7 insertions(+), 98 deletions(-)

diff --git a/src/components/Navigation.tsx b/src/components/Navigation.tsx
index 63b46a4..f7647ba 100644
--- a/src/components/Navigation.tsx
+++ b/src/components/Navigation.tsx
@@ -329,107 +329,16 @@ export default function Navigation({ pageTitle }: NavigationProps) {
                         animate="open"
                       >
                         <Link
-                          href={localizedLink("/dashboard")}
+                          href={localizedLink("/admin/clubs")}
                           onClick={closeMenu}
                           className="flex items-center space-x-3 p-4 rounded-xl hover:bg-slate-50 transition-colors duration-200 group"
                         >
-                          <div className="p-2 rounded-lg bg-slate-100 group-hover:bg-slate-200 transition-colors duration-200">
-                            <BarChart3 className="w-5 h-5 text-slate-700" />
+                          <div className="p-2 rounded-lg bg-purple-100 group-hover:bg-purple-200 transition-colors duration-200">
+                            <Settings className="w-5 h-5 text-purple-700" />
                           </div>
-                          <span className="font-medium text-slate-900">{t('Dashboard')}</span>
+                          <span className="font-medium text-slate-900">{t('Venue Management')}</span>
                         </Link>
                       </motion.div>
-
-                  <motion.div
-                    custom={1}
-                    variants={menuItemVariants}
-                    initial="closed"
-                    animate="open"
-                  >
-                    <button
-                      onClick={() => {
-                        closeMenu();
-                        setTimeout(() => {
-                          router.push(bookingLink);
-                        }, 100);
-                      }}
-                      className="w-full flex items-center space-x-3 p-4 rounded-xl hover:bg-slate-50 transition-colors duration-200 group text-left"
-                    >
-                      <div className="p-2 rounded-lg bg-slate-100 group-hover:bg-slate-200 transition-colors duration-200">
-                        <Calendar className="w-5 h-5 text-slate-700" />
-                      </div>
-                      <span className="font-medium text-slate-900">{t('Booking')}</span>
-                    </button>
-                  </motion.div>
-
-                  {/* Divider */}
-                  <div className="my-2 border-t border-slate-100"></div>
-
-                  <motion.div
-                    custom={2}
-                    variants={menuItemVariants}
-                    initial="closed"
-                    animate="open"
-                  >
-                    <div
-                      onClick={() => {
-                        const remoteSlug = settings?.default_remote_sport.facility_slug;
-                        const sportSlug = settings?.default_remote_sport.sport_slug;
-                        const hasCreds = !!getClaimedRemoteMember(settings);
-                        closeMenu();
-                        if (remoteSlug && sportSlug && hasCreds) {
-                          setShowPartnerModal(true);
-                        } else {
-                          setShowClaimModal(true);
-                        }
-                      }}
-                      className="flex items-center space-x-3 p-4 rounded-xl hover:bg-slate-50 transition-colors duration-200 group cursor-pointer"
-                    >
-                      <div className="p-2 rounded-lg bg-slate-100 group-hover:bg-slate-200 transition-colors duration-200">
-                        <Users className="w-5 h-5 text-slate-700" />
-                      </div>
-                      <span className="font-medium text-slate-900">{t('Player Lookup')}</span>
-                    </div>
-                  </motion.div>
-
-                  <motion.div
-                    custom={3}
-                    variants={menuItemVariants}
-                    initial="closed"
-                    animate="open"
-                  >
-                    <Link
-                      href={localizedLink("/assessment")}
-                      onClick={closeMenu}
-                      className="flex items-center space-x-3 p-4 rounded-xl hover:bg-slate-50 transition-colors duration-200 group"
-                    >
-                      <div className="p-2 rounded-lg bg-slate-100 group-hover:bg-slate-200 transition-colors duration-200">
-                        <HelpCircle className="w-5 h-5 text-slate-700" />
-                      </div>
-                      <span className="font-medium text-slate-900">{t("What's my level?")}</span>
-                    </Link>
-                  </motion.div>
-
-                  {/* Admin Section */}
-                  <div className="my-4 border-t border-slate-100"></div>
-
-                  <motion.div
-                    custom={4}
-                    variants={menuItemVariants}
-                    initial="closed"
-                    animate="open"
-                  >
-                    <Link
-                      href={localizedLink("/admin/clubs")}
-                      onClick={closeMenu}
-                      className="flex items-center space-x-3 p-4 rounded-xl hover:bg-slate-50 transition-colors duration-200 group"
-                    >
-                      <div className="p-2 rounded-lg bg-purple-100 group-hover:bg-purple-200 transition-colors duration-200">
-                        <Settings className="w-5 h-5 text-purple-700" />
-                      </div>
-                      <span className="font-medium text-slate-900">{t('Venue Admin')}</span>
-                    </Link>
-                  </motion.div>
                     </>
                   )}
 
@@ -437,7 +346,7 @@ export default function Navigation({ pageTitle }: NavigationProps) {
                   <div className="my-4 border-t border-slate-100"></div>
 
                   <motion.div
-                    custom={isLoggedIn ? 5 : 2}
+                    custom={isLoggedIn ? 1 : 2}
                     variants={menuItemVariants}
                     initial="closed"
                     animate="open"
@@ -497,7 +406,7 @@ export default function Navigation({ pageTitle }: NavigationProps) {
                   {isLoggedIn && (
                     <div className="flex items-center gap-3">
                       <motion.div
-                        custom={6}
+                        custom={2}
                         variants={menuItemVariants}
                         initial="closed"
                         animate="open"
@@ -515,7 +424,7 @@ export default function Navigation({ pageTitle }: NavigationProps) {
                         </Link>
                       </motion.div>
                       <motion.div
-                        custom={7}
+                        custom={3}
                         variants={menuItemVariants}
                         initial="closed"
                         animate="open"