Merge pull request #753 from Nickatak/feat/navbar-trim-org-links

feat: Drop How to Join / Projects from the header nav

Author: RSkuma

frontend/src/shared/components/nav/HeaderNav.tsx

@@ -1,10 +1,10 @@
 /**
  * Top-of-page navigation header for the `(with-nav)` route group.
  *
- * Renders the CTJ logo (linking home), three external links to
- * Hack for LA org pages, an auth control on the right, and a mobile
- * hamburger menu trigger. The `(auth)` route group uses a different
- * `AuthNav` component.
+ * Renders the CTJ logo (linking home), a single external link to
+ * the Hack for LA org site, an auth control on the right, and a
+ * mobile hamburger menu trigger. The `(auth)` route group uses a
+ * different `AuthNav` component.
  *
  * Auth control: there is no Figma frame for the signed-in nav state
  * - the original app had no auth UI at all. Until a design exists,
@@ -39,10 +39,13 @@ interface MenuObject {
   link: string;
 }
 
+// "How to Join" and "Projects" were dropped per the 2026-05-14 nav
+// decision; "View Opportunities" is the intended replacement but is
+// blocked on its destination (internal route vs hackforla.org), so
+// it lands in a follow-up. Kept as an array since that follow-up
+// re-adds an entry.
 const menuItems: MenuObject[] = [
   { name: "Hack for LA", link: "https://www.hackforla.org/" },
-  { name: "How to Join", link: "https://www.hackforla.org/getting-started" },
-  { name: "Projects", link: "https://www.hackforla.org/projects/" },
 ];
 
 function Logo() {

frontend/tests/components/HeaderNav.test.tsx

@@ -1,5 +1,6 @@
 /**
- * Tests for `HeaderNav`'s auth-state-dependent auth control.
+ * Tests for `HeaderNav`'s auth-state-dependent auth control and its
+ * external-link list.
  *
  * `HeaderNav` is tested in isolation against a mocked `useAuth`; the
  * provider's bootstrap (csrf + me round-trips) is exercised in
@@ -56,3 +57,40 @@ describe("HeaderNav auth control", () => {
     expect(mockLogout).toHaveBeenCalledTimes(1);
   });
 });
+
+describe("HeaderNav structure", () => {
+  beforeEach(() => {
+    mockAuth = { user: null, loading: false, logout: mockLogout };
+  });
+
+  /** The logo renders and links home (SVG is aria-hidden; the link
+   * carries the accessible name). */
+  test("renders the logo linking home", () => {
+    render(<HeaderNav />);
+    expect(
+      screen.getByRole("link", { name: "Civic Tech Jobs - Home" }),
+    ).toHaveAttribute("href", "/");
+  });
+});
+
+describe("HeaderNav external links", () => {
+  beforeEach(() => {
+    mockAuth = { user: null, loading: false, logout: mockLogout };
+  });
+
+  /** Only the "Hack for LA" org link remains after the 2026-05-14 trim. */
+  test("renders the Hack for LA link", () => {
+    render(<HeaderNav />);
+    expect(screen.getByText("Hack for LA").closest("a")).toHaveAttribute(
+      "href",
+      "https://www.hackforla.org/",
+    );
+  });
+
+  /** "How to Join" and "Projects" were dropped; guard against regression. */
+  test("does not render the dropped How to Join / Projects links", () => {
+    render(<HeaderNav />);
+    expect(screen.queryByText("How to Join")).not.toBeInTheDocument();
+    expect(screen.queryByText("Projects")).not.toBeInTheDocument();
+  });
+});