fix: Include phone param on get-messages

wppconnect-team · Aug 8, 2022 · 46cc839 · 46cc839
1 parent fcb6a50
commit 46cc839
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/src/controller/deviceController.js b/src/controller/deviceController.js
@@ -470,11 +470,11 @@ export async function loadAndGetAllMessagesInChat(req, res) {
 }
 export async function getMessages(req, res) {
   const { phone } = req.params;
-  const { count = 20, direction = 'before', id = 0 } = req.query;
+  const { count = 20, direction = 'before', id = null } = req.query;
   try {
     const response = await req.client.getMessages(`${phone}`, {
       count: parseInt(count),
-      direction: direction,
+      direction: direction.toString(),
       id: id,
     });
     return res.status(200).json({ status: 'success', response: response });

diff --git a/src/swagger.json b/src/swagger.json
@@ -1604,7 +1604,7 @@
         "deprecated": false
       }
     },
-    "/get-messages": {
+    "/get-messages/{phone}": {
       "get": {
         "tags": ["Chat"],
         "summary": "Get messages in Chat",
@@ -1622,6 +1622,16 @@
               "example": "application/json"
             }
           },
+          {
+            "in": "path",
+            "name": "phone",
+            "schema": {
+              "type": "string"
+            },
+            "required": true,
+            "description": "The contact ID, with @c.us or @g.us"
+          },
+
           {
             "name": "count",
             "in": "query",