From 169edeed6b3e8249aa8f1114e08155ee172aacf9 Mon Sep 17 00:00:00 2001
From: David Crespo <david.crespo@oxidecomputer.com>
Date: Wed, 4 Sep 2024 16:31:22 -0500
Subject: [PATCH] remove unreachable sign in button in user dropdown

---
 app/components/TopBar.tsx | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/app/components/TopBar.tsx b/app/components/TopBar.tsx
index 708662be3a..4c5e2e2366 100644
--- a/app/components/TopBar.tsx
+++ b/app/components/TopBar.tsx
@@ -22,8 +22,6 @@ export function TopBar({ children }: { children: React.ReactNode }) {
   // fetch happens in loader wrapping all authed pages
   const { me } = useCurrentUser()
 
-  const loggedIn = !!me
-
   // toArray filters out nulls, which is essential because the silo/system
   // picker is going to come in null when the user isn't supposed to see it
   const [cornerPicker, ...otherPickers] = React.Children.toArray(children)
@@ -62,15 +60,9 @@ export function TopBar({ children }: { children: React.ReactNode }) {
               </DropdownMenu.Trigger>
               <DropdownMenu.Content align="end" sideOffset={8}>
                 <DropdownMenu.LinkItem to={pb.profile()}>Settings</DropdownMenu.LinkItem>
-                {loggedIn ? (
-                  <DropdownMenu.Item onSelect={() => logout.mutate({})}>
-                    Sign out
-                  </DropdownMenu.Item>
-                ) : (
-                  <DropdownMenu.Item onSelect={() => navToLogin({ includeCurrent: true })}>
-                    Sign In
-                  </DropdownMenu.Item>
-                )}
+                <DropdownMenu.Item onSelect={() => logout.mutate({})}>
+                  Sign out
+                </DropdownMenu.Item>
               </DropdownMenu.Content>
             </DropdownMenu.Root>
           </div>