(core) Skipping docNotifications for users who haven't logged in yet

Summary: Document notifications are not sent to users who haven't logged into grist previously.

Test Plan: Updated

Reviewers: paulfitz

Reviewed By: paulfitz

Subscribers: paulfitz

Differential Revision: https://phab.getgrist.com/D4681

Author: berhalak

app/gen-server/lib/homedb/HomeDBManager.ts

@@ -2393,6 +2393,7 @@ export class HomeDBManager {
       const orgAccess = orgMapWithMembership[u.id] || null;
       return {
         ...this.makeFullUser(u),
+        firstLoginAt: undefined, // Not part of PermissionData.
         loginEmail: undefined,    // Not part of PermissionData.
         access: docMap[u.id] || null,
         parentAccess: roles.getEffectiveRole(

app/gen-server/lib/homedb/UsersManager.ts

@@ -201,6 +201,7 @@ export class UsersManager {
       ref: user.ref,
       locale: user.options?.locale,
       prefs: user.prefs?.find((p)=> p.orgId === null)?.prefs,
+      firstLoginAt: user.firstLoginAt || null,
     };
     if (user.firstLoginAt) {
       result.firstLoginAt = user.firstLoginAt;

test/gen-server/lib/homedb/UsersManager.ts

@@ -464,7 +464,8 @@ describe('UsersManager', function () {
           picture: input.picture,
           ref: input.ref,
           locale: someUserLocale,
-          prefs: prefWithoutOrg.prefs
+          prefs: prefWithoutOrg.prefs,
+          firstLoginAt: null,
         });
       });