Node module to fetch/post API data

Question

I'm fairly new to JavaScript and Node and am wondering if I'm on the right track.

Module file:

'use strict';

var chalk = require('chalk');
var config = require('../config');
var moment = require('moment');
var rp = require('request-promise');

var todayDay = moment().format('dddd'); // ["Monday"]" 
var todayDate = moment().format('DD').replace(/^0+/, ''); // "1"
var todayMonth = moment().format('MMMM'); // ["January"]

rp({
  method: 'GET',
  uri: config.api.uri + '/tasks',
  qs: {
    access_token: config.api.accessToken,
    project_id: config.api.projectId,
    page_size: 5
  },
  json: true
})
  .then(function(res) {
    console.log(res);
    var data = res.data;

    data.forEach(function(val) {

      // day is today or date and month is today
      if (val.custom_fields.custom_353.indexOf(todayDay) > -1 ||
        val.custom_fields.custom_356.indexOf(todayMonth) > -1 && val.custom_fields.custom_358 == todayDate) {

        // post 
        rp({
          method: 'POST',
          uri: config.api.uri + '/incidents',
          qs: {
            access_token: config.api.accessToken
          },
          body: {
            item: {
              name: val.name,
              description: val.description,
              due_date: moment().toISOString(),
              assigned_to: {
                type: val.assigned_to.type,
                id: val.assigned_to.id
              },
              project: {
                id: config.api.projectId
              },
              custom_fields: {
                custom_361: val.custom_fields.custom_384, // sequence
                custom_352: val.custom_fields.custom_270, // time
                custom_354: val.custom_fields.custom_269, // procedure
                custom_277: val.custom_fields.custom_183, // service group
                custom_278: val.custom_fields.custom_184 // service
              }
            }
          },
          json: true
        })
          .then(function(res) {
            console.log(chalk.green(val.name));
          })
          .catch(function(err) {
            console.log(chalk.red(err));
          });
      } else {
        console.log(chalk.grey(val.name));
      }
    });
  })
  .catch(function(err) {
    console.log(chalk.red(err));
  });

Config file:

'use strict';

var env = process.env.NODE_ENV || 'development';
var config = {};

switch (env) {
  case 'development':
    config = require('../env/development/config.json');
    break;
  case 'production':
    config = require('../env/production/config.json');
    break;
};

module.exports = config;

Answer 1

var todayDate = moment().format('DD').replace(/^0+/, ''); // "1"

Looks like you're taking a zero-padded day of month here... and then stripping off the leading zero.

You should use moment().format('D'), it will give you day of month without a leading zero.

Aside from that... you shouldn't use multiple instances of moment(). What if the time changes between the two lines of code? You might get things like 31 February.

var todayMoment = moment();

var todayDay = todayMoment.format('dddd'); // ["Monday"]
var todayDate = todayMoment.format('D'); // "1"
var todayMonth = todayMoment.format('MMMM'); // ["January"]

Like so, both problems are fixed.