Skip to content

Commit e843c56

Browse files
committed
chore(notifications): Updating cleanup comment
1 parent 9c54a41 commit e843c56

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

functions/src/notifications/cleanupNotifications.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const db = admin.firestore()
1010
// TODO
1111
// 1.) Do we actually want to delete old notifications? (check with Matt V)
1212
// 2.) Do we actually want to delete old notificaiton events? Why not keep the history?
13-
// 3.) Should we just use the builtin TTL feature for all of these?
13+
// 3.) We should just use the builtin TTL feature for all of these, right?
1414
// 4.) Should we use a bulk delete operation for firestore for all of these?
1515

1616
// Delete old notifications from userNotificationFeed collections

0 commit comments

Comments
 (0)