Improve telegram integration (GladysAssistant#840)

* Fix GladysAssistant#730 : Send a message when Telegram integration was configured with success

* Fix GladysAssistant#607 : In Telegram service, use the chat.id to answer to a user, not the from.id

* .gitignore .DS_Store

* Fix telegram tests

.gitignore

@@ -16,3 +16,4 @@ coverage
 # VSCode
 .vscode
 .tmp
+.DS_Store

server/services/telegram/lib/message.linkUser.js

@@ -20,6 +20,10 @@ async function linkUser(apiKey, chatId) {
     telegram_user_id: chatId,
   });
   logger.debug(`Telegram user linked with success.`);
+  // sending message to confirm
+  await this.send(chatId, {
+    text: '👋',
+  });
 }
 
 module.exports = {

server/services/telegram/lib/message.new.js

@@ -12,18 +12,21 @@ const { EVENTS } = require('../../../utils/constants');
 async function newMessage(msg) {
   logger.debug(`new message from telegram, ${msg.text}`);
   logger.debug(msg);
+
   const telegramUserId = msg.from.id;
+  const telegramChatId = msg.chat.id;
 
   if (msg.text.startsWith('/start')) {
     const splitted = msg.text.split(' ');
     await this.linkUser(splitted[1], telegramUserId);
     return null;
   }
+
   const user = await this.gladys.user.getByTelegramUserId(telegramUserId);
 
   const message = {
     source: 'telegram',
-    source_user_id: telegramUserId,
+    source_user_id: telegramChatId,
     user_id: user.id,
     user,
     language: user.language,

server/test/services/telegram/lib/messageHandler.test.js

@@ -35,6 +35,9 @@ describe('Telegram.message', () => {
       from: {
         id: '89798374Z7',
       },
+      chat: {
+        id: '89798374Z7',
+      },
       text: 'Hey',
       date: new Date(),
     });
@@ -44,6 +47,9 @@ describe('Telegram.message', () => {
       from: {
         id: '89798374Z7',
       },
+      chat: {
+        id: '89798374Z7',
+      },
       text: '/start testapikey',
       date: new Date(),
     });