From 29a98034a98b55d9ca8f6e8dd267c6b258ed0554 Mon Sep 17 00:00:00 2001
From: Thomas <thomas.lapiana+github@pm.me>
Date: Mon, 12 Sep 2022 10:24:21 -0700
Subject: [PATCH] Remove disabled Nav Buttons (#1067)

* Update NavBar.tsx

* remove "more" from the navbar

* fix a linting error

* remove cypress tests for disabled links
---
 clients/ctl/admin-ui/cypress/e2e/nav-bar.cy.ts          | 5 -----
 clients/ctl/admin-ui/src/features/common/nav/NavBar.tsx | 9 ---------
 2 files changed, 14 deletions(-)

diff --git a/clients/ctl/admin-ui/cypress/e2e/nav-bar.cy.ts b/clients/ctl/admin-ui/cypress/e2e/nav-bar.cy.ts
index 08bfba8d1b..72d93f19bb 100644
--- a/clients/ctl/admin-ui/cypress/e2e/nav-bar.cy.ts
+++ b/clients/ctl/admin-ui/cypress/e2e/nav-bar.cy.ts
@@ -7,11 +7,6 @@ describe("Nav Bar", () => {
     cy.getByTestId("nav-link-Taxonomy");
     cy.getByTestId("nav-link-Systems");
 
-    // Disabled links
-
-    cy.getByTestId("nav-link-Policies").should("have.attr", "disabled");
-    cy.getByTestId("nav-link-User Management").should("have.attr", "disabled");
-    cy.getByTestId("nav-link-More").should("have.attr", "disabled");
   });
 
   it("Renders the active page based on the current route", () => {
diff --git a/clients/ctl/admin-ui/src/features/common/nav/NavBar.tsx b/clients/ctl/admin-ui/src/features/common/nav/NavBar.tsx
index 8ccc6fbc2e..da778c5675 100644
--- a/clients/ctl/admin-ui/src/features/common/nav/NavBar.tsx
+++ b/clients/ctl/admin-ui/src/features/common/nav/NavBar.tsx
@@ -3,7 +3,6 @@ import dynamic from "next/dynamic";
 import React from "react";
 
 import { useFeatures } from "~/features/common/features.slice";
-import { ArrowDownLineIcon } from "~/features/common/Icon";
 
 // Cross-zone navigation requires building URLs from the current `window.location`
 // which is not available in Server-Side-Rendered components.
@@ -27,17 +26,9 @@ const NavBar = () => {
         {/* Links under the datamap zone: */}
         {features.plus ? <NavLink title="Data Map" href="/datamap" /> : null}
 
-        <NavLink title="Policies" href="/policy" disabled />
         <NavLink title="Taxonomy" href="/taxonomy" />
-        <NavLink title="User Management" href="/user-management" disabled />
         {/* This is a temporary link to the config wizard while it's still in progress */}
         <NavLink title="Config Wizard" href="/config-wizard" />
-        <NavLink
-          title="More"
-          href="#"
-          rightIcon={<ArrowDownLineIcon />}
-          disabled
-        />
       </nav>
     </Flex>
   );