fix: Adding missing index for userNotificationFeed query

Mephistic · Mephistic · commit c47d07afa57e · 2025-02-25T00:16:23.000-05:00
diff --git a/firestore.indexes.json b/firestore.indexes.json
@@ -786,6 +786,20 @@
         { "queryScope": "COLLECTION", "order": "DESCENDING" },
         { "queryScope": "COLLECTION", "arrayConfig": "CONTAINS" }
       ]
+    },
+    {
+      "collectionGroup": "userNotificationFeed",
+      "queryScope": "COLLECTION",
+      "fields": [
+        {
+          "fieldPath": "notification.type",
+          "order": "ASCENDING"
+        },
+        {
+          "fieldPath": "notification.timestamp",
+          "order": "ASCENDING"
+        }
+      ]
     }
   ]
 }
diff --git a/functions/src/notifications/deliverNotifications.ts b/functions/src/notifications/deliverNotifications.ts
@@ -44,7 +44,6 @@ const deliverEmailNotifications = async () => {
   prepareHandlebars()
   console.log("Handlebars helpers and partials prepared")
 
-  // TODO: Add index
   const usersSnapshot = await db
     .collection("users")
     .where("nextDigestAt", "<=", now)

Original file line number	Diff line number	Diff line change
`@@ -786,6 +786,20 @@`
`786`	`786`	`{ "queryScope": "COLLECTION", "order": "DESCENDING" },`
`787`	`787`	`{ "queryScope": "COLLECTION", "arrayConfig": "CONTAINS" }`
`788`	`788`	`]`
	`789`	`+ },`
	`790`	`+ {`
	`791`	`+ "collectionGroup": "userNotificationFeed",`
	`792`	`+ "queryScope": "COLLECTION",`
	`793`	`+ "fields": [`
	`794`	`+ {`
	`795`	`+ "fieldPath": "notification.type",`
	`796`	`+ "order": "ASCENDING"`
	`797`	`+ },`
	`798`	`+ {`
	`799`	`+ "fieldPath": "notification.timestamp",`
	`800`	`+ "order": "ASCENDING"`
	`801`	`+ }`
	`802`	`+ ]`
`789`	`803`	`}`
`790`	`804`	`]`
`791`	`805`	`}`