feature/IO-3182-Phone-Number-Consent - Checkpoint

server/routes/smsRoutes.js

@@ -7,6 +7,7 @@ const { status, markConversationRead } = require("../sms/status");
 const validateFirebaseIdTokenMiddleware = require("../middleware/validateFirebaseIdTokenMiddleware");
 
 // Twilio Webhook Middleware for production
+// TODO: This is never actually doing anything, we should probably verify
 const twilioWebhookMiddleware = twilio.webhook({ validate: process.env.NODE_ENV === "PRODUCTION" });
 
 router.post("/receive", twilioWebhookMiddleware, receive);

server/sms/receive.js

@@ -16,7 +16,6 @@ const InstanceManager = require("../utils/instanceMgr").default;
  * @returns {Promise<*>}
  */
 const receive = async (req, res) => {
-  console.dir(req.body);
   const {
     logger,
     ioRedis,
@@ -118,6 +117,7 @@ const receive = async (req, res) => {
     };
 
     const broadcastRoom = getBodyshopRoom(conversation.bodyshop.id);
+
     ioRedis.to(broadcastRoom).emit("new-message-summary", {
       ...commonPayload,
       existingConversation: !!existingConversation,

server/sms/send.js

@@ -5,7 +5,7 @@ const client = twilio(process.env.TWILIO_AUTH_TOKEN, process.env.TWILIO_AUTH_KEY
 const gqlClient = require("../graphql-client/graphql-client").client;
 
 const send = async (req, res) => {
-  const { to, messagingServiceSid, body, conversationid, selectedMedia, imexshopid, bodyshopid } = req.body;
+  const { to, messagingServiceSid, body, conversationid, selectedMedia, imexshopid } = req.body;
   const {
     ioRedis,
     logger,
@@ -25,7 +25,7 @@ const send = async (req, res) => {
     image_path: selectedMedia.length > 0 ? selectedMedia.map((i) => i.src) : []
   });
 
-  if (!to || !messagingServiceSid || (!body && selectedMedia.length === 0) || !conversationid || !bodyshopid) {
+  if (!to || !messagingServiceSid || (!body && selectedMedia.length === 0) || !conversationid) {
     logger.log("sms-outbound-error", "ERROR", req.user.email, null, {
       type: "missing-parameters",
       messagingServiceSid,

server/sms/status.js

@@ -9,7 +9,6 @@ const logger = require("../utils/logger");
  * @returns {Promise<*>}
  */
 const status = async (req, res) => {
-  console.dir(req.body);
   const { SmsSid, SmsStatus } = req.body;
   const {
     ioRedis,