remove course filter from featured carousel (#1164)

* remove course filter from featured carousel * fix homepage tests
mitodl · Jun 25, 2024 · 6a5d2e0 · 6a5d2e0
1 parent 4bff2f5
commit 6a5d2e0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/frontends/mit-open/src/pages/HomePage/HomePage.test.tsx b/frontends/mit-open/src/pages/HomePage/HomePage.test.tsx
@@ -115,32 +115,31 @@ describe("Home Page Carousel", () => {
   test.each<{ tab: string; params: FeaturedRequest }>([
     {
       tab: "All",
-      params: { limit: 12, resource_type: ["course"] },
+      params: { limit: 12 },
     },
     {
       tab: "Free",
-      params: { limit: 12, resource_type: ["course"], free: true },
+      params: { limit: 12, free: true },
     },
     {
       tab: "With Certificate",
       params: {
-        resource_type: ["course"],
         limit: 12,
         certification: true,
         professional: false,
       },
     },
     {
       tab: "Professional & Executive Learning",
-      params: { resource_type: ["course"], limit: 12, professional: true },
+      params: { limit: 12, professional: true },
     },
   ])("Featured Courses Carousel Tabs", async ({ tab, params }) => {
     const resources = learningResources.resources({ count: 12 })
     setupAPIs()
 
     // The "All" tab is initially visible, so it needs a response.
     setMockResponse.get(
-      urls.learningResources.featured({ limit: 12, resource_type: ["course"] }),
+      urls.learningResources.featured({ limit: 12 }),
       learningResources.resources({ count: 0 }),
     )
     // This is for the clicked tab (which might be "All")

diff --git a/frontends/mit-open/src/pages/HomePage/carousels.ts b/frontends/mit-open/src/pages/HomePage/carousels.ts
@@ -2,7 +2,6 @@ import type { ResourceCarouselProps } from "@/page-components/ResourceCarousel/R
 import type { FeaturedApiFeaturedListRequest as FeaturedListParams } from "api"
 
 const FEATURED_COMMON_PARAMS: FeaturedListParams = {
-  resource_type: ["course"],
   limit: 12,
 }
 const FEATURED_RESOURCES_CAROUSEL: ResourceCarouselProps["config"] = [