Merge pull request #169 from alkem-io/notifications-168

Fixed wrong email payload for user removal

.gitignore

@@ -1,6 +1,7 @@
 /docs/diagrams/.$*
 /service/coverage/*
 /service/coverage-ci/*
+/node_modules/*
 
 *.log
 .env

service/package.json

@@ -1,6 +1,6 @@
 {
   "name": "alkemio-notifications",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "description": "Alkemio notifications service",
   "author": "Alkemio Foundation",
   "private": false,

service/src/services/domain/builders/platform-user-removed/platform.user.removed.notification.builder.ts

@@ -51,15 +51,8 @@ export class PlatformUserRemovedNotificationBuilder
 
   private createTemplatePayload(
     eventPayload: PlatformUserRemovedEventPayload,
-    recipient: User,
-    registrant?: User
+    recipient: User
   ): PlatformUserRemovedEmailPayload {
-    if (!registrant) {
-      throw Error(
-        `Registrant not provided for '${NotificationEventType.PLATFORM_USER_REMOVED}' event`
-      );
-    }
-
     const notificationPreferenceURL =
       this.alkemioUrlGenerator.createUserNotificationPreferencesURL(
         recipient.nameID
@@ -68,7 +61,7 @@ export class PlatformUserRemovedNotificationBuilder
       emailFrom: '[email protected]',
       registrant: {
         displayName: eventPayload.user.displayName,
-        email: registrant.email,
+        email: eventPayload.user.email,
       },
       recipient: {
         firstName: recipient.firstName,