Merge pull request #68 from astoria-tech/twilio-credentials-flag

Check for Twilio credentials before attempting to send a followup text

Author: Peter Valdez

src/index.js

@@ -296,6 +296,14 @@ async function checkForNewSubmissions() {
       nextPage();
     });
 
+  // Check if Twilio credentials are present, in order to send followup text
+  const accountSid = config.TWILIO_ACCOUNT_SID;
+  const authToken = config.TWILIO_AUTH_TOKEN;
+  const client = require("twilio")(accountSid, authToken);
+  const twilioPhoneNumber = config.TWILIO_PHONE_NUMBER;
+  const hasTwilioCredentials =
+    (accountSid && authToken && twilioPhoneNumber) || false;
+
   // Check Airtable for tasks completed in the last day, then send volunteer
   // a followup text
   base(config.AIRTABLE_REQUESTS_TABLE_NAME)
@@ -322,48 +330,53 @@ async function checkForNewSubmissions() {
             logger.error(err);
             return;
           }
-          const accountSid = config.TWILIO_ACCOUNT_SID;
-          const authToken = config.TWILIO_AUTH_TOKEN;
-          const client = require("twilio")(accountSid, authToken);
           const phoneNumber = rec.get(
             "Please provide your contact phone number:"
           );
           const formattedPhoneNumber = formatPhoneNumber(phoneNumber);
 
-          logger.info(`Sending followup text to: ${formattedPhoneNumber}`);
-          client.messages
-            .create({
-              body: "Thank you for being a great neighbor!",
-              from: config.TWILIO_PHONE_NUMBER,
-              to: formattedPhoneNumber,
-            })
-            .then((message) => {
-              logger.info(`Message SID: ${message.sid}`);
-              base(config.AIRTABLE_REQUESTS_TABLE_NAME).update(
-                [
-                  {
-                    id: record.id,
-                    fields: {
-                      "Followup SMS Sent?": "Yes",
+          if (hasTwilioCredentials) {
+            logger.info(`Sending followup text to: ${formattedPhoneNumber}`);
+            client.messages
+              .create({
+                body: "Thank you for being a great neighbor!",
+                from: config.TWILIO_PHONE_NUMBER,
+                to: formattedPhoneNumber,
+              })
+              .then((message) => {
+                logger.info(`Message SID: ${message.sid}`);
+                base(config.AIRTABLE_REQUESTS_TABLE_NAME).update(
+                  [
+                    {
+                      id: record.id,
+                      fields: {
+                        "Followup SMS Sent?": "Yes",
+                      },
                     },
-                  },
-                ],
-                function (err, records) {
-                  if (err) {
-                    logger.error(err);
-                    return;
+                  ],
+                  function (err, records) {
+                    if (err) {
+                      logger.error(err);
+                      return;
+                    }
+                    records.forEach(function (record) {
+                      logger.info(
+                        `Followup text sent?: ${record.get(
+                          "Followup SMS Sent?"
+                        )}`
+                      );
+                    });
                   }
-                  records.forEach(function (record) {
-                    logger.info(
-                      `Followup text sent?: ${record.get("Followup SMS Sent?")}`
-                    );
-                  });
-                }
-              );
-            })
-            .catch((error) => {
-              logger.error(`onRejected function called: ${error.message}`);
-            });
+                );
+              })
+              .catch((error) => {
+                logger.error(`onRejected function called: ${error.message}`);
+              });
+          } else {
+            logger.error(
+              "Twilio credentials missing -- Followup text not sent"
+            );
+          }
         });
       }
       nextPage();