@itentialopensource/adapter-slack 2.6.1 → 2.6.2

package/CHANGELOG.md

@@ -1,4 +1,12 @@
 
+## 2.6.2 [08-03-2022]
+
+* Add parameters for postFilesupload
+
+See merge request itentialopensource/adapters/notification-messaging/adapter-slack!13
+
+---
+
 ## 2.6.1 [07-25-2022]
 
 * Fix schema for threadTs

package/adapter.js

@@ -5283,10 +5283,11 @@ class Slacktest extends AdapterBaseCl {
    * @param {string} content - File contents via a POST variable. If omitting this parameter, you must provide a `file`.
    * @param {string} token - Authentication token. Requires scope: `files:write:user`
    * @param {string} file - File contents via `multipart\/form-data`. If omitting this parameter, you must submit `content`.
+   * @param {number} threadTs - Unique identifier of a thread's parent message
    * @param {getCallback} callback - a callback function to return the result
    */
   /* YOU CAN CHANGE THE PARAMETERS YOU TAKE IN HERE AND IN THE pronghorn.json FILE */
-  postFilesupload(channels, title, initialComment, filetype, filename, content, token, file, callback) {
+  postFilesupload(channels, title, initialComment, filetype, filename, content, token, file, threadTs, callback) {
     const meth = 'adapter-postFilesupload';
     const origin = `${this.id}-${meth}`;
     log.trace(origin);
@@ -5303,7 +5304,7 @@ class Slacktest extends AdapterBaseCl {
     const queryParamsAvailable = {};
     const queryParams = {};
     const pathVars = [];
-    const bodyVars = {};
+    const bodyVars = { channels, title, initialComment, filetype, filename, content, file, threadTs };
 
     // loop in template. long callback arg name to avoid identifier conflicts
     Object.keys(queryParamsAvailable).forEach((thisKeyInQueryParamsAvailable) => {

package/entities/Channels/schema.json

@@ -29,7 +29,7 @@
       "external_name": "ph_request_type"
     },
     "threadTs": {
-      "type": "string",
+      "type": ["string", "number"],
       "description": "Unique identifier of a thread's parent message",
       "parse": false,
       "encode": false,

package/entities/Chat/schema.json

@@ -21,7 +21,7 @@
       "external_name": "ph_request_type"
     },
     "threadTs": {
-      "type": "string",
+      "type": ["string", "number"],
       "description": "Unique identifier of a thread's parent message",
       "parse": false,
       "encode": false,

package/entities/Files/schema.json

@@ -2,7 +2,7 @@
   "$id": "schema.json",
   "type": "object",
   "schema": "http://json-schema.org/draft-07/schema#",
-  "translate": false,
+  "translate": true,
   "dynamicfields": true,
   "properties": {
     "ph_request_type": {
@@ -18,6 +18,17 @@
         "postFilesupload"
       ],
       "external_name": "ph_request_type"
+    },
+    "threadTs": {
+      "type": ["string", "number"],
+      "description": "Unique identifier of a thread's parent message",
+      "parse": false,
+      "encode": false,
+      "encrypt": {
+        "type": "AES",
+        "key": ""
+      },
+      "external_name": "thread_ts"
     }
   },
   "definitions": {}

package/entities/Groups/schema.json

@@ -30,7 +30,7 @@
       "external_name": "ph_request_type"
     },
     "threadTs": {
-      "type": "string",
+      "type": ["string", "number"],
       "description": "Unique identifier of a thread's parent message",
       "parse": false,
       "encode": false,

package/entities/Im/schema.json

@@ -20,7 +20,7 @@
       "external_name": "ph_request_type"
     },
     "threadTs": {
-      "type": "string",
+      "type": ["string", "number"],
       "description": "Unique identifier of a thread's parent message",
       "parse": false,
       "encode": false,

package/entities/Mpim/schema.json

@@ -20,7 +20,7 @@
       "external_name": "ph_request_type"
     },
     "threadTs": {
-      "type": "string",
+      "type": ["string", "number"],
       "description": "Unique identifier of a thread's parent message",
       "parse": false,
       "encode": false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@itentialopensource/adapter-slack",
-  "version": "2.6.1",
+  "version": "2.6.2",
   "description": "This adapter integrates with system Slack",
   "main": "adapter.js",
   "systemName": "Slack",

package/pronghorn.json

@@ -4332,6 +4332,16 @@
             "title": "file",
             "type": "string"
           }
+        },
+        {
+          "name": "threadTs",
+          "type": "number",
+          "info": "Unique identifier of a thread's parent message",
+          "required": false,
+          "schema": {
+            "title": "threadTs",
+            "type": "number"
+          }
         }
       ],
       "output": {

package/report/adapterInfo.json

@@ -1,10 +1,10 @@
 {
-  "version": "2.5.5",
-  "configLines": 10347,
+  "version": "2.6.1",
+  "configLines": 10357,
   "scriptLines": 1707,
-  "codeLines": 12509,
+  "codeLines": 12510,
   "testLines": 9108,
   "testCases": 394,
-  "totalCodeLines": 23324,
+  "totalCodeLines": 23325,
   "wfTasks": 155
 }

package/test/integration/adapterTestIntegration.js

@@ -238,7 +238,7 @@ const Slacktest = require('../../adapter');
 
 // begin the testing - these should be pretty well defined between the describe and the it!
 describe('[integration] Slack Adapter Test', () => {
-  describe('Slacktest Class Tests', () => {
+  describe('Slack Class Tests', () => {
     const a = new Slacktest(
       pronghornProps.adapterProps.adapters[0].id,
       pronghornProps.adapterProps.adapters[0].properties
@@ -2848,7 +2848,7 @@ describe('[integration] Slack Adapter Test', () => {
     describe('#postFilesupload - errors', () => {
       it('should work if integrated or standalone with mockdata', (done) => {
         try {
-          a.postFilesupload('fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', (data, error) => {
+          a.postFilesupload('fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 0, (data, error) => {
             try {
               if (stub) {
                 assert.equal(undefined, error);

package/test/unit/adapterTestUnit.js

@@ -131,7 +131,7 @@ if (fs.existsSync(dirPath)) {
 
 // begin the testing - these should be pretty well defined between the describe and the it!
 describe('[unit] Slack Adapter Test', () => {
-  describe('Slacktest Class Tests', () => {
+  describe('Slack Class Tests', () => {
     const a = new Slacktest(
       pronghornProps.adapterProps.adapters[0].id,
       pronghornProps.adapterProps.adapters[0].properties