Removes instances where we've feature flagged the new (current) design (

#2064)
mitodl · Jan 19, 2024 · e59c750 · e59c750
1 parent d6de83b
commit e59c750
Show file tree

Hide file tree

Showing 17 changed files with 335 additions and 959 deletions.
diff --git a/cms/templates/product_page.html b/cms/templates/product_page.html
diff --git a/frontend/public/src/components/AnonymousMenu.js b/frontend/public/src/components/AnonymousMenu.js
@@ -4,15 +4,13 @@ import React from "react"
 
 import MixedLink from "./MixedLink"
 import { routes } from "../lib/urls"
-import { checkFeatureFlag } from "../lib/util"
 
 type Props = {
   mobileView: boolean
 }
 
 const AnonymousMenu = ({ mobileView }: Props) => {
   const identifierPostfix = mobileView ? "Mobile" : "Desktop"
-  const newDesign = checkFeatureFlag("mitxonline-new-header", "anonymousUser")
   const makeNavLink = (text: string) => {
     return mobileView ? (
       <span data-bs-target="#nav" data-bs-toggle="collapse">
@@ -24,18 +22,16 @@ const AnonymousMenu = ({ mobileView }: Props) => {
   }
   return (
     <ul>
-      {newDesign ? (
-        <li>
-          <MixedLink
-            id="catalog"
-            dest={routes.catalog}
-            className="top-nav-link"
-            aria-label="Catalog"
-          >
-            {makeNavLink("Catalog")}
-          </MixedLink>
-        </li>
-      ) : null}
+      <li>
+        <MixedLink
+          id="catalog"
+          dest={routes.catalog}
+          className="top-nav-link"
+          aria-label="Catalog"
+        >
+          {makeNavLink("Catalog")}
+        </MixedLink>
+      </li>
       <li>
         <MixedLink
           id={"login".concat(identifierPostfix)}

diff --git a/frontend/public/src/components/AnonymousMenu_test.js b/frontend/public/src/components/AnonymousMenu_test.js
@@ -11,7 +11,7 @@ describe("AnonymousMenu component", () => {
     assert.equal(
       shallow(<AnonymousMenu mobileView={false} />)
         .find("MixedLink")
-        .at(0)
+        .at(1)
         .prop("dest"),
       routes.login
     )
@@ -21,7 +21,7 @@ describe("AnonymousMenu component", () => {
     assert.equal(
       shallow(<AnonymousMenu mobileView={false} />)
         .find("MixedLink")
-        .at(1)
+        .at(2)
         .prop("dest"),
       routes.register.begin
     )
@@ -31,7 +31,7 @@ describe("AnonymousMenu component", () => {
     assert.equal(
       shallow(<AnonymousMenu mobileView={true} />)
         .find("MixedLink")
-        .at(1)
+        .at(2)
         .prop("dest"),
       routes.register.begin
     )