Merge pull request #297 from PastVu/notifier

Manage notifications in config

config/default.config.js

@@ -5,6 +5,7 @@ module.exports = {
     env: 'development', // Enviroment: development, testing, production
     locales: ['en', 'ru'], // List of supported locales. First one is the default locale, if user transferred nothing
     lang: 'ru', // Language: ru, en
+    notifier: false, // If true, send periodical emails to users
 
     admin: {
         email: '',

config/local.config.js.docker-example

@@ -2,6 +2,8 @@ module.exports = function (config, appRequire) {
     const _ = appRequire('lodash');
 
     _.merge(config, {
+        // Send periodical emails to users depending on environment variable
+        notifier: process.env.NOTIFIER === 'true',
         client: {
             hostname: 'localhost',
         },

config/local.config.js.example

@@ -2,6 +2,7 @@ module.exports = function (config, appRequire) {
     const _ = appRequire('lodash');
 
     _.merge(config, {
+        notifier: true, // Send periodical emails to user
         client: {
             hostname: 'pastvu.local',
         },

controllers/subscr.js

@@ -553,6 +553,12 @@ async function giveUserSubscriptions({ login, page = 1, type = 'photo' }) {
 }
 
 export const ready = (async () => {
+    if (! config.notifier) {
+        return;
+    } //Turn off notifier if disabled in config
+
+    logger.info('Proceeding to load notifier...');
+
     try {
         const data = await fsAsync.readFile(noticeTplPath, 'utf-8');
 

docker-compose.yml

@@ -32,6 +32,16 @@ services:
       - store:/store
     command: npm run app
 
+  notifier:
+    image: pastvu/node
+    depends_on:
+      - "app"
+    environment:
+      - NOTIFIER=true
+    volumes:
+      - .:/code
+    command: npm run app
+
   uploader:
     image: pastvu/node
     expose: