limit offered by facet to specific offerors

frontends/main/src/app-pages/SearchPage/SearchPage.test.tsx

@@ -583,9 +583,17 @@ describe("Search Page Tabs", () => {
 
 test("Facet 'Offered By' uses API response for names", async () => {
   const offerors = factories.learningResources.offerors({ count: 3 })
+  for (const offeror of offerors.results) {
+    offeror.display_facet = true
+  }
+  const resources = factories.learningResources.resources({
+    count: 3,
+  }).results
+
   setMockApiResponses({
     offerors,
     search: {
+      results: resources,
       metadata: {
         aggregations: {
           offered_by: offerors.results.map((o, i) => ({
@@ -618,6 +626,53 @@ test("Facet 'Offered By' uses API response for names", async () => {
   expect(offeror2).toBeVisible()
 })
 
+test("Facet 'Offered By' only shows facets with 'display_facet' set to true", async () => {
+  const offerors = factories.learningResources.offerors({ count: 3 })
+
+  const resources = factories.learningResources.resources({
+    count: 3,
+  }).results
+
+  offerors.results[0].display_facet = true
+  offerors.results[1]!.display_facet = false
+  offerors.results[2]!.display_facet = false
+
+  setMockApiResponses({
+    search: {
+      results: resources,
+      metadata: {
+        aggregations: {
+          offered_by: offerors.results.map((o, i) => ({
+            key: o.code,
+            doc_count: 10 + i,
+          })),
+        },
+        suggestions: [],
+      },
+    },
+    offerors: offerors,
+  })
+  renderWithProviders(<SearchPage />)
+  const showFacetButton = await screen.findByRole("button", {
+    name: /Offered By/i,
+  })
+
+  await user.click(showFacetButton)
+
+  const offeror0 = await screen.findByRole("checkbox", {
+    name: `${offerors.results[0].name} 10`,
+  })
+  const offeror1 = screen.queryByRole("checkbox", {
+    name: `${offerors.results[1].name} 11`,
+  })
+  const offeror2 = screen.queryByRole("checkbox", {
+    name: `${offerors.results[2].name} 12`,
+  })
+  expect(offeror0).toBeVisible()
+  expect(offeror1).not.toBeInTheDocument()
+  expect(offeror2).not.toBeInTheDocument()
+})
+
 test("Set sort", async () => {
   setMockApiResponses({ search: { count: 137 } })
 

frontends/main/src/page-components/SearchDisplay/SearchDisplay.tsx

@@ -19,7 +19,7 @@ import {
   childCheckboxStyles,
   VisuallyHidden,
 } from "@mitodl/smoot-design"
-
+import { keyBy } from "lodash"
 import {
   RiCloseLine,
   RiArrowLeftLine,
@@ -28,13 +28,16 @@ import {
   RiArrowUpSLine,
   RiArrowDownSLine,
 } from "@remixicon/react"
-
+import {
+  useOfferorsList,
+  learningResourceQueries,
+} from "api/hooks/learningResources"
 import {
   LearningResourcesSearchApiLearningResourcesSearchRetrieveRequest as LRSearchRequest,
   ResourceCategoryEnum,
   SearchModeEnumDescriptions,
 } from "api"
-import { useLearningResourcesSearch } from "api/hooks/learningResources"
+import { keepPreviousData, useQuery } from "@tanstack/react-query"
 import { useAdminSearchParams } from "api/hooks/adminSearchParams"
 import {
   AvailableFacets,
@@ -568,12 +571,39 @@ const SearchDisplay: React.FC<SearchDisplayProps> = ({
     facetNames,
     page,
   ])
+  const offerorsQuery = useOfferorsList()
+  const offerors = useMemo(() => {
+    return keyBy(offerorsQuery.data?.results ?? [], (o) => o.code)
+  }, [offerorsQuery.data?.results])
+
+  const { data, isLoading, isFetching } = useQuery({
+    ...learningResourceQueries.search(allParams as LRSearchRequest),
+    placeholderData: keepPreviousData,
+    select: (data) => {
+      // Handle missing data gracefully
+      if (!data.metadata.aggregations.offered_by || data.results.length === 0) {
+        return data
+      }
 
-  const { data, isLoading, isFetching } = useLearningResourcesSearch(
-    allParams as LRSearchRequest,
-    { keepPreviousData: true },
-  )
-
+      // only show offerors with display_facet set
+      const displayOfferors = Object.values(offerors)
+        .filter((value) => value.code && value.display_facet)
+        .map((value) => value?.code)
+
+      return {
+        ...data,
+        metadata: {
+          ...data.metadata,
+          aggregations: {
+            ...data.metadata.aggregations,
+            offered_by: data.metadata.aggregations.offered_by.filter(
+              (value) => value && displayOfferors.includes(value.key),
+            ),
+          },
+        },
+      }
+    },
+  })
   const { data: user } = useUserMe()
 
   const [mobileDrawerOpen, setMobileDrawerOpen] = React.useState(false)

learning_resources/migrations/0100_learningresourceofferor_display_facet.py

@@ -0,0 +1,17 @@
+# Generated by Django 4.2.25 on 2025-11-06 19:55
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("learning_resources", "0099_alter_learningresource_resource_type"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="learningresourceofferor",
+            name="display_facet",
+            field=models.BooleanField(default=True),
+        ),
+    ]

learning_resources/models.py

@@ -173,6 +173,8 @@ class LearningResourceOfferor(TimestampedModel):
     more_information = models.URLField(blank=True)
     # This field name means "value proposition"
     value_prop = models.TextField(blank=True)
+    # whether or not to show this offeror as a facet in the UI
+    display_facet = models.BooleanField(default=True)
 
     @cached_property
     def channel_url(self):

learning_resources/serializers.py

@@ -113,7 +113,7 @@ class LearningResourceOfferorSerializer(serializers.ModelSerializer):
 
     class Meta:
         model = models.LearningResourceOfferor
-        fields = ("code", "name", "channel_url")
+        fields = ("code", "name", "channel_url", "display_facet")
 
 
 class LearningResourceOfferorDetailSerializer(LearningResourceOfferorSerializer):

learning_resources/serializers_test.py

@@ -558,6 +558,7 @@ def test_content_file_serializer(settings, expected_types, has_channels):
             "offered_by": {
                 "name": content_file.run.learning_resource.offered_by.name,
                 "code": content_file.run.learning_resource.offered_by.code,
+                "display_facet": True,
                 "channel_url": frontend_absolute_url(
                     f"/c/unit/{Channel.objects.get(unit_detail__unit=content_file.run.learning_resource.offered_by).name}/"
                 )

openapi/specs/v0.yaml

@@ -3379,6 +3379,8 @@ components:
           type: string
           readOnly: true
           nullable: true
+        display_facet:
+          type: boolean
       required:
       - channel_url
       - code
@@ -3435,6 +3437,8 @@ components:
           maxLength: 200
         value_prop:
           type: string
+        display_facet:
+          type: boolean
       required:
       - channel_url
       - code
@@ -3495,6 +3499,8 @@ components:
           maxLength: 200
         value_prop:
           type: string
+        display_facet:
+          type: boolean
       required:
       - code
       - name

openapi/specs/v1.yaml

@@ -11626,6 +11626,8 @@ components:
           type: string
           readOnly: true
           nullable: true
+        display_facet:
+          type: boolean
       required:
       - channel_url
       - code
@@ -11682,6 +11684,8 @@ components:
           maxLength: 200
         value_prop:
           type: string
+        display_facet:
+          type: boolean
       required:
       - channel_url
       - code
@@ -11698,6 +11702,8 @@ components:
           type: string
           minLength: 1
           maxLength: 256
+        display_facet:
+          type: boolean
       required:
       - code
       - name

vector_search/tasks.py

@@ -381,7 +381,8 @@ def embeddings_healthcheck():
     """
     Check for missing embeddings and summaries in Qdrant and log warnings to Sentry
     """
-    remaining_content_files = []
+
+    remaining_content_file_ids = []
     remaining_resources = []
     resource_point_ids = {}
     all_resources = LearningResource.objects.filter(
@@ -405,10 +406,14 @@ def embeddings_healthcheck():
                     )
                     content_file_point_ids[point_id] = {"key": cf.key, "id": cf.id}
             for batch in chunks(content_file_point_ids.keys(), chunk_size=200):
-                remaining_content_files.extend(
-                    filter_existing_qdrant_points_by_ids(
-                        batch, collection_name=CONTENT_FILES_COLLECTION_NAME
-                    )
+                remaining_content_files = filter_existing_qdrant_points_by_ids(
+                    batch, collection_name=CONTENT_FILES_COLLECTION_NAME
+                )
+                remaining_content_file_ids.extend(
+                    [
+                        content_file_point_ids.get(p, {}).get("id")
+                        for p in remaining_content_files
+                    ]
                 )
 
     for batch in chunks(
@@ -422,16 +427,13 @@ def embeddings_healthcheck():
             )
         )
 
-    remaining_content_file_ids = [
-        content_file_point_ids.get(p, {}).get("id") for p in remaining_content_files
-    ]
     remaining_resource_ids = [
         resource_point_ids.get(p, {}).get("id") for p in remaining_resources
     ]
     missing_summaries = _missing_summaries()
     log.info(
         "Embeddings healthcheck found %d missing content file embeddings",
-        len(remaining_content_files),
+        len(remaining_content_file_ids),
     )
     log.info(
         "Embeddings healthcheck found %d missing resource embeddings",
@@ -442,20 +444,20 @@ def embeddings_healthcheck():
         len(missing_summaries),
     )
 
-    if len(remaining_content_files) > 0:
+    if len(remaining_content_file_ids) > 0:
         _sentry_healthcheck_log(
             "embeddings",
             "missing_content_file_embeddings",
             {
-                "count": len(remaining_content_files),
+                "count": len(remaining_content_file_ids),
                 "ids": remaining_content_file_ids,
                 "run_ids": set(
                     ContentFile.objects.filter(
                         id__in=remaining_content_file_ids
                     ).values_list("run__run_id", flat=True)[:100]
                 ),
             },
-            f"Warning: {len(remaining_content_files)} missing content file "
+            f"Warning: {len(remaining_content_file_ids)} missing content file "
             "embeddings detected",
         )