Merge pull request #36 from topcoder-platform/dev

Production update with notification email improvements

Author: vikasrohit

.circleci/config.yml

@@ -82,7 +82,7 @@ workflows:
       - "build-dev":
           filters:
             branches:
-              only: [dev, 'feature/discourseMigration']
+              only: [dev, 'feature/notification-email-improvements']
       - "build-prod":
           filters:
             branches:

config/default.js

@@ -2,6 +2,7 @@
  * The configuration file.
  */
 module.exports = {
+  ENV: process.env.ENV,
   LOG_LEVEL: process.env.LOG_LEVEL,
   PORT: process.env.PORT,
   authSecret: process.env.authSecret,

src/app.js

@@ -91,9 +91,11 @@ function startKafkaConsumer(handlers) {
                     userEmail = config.DEV_MODE_EMAIL;
                   }
                   const recipients = [userEmail];
-                  if (notificationType === BUS_API_EVENT.EMAIL.MENTIONED_IN_POST) {
-                    recipients.push(config.MENTION_EMAIL);
+                  const cc = [];
+                  if (eventType === BUS_API_EVENT.EMAIL.MENTIONED_IN_POST) {
+                    cc.push(config.MENTION_EMAIL);
                   }
+                  const categories = [`${config.ENV}:${eventType}`.toLowerCase()];
 
                   // get jwt token then encode it with base64
                   const body = {
@@ -123,6 +125,12 @@ function startKafkaConsumer(handlers) {
                     },
                     recipients,
                     replyTo,
+                    cc,
+                    from: {
+                      name: notification.contents.userHandle,
+                      email: '[email protected]',//TODO pick from config
+                    },
+                    categories,
                   };
                   // send event to bus api
                   return service.postEvent({