🎉 include non-grapher explorer views in homepage chart count

owid · Apr 16, 2024 · fe0fb50 · fe0fb50
1 parent 681fab1
commit fe0fb50
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 1 deletion.
diff --git a/db/db.ts b/db/db.ts
@@ -310,3 +310,27 @@ export const getHomepageId = (
             AND published = TRUE`
     ).then((result) => result?.id)
 }
+
+export const getNonGrapherExplorerViewCount = (
+    knex: KnexReadonlyTransaction
+): Promise<number> => {
+    return knexRawFirst<{ count: number }>(
+        knex,
+        `-- sql
+        SELECT
+            COUNT(*) as count
+        FROM
+            explorers,
+            json_table(config, "$.blocks[*]"
+                COLUMNS (
+                    type TEXT PATH "$.type",
+                    NESTED PATH "$.block[*]"
+                        COLUMNS (grapherId INT PATH "$.grapherId")
+                    )
+            ) t1
+        WHERE
+            isPublished = 1
+            AND type = "graphers"
+            AND grapherId IS NULL`
+    ).then((res) => res?.count ?? 0)
+}
diff --git a/db/model/Gdoc/GdocHomepage.ts b/db/model/Gdoc/GdocHomepage.ts
@@ -63,8 +63,12 @@ export class GdocHomepage
     _loadSubclassAttachments = async (
         knex: db.KnexReadonlyTransaction
     ): Promise<void> => {
+        const grapherCount = await db.getTotalNumberOfCharts(knex)
+        const nonGrapherExplorerViewCount =
+            await db.getNonGrapherExplorerViewCount(knex)
+
         this.homepageMetadata = {
-            chartCount: await db.getTotalNumberOfCharts(knex),
+            chartCount: grapherCount + nonGrapherExplorerViewCount,
             topicCount: UNIQUE_TOPIC_COUNT,
         }