fix: Show audit log in production if allowed (#3960)

kylecarbs · web-flow · commit 2c41343ce58c · 2022-09-08T14:58:53.000Z
diff --git a/site/src/components/NavbarView/NavbarView.test.tsx b/site/src/components/NavbarView/NavbarView.test.tsx
@@ -69,17 +69,6 @@ describe("NavbarView", () => {
     expect((auditLink as HTMLAnchorElement).href).toContain("/audit")
   })
 
-  it("audit nav link is only visible in development", async () => {
-    process.env = {
-      ...env,
-      NODE_ENV: "production",
-    }
-
-    render(<NavbarView user={MockUser} onSignOut={noop} canViewAuditLog />)
-    const auditLink = screen.queryByText(navLanguage.audit)
-    expect(auditLink).not.toBeInTheDocument()
-  })
-
   it("audit nav link is hidden for members", async () => {
     render(<NavbarView user={MockUser2} onSignOut={noop} canViewAuditLog={false} />)
     const auditLink = screen.queryByText(navLanguage.audit)
diff --git a/site/src/components/NavbarView/NavbarView.tsx b/site/src/components/NavbarView/NavbarView.tsx
@@ -52,8 +52,7 @@ const NavItems: React.FC<
           {Language.users}
         </NavLink>
       </ListItem>
-      {/* REMARK: the below link is under-construction  */}
-      {process.env.NODE_ENV !== "production" && canViewAuditLog && (
+      {canViewAuditLog && (
         <ListItem button className={styles.item}>
           <NavLink className={styles.link} to="/audit">
             {Language.audit}
