Merge pull request #441 from sundeepgupta/chatUpdateSanitizeOptions

Slack's `chat.update` should work with attachments
howdyai · Nov 7, 2016 · 8e8ffcc · 8e8ffcc
2 parents ba8c91d + 56802e2
commit 8e8ffcc
Showing 1 changed file with 18 additions and 8 deletions.
diff --git a/lib/Slack_web_api.js b/lib/Slack_web_api.js
@@ -161,17 +161,27 @@ module.exports = function(bot, config) {
 
     // overwrite default behavior
     slack_api.chat.postMessage = function(options, cb) {
-        if (options.attachments && typeof(options.attachments) != 'string') {
-            try {
-                options.attachments = JSON.stringify(options.attachments);
-            } catch (err) {
-                delete options.attachments;
-                bot.log.error('Could not parse attachments', err);
-            }
-        }
+        sanitizeOptions(options);
         slack_api.callAPI('chat.postMessage', options, cb);
     };
 
+    slack_api.chat.update = function(options, cb) {
+        sanitizeOptions(options);
+        slack_api.callAPI('chat.update', options, cb);
+    };
+
+    function sanitizeOptions(options) {
+      if (options.attachments && typeof(options.attachments) != 'string') {
+          try {
+              options.attachments = JSON.stringify(options.attachments);
+          } catch (err) {
+              delete options.attachments;
+              bot.log.error('Could not parse attachments', err);
+          }
+      }
+    }
+
+
     return slack_api;