Browse Source

Merge pull request #9434 from weseek/fix/notification-for-new-user-creation

fix: Notification for new user creation
mergify[bot] 1 year ago
parent
commit
0c32e88e6b
1 changed files with 6 additions and 1 deletions
  1. 6 1
      apps/app/src/server/routes/login.js

+ 6 - 1
apps/app/src/server/routes/login.js

@@ -50,10 +50,15 @@ module.exports = function(crowi, app) {
       targetModel: SupportedTargetModel.MODEL_USER,
       targetModel: SupportedTargetModel.MODEL_USER,
     });
     });
 
 
+    /**
+     * @param {import('../service/pre-notify').PreNotifyProps} props
+     */
     const preNotify = async(props) => {
     const preNotify = async(props) => {
+      /** @type {(import('mongoose').HydratedDocument<import('@growi/core').IUser>)[]} */
       const adminUsers = await User.findAdmins();
       const adminUsers = await User.findAdmins();
 
 
-      props.push(...adminUsers);
+      const { notificationTargetUsers } = props;
+      notificationTargetUsers?.push(...adminUsers);
     };
     };
 
 
     await activityEvent.emit('updated', activity, user, preNotify);
     await activityEvent.emit('updated', activity, user, preNotify);