@chat21/chat21-ionic 3.0.82 → 3.0.85

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # chat21-ionic ver 3.0
 
+## 3.0.85 in PROD
+- bug-fixed: remove focus from last edited canned
+- changed: do not cache right side-bar conversation-info component
+- changed: chat21client.js file to 1.12.3
+
+## 3.0.84 in PROD
+- bug-fixed: handle window['analytics] errors
+
+## 3.0.83 in PROD
+- bug-fixed: canned item is not visible (missing scroll position) if ArrowDown keyboard button is pressed
+- bug-fixed: if ArrowUp keyboard button is pressed, text-area cursor moved left and canned not substituted correctly
+
 ## 3.0.82 in PROD
 - bug-fixed: do not disable textArea if is a direct archived conversation
 - bug-fixed: canned item not fire click event on Firefox browser

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@chat21/chat21-ionic",
     "author": "Tiledesk SRL",
-    "version": "3.0.82",
+    "version": "3.0.85",
     "license": "MIT License",
     "homepage": "https://tiledesk.com/",
     "repository": {

package/src/app/app.component.ts

@@ -1270,116 +1270,122 @@ export class AppComponent implements OnInit {
 
   private segmentSignIn(){
     let user = this.tiledeskAuthService.getCurrentUser();
-    try {
-      window['analytics'].page("Chat Auth Page, Signin", {});
-    } catch (err) {
-      this.logger.error('Event:Signed In [page] error', err);
-    }
-
-    try {
-      window['analytics'].identify(user.uid, {
-        name: user.firstname + ' ' + user.lastname,
-        email: user.email,
-        logins: 5,
-
-      });
-    } catch (err) {
-      this.logger.error('Event:Signed In [identify] error', err);
-    }
-    // Segments
-    try {
-      window['analytics'].track('Signed In', {
-        "username": user.firstname + ' ' + user.lastname,
-        "userId": user.uid
-      });
-    } catch (err) {
-      this.logger.error('Event:Signed In [track] error', err);
+    if(window['analytics']){
+      try {
+        window['analytics'].page("Chat Auth Page, Signin", {});
+      } catch (err) {
+        this.logger.error('Event:Signed In [page] error', err);
+      }
+  
+      try {
+        window['analytics'].identify(user.uid, {
+          name: user.firstname + ' ' + user.lastname,
+          email: user.email,
+          logins: 5,
+  
+        });
+      } catch (err) {
+        this.logger.error('Event:Signed In [identify] error', err);
+      }
+      // Segments
+      try {
+        window['analytics'].track('Signed In', {
+          "username": user.firstname + ' ' + user.lastname,
+          "userId": user.uid
+        });
+      } catch (err) {
+        this.logger.error('Event:Signed In [track] error', err);
+      }
     }
   }
 
 
   private segmentSignedOut(){
     let user = this.tiledeskAuthService.getCurrentUser();
-    try {
-      window['analytics'].page("Chat Auth Page, Signed Out", {});
-    } catch (err) {
-      this.logger.error('Event:Signed Out [page] error', err);
-    }
-
-    try {
-      window['analytics'].identify(user.uid, {
-        name: user.firstname + ' ' + user.lastname,
-        email: user.email,
-        logins: 5,
-
-      });
-    } catch (err) {
-      this.logger.error('Event:Signed Out [identify] error', err);
-    }
-
-    try {
-      window['analytics'].track('Signed Out', {
-        "username": user.firstname + ' ' + user.lastname,
-        "userId": user.uid
-      });
-    } catch (err) {
-      this.logger.error('Event:Signed Out [track] error', err);
-    }
-
-    try {
-      // setTimeout(() => {
-      window['analytics'].reset()
-      // }, 0);
-    } catch (err) {
-      this.logger.error('Event:reset error', err);
+    if(window['analytics']){
+      try {
+        window['analytics'].page("Chat Auth Page, Signed Out", {});
+      } catch (err) {
+        this.logger.error('Event:Signed Out [page] error', err);
+      }
+  
+      try {
+        window['analytics'].identify(user.uid, {
+          name: user.firstname + ' ' + user.lastname,
+          email: user.email,
+          logins: 5,
+  
+        });
+      } catch (err) {
+        this.logger.error('Event:Signed Out [identify] error', err);
+      }
+  
+      try {
+        window['analytics'].track('Signed Out', {
+          "username": user.firstname + ' ' + user.lastname,
+          "userId": user.uid
+        });
+      } catch (err) {
+        this.logger.error('Event:Signed Out [track] error', err);
+      }
+  
+      try {
+        // setTimeout(() => {
+        window['analytics'].reset()
+        // }, 0);
+      } catch (err) {
+        this.logger.error('Event:reset error', err);
+      }
     }
   }
 
   private segmentResolved(conversation: ConversationModel){
     let user = this.tiledeskAuthService.getCurrentUser();
-    try {
-      window['analytics'].page("Chat List Conversations Page, Chat Resolved", {});
-    } catch (err) {
-      this.logger.error('Event:Chat Resolved [page] error', err);
-    }
-
-    try {
-      window['analytics'].identify(user.uid, {
-        name: user.firstname + ' ' + user.lastname,
-        email: user.email,
-        logins: 5,
-
-      });
-    } catch (err) {
-      this.logger.error('Event:Chat Resolved [identify] error', err);
-    }
-
-    try {
-      window['analytics'].track('Chat Resolved', {
-        "username": user.firstname + ' ' + user.lastname,
-        "userId": user.uid,
-        "conversation_id": conversation.uid,
-        "channel_type": conversation.channel_type,
-        "conversation_with":(conversation.channel_type !== TYPE_DIRECT)? null: conversation.conversation_with,
-        "department_name":(conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.departmentName: null,
-        "department_id":(conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.departmentId: null
-      },
-      { "context": {
-          "groupId": (conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.projectId: null
-        }
-      });
-    } catch (err) {
-      this.logger.error('Event:Chat Resolved [track] error', err);
-    }
-
-    if(conversation.channel_type !== TYPE_DIRECT){
+    if( window['analytics']){
       try {
-        window['analytics'].group(conversation.attributes.projectId, {
-          name: (conversation.attributes.project_name)? conversation.attributes.project_name : null,
-          // plan: projectProfileName,
+        window['analytics'].page("Chat List Conversations Page, Chat Resolved", {});
+      } catch (err) {
+        this.logger.error('Event:Chat Resolved [page] error', err);
+      }
+  
+      try {
+        window['analytics'].identify(user.uid, {
+          name: user.firstname + ' ' + user.lastname,
+          email: user.email,
+          logins: 5,
+  
+        });
+      } catch (err) {
+        this.logger.error('Event:Chat Resolved [identify] error', err);
+      }
+  
+      try {
+        window['analytics'].track('Chat Resolved', {
+          "username": user.firstname + ' ' + user.lastname,
+          "userId": user.uid,
+          "conversation_id": conversation.uid,
+          "channel_type": conversation.channel_type,
+          "conversation_with":(conversation.channel_type !== TYPE_DIRECT)? null: conversation.conversation_with,
+          "department_name":(conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.departmentName: null,
+          "department_id":(conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.departmentId: null
+        },
+        { "context": {
+            "groupId": (conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.projectId: null
+          }
         });
       } catch (err) {
-        this.logger.error('Event:Chat Resolved [group] error', err);
+        this.logger.error('Event:Chat Resolved [track] error', err);
+      }
+  
+      if(conversation.channel_type !== TYPE_DIRECT){
+        try {
+          window['analytics'].group(conversation.attributes.projectId, {
+            name: (conversation.attributes.project_name)? conversation.attributes.project_name : null,
+            // plan: projectProfileName,
+          });
+        } catch (err) {
+          this.logger.error('Event:Chat Resolved [group] error', err);
+        }
       }
     }
   }

package/src/app/components/canned-response/canned-response.component.html

@@ -5,8 +5,8 @@
       *ngFor="let canned of tagsCannedFilter; let i = index;"
       (click)="onClickCannedFN(canned, $event)">
       <div class="cannedContent">
-        <ion-input [(ngModel)]="canned.title" class="title" id="{{'titleCanned_'+canned._id}}" [class.readonly]="canned?.disabled"></ion-input>
-        <ion-input [(ngModel)]="canned.text" *ngIf="canned.text" class="text truncate" [class.readonly]="canned?.disabled"></ion-input>
+        <ion-input [(ngModel)]="canned.title" class="title" id="{{'titleCanned_'+canned._id}}" [class.readonly]="canned?.disabled" [readonly]="canned?.disabled"></ion-input>
+        <ion-input [(ngModel)]="canned.text" *ngIf="canned.text" class="text truncate" [class.readonly]="canned?.disabled" [readonly]="canned?.disabled"></ion-input>
       </div>
       <ion-icon class="canned-item-icon"  name="checkmark-sharp" slot=end *ngIf="canned.createdBy === loggedUser.uid && !canned.disabled" (click)="onConfirmEditCanned(canned, $event)"></ion-icon>
       <ion-icon class="canned-item-icon"  name="pencil-sharp" slot=end *ngIf="canned.createdBy === loggedUser.uid && canned.disabled" (click)="onEditCanned(canned, $event)"></ion-icon>

package/src/app/components/canned-response/canned-response.component.ts

@@ -286,6 +286,11 @@ export class CannedResponseComponent implements OnInit {
         // this.replaceTagInMessage(this.tagsCannedFilter[this.arrowkeyLocation])
       }
 
+      // set the focus on current canned 
+      setTimeout(() => {
+        this.el.nativeElement.querySelector('.canned-list').scrollTop = this.arrowkeyLocation * 59 //59 is the height of the single element
+      }, 0);
+
       if (event.key === 'Enter') {
         const canned_selected = this.tagsCannedFilter[this.arrowkeyLocation]
         this.logger.log('[CONVS-DETAIL] replaceTagInMessage  canned_selected ',canned_selected)

package/src/app/components/conversation-detail/message-text-area/message-text-area.component.ts

@@ -699,6 +699,12 @@ export class MessageTextAreaComponent implements OnInit, AfterViewInit, OnChange
     this.logger.log("[CONVS-DETAIL][MSG-TEXT-AREA] handleKeyboardEvent  event.key ", event);
     // Note: on mac keyboard "metakey" matches "cmd"
     
+    //do not move cursor on ArrowDown/ArrowUp
+    if ((event.key === 'ArrowDown' || event.key === 'ArrowUp') && !event.shiftKey) {
+      this.logger.log('[CONVS-DETAIL][MSG-TEXT-AREA] HAS PRESSED event.key', event.key);
+      event.preventDefault();
+    }
+
     if (event.key === 'Enter' && event.altKey || event.key === 'Enter' && event.ctrlKey || event.key === 'Enter' && event.metaKey) {
       this.logger.log('[CONVS-DETAIL][MSG-TEXT-AREA] HAS PRESSED COMBO KEYS this.messageString', this.messageString);
       if (this.messageString !== undefined && this.messageString.trim() !== '') {

package/src/app/components/conversation-info/info-content/info-content.component.ts

@@ -52,8 +52,6 @@ export class InfoContentComponent implements OnInit {
   public IS_GROUP_PANEL: boolean = false
 
   constructor(
-    public archivedConversationsHandlerService: ArchivedConversationsHandlerService,
-    public conversationsHandlerService: ConversationsHandlerService,
     public tiledeskAuthService: TiledeskAuthService,
     private route: ActivatedRoute,
     public contactsService: ContactsService,

package/src/app/pages/conversation-detail/conversation-detail.page.ts

@@ -210,17 +210,13 @@ export class ConversationDetailPage implements OnInit, OnDestroy, AfterViewInit
     public archivedConversationsHandlerService: ArchivedConversationsHandlerService,
     public conversationHandlerService: ConversationHandlerService,
     public groupService: GroupsHandlerService,
-    public contactsService: ContactsService,
     public conversationHandlerBuilderService: ConversationHandlerBuilderService,
     public cannedResponsesService: CannedResponsesService,
-    public imageRepoService: ImageRepoService,
     public presenceService: PresenceService,
     public toastController: ToastController,
     public tiledeskService: TiledeskService,
     private networkService: NetworkService,
     private events: EventsService,
-    private renderer: Renderer2,
-    private el: ElementRef
   ) {
     // Change list on date change
     this.route.paramMap.subscribe((params) => {
@@ -1668,48 +1664,50 @@ export class ConversationDetailPage implements OnInit, OnDestroy, AfterViewInit
 
   segmentNewAgentMessage(conversation: ConversationModel){
     let user = this.loggedUser
-    try {
-      window['analytics'].page("Chat Conversation Detail Page, Message Sent", {});
-    } catch (err) {
-      this.logger.error('Event:Message Sent [page] error', err);
-    }
-
-    try {
-      window['analytics'].identify(user.uid, {
-        name: user.firstname + ' ' + user.lastname,
-        email: user.email,
-        logins: 5,
-      });
-    } catch (err) {
-      this.logger.error('Event:Message Sent [identify] error', err);
-    }
-
-    try {
-      window['analytics'].track('Message Sent', {
-        "username": user.firstname + ' ' + user.lastname,
-        "userId": user.uid,
-        "conversation_id": conversation.uid,
-        "channel_type": conversation.channel_type,
-        "conversation_with": conversation.conversation_with,
-        "department_name":(conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.departmentName: null,
-        "department_id":(conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.departmentId: null,
-      },
-      {
-        "context": {
-          "groupId": (conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.projectId: null
-        }
-      });
-    } catch (err) {
-      this.logger.error('Event:Message Sent [track] error', err);
-    }
-
-    try {
-      window['analytics'].group(conversation.attributes.projectId, {
-        name: (conversation.attributes.project_name)? conversation.attributes.project_name : null,
-        // plan: projectProfileName,
-      });
-    } catch (err) {
-      this.logger.error('Event:Message Sent [group] error', err);
+    if(window['analytics']){
+      try {
+        window['analytics'].page("Chat Conversation Detail Page, Message Sent", {});
+      } catch (err) {
+        this.logger.error('Event:Message Sent [page] error', err);
+      }
+  
+      try {
+        window['analytics'].identify(user.uid, {
+          name: user.firstname + ' ' + user.lastname,
+          email: user.email,
+          logins: 5,
+        });
+      } catch (err) {
+        this.logger.error('Event:Message Sent [identify] error', err);
+      }
+  
+      try {
+        window['analytics'].track('Message Sent', {
+          "username": user.firstname + ' ' + user.lastname,
+          "userId": user.uid,
+          "conversation_id": conversation.uid,
+          "channel_type": conversation.channel_type,
+          "conversation_with": conversation.conversation_with,
+          "department_name":(conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.departmentName: null,
+          "department_id":(conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.departmentId: null,
+        },
+        {
+          "context": {
+            "groupId": (conversation.channel_type !== TYPE_DIRECT)? conversation.attributes.projectId: null
+          }
+        });
+      } catch (err) {
+        this.logger.error('Event:Message Sent [track] error', err);
+      }
+  
+      try {
+        window['analytics'].group(conversation.attributes.projectId, {
+          name: (conversation.attributes.project_name)? conversation.attributes.project_name : null,
+          // plan: projectProfileName,
+        });
+      } catch (err) {
+        this.logger.error('Event:Message Sent [group] error', err);
+      }
     }
   }
   

package/src/app/pages/conversations-list/conversations-list.page.ts

@@ -797,7 +797,7 @@ export class ConversationListPage implements OnInit {
       let pageUrl = 'conversation-detail/' + this.uidConvSelected + '/' + this.conversationSelected.conversation_with_fullname + '/' + converationType
       this.logger.log('[CONVS-LIST-PAGE] pageURL', pageUrl)
         // replace(/\(/g, '%28').replace(/\)/g, '%29') -> used for the encoder of any round brackets
-      this.router.navigateByUrl(pageUrl.replace(/\(/g, '%28').replace(/\)/g, '%29').replace( /#/g, "%23" ))
+      this.router.navigateByUrl(pageUrl.replace(/\(/g, '%28').replace(/\)/g, '%29').replace( /#/g, "%23" ), {replaceUrl: true})
     } else {
       this.logger.log('[CONVS-LIST-PAGE] navigateByUrl this.conversationSelected conversation_with_fullname ', this.conversationSelected.conversation_with_fullname)
       this.logger.log('[CONVS-LIST-PAGE] checkPlatformIsMobile(): ', checkPlatformIsMobile())
@@ -808,10 +808,10 @@ export class ConversationListPage implements OnInit {
       }
       this.logger.log('[CONVS-LIST-PAGE] setUidConvSelected navigateByUrl--->: ', pageUrl)
       // replace(/\(/g, '%28').replace(/\)/g, '%29') -> used for the encoder of any round brackets
-      this.router.navigateByUrl(pageUrl.replace(/\(/g, '%28').replace(/\)/g, '%29').replace( /#/g, "%23" ))
+      this.router.navigateByUrl(pageUrl.replace(/\(/g, '%28').replace(/\)/g, '%29').replace( /#/g, "%23" ), {replaceUrl: true})
     }
   }
-
+  
   // ---------------------------------------------------------
   // Opens the list of contacts for direct convs
   // ---------------------------------------------------------
@@ -979,51 +979,53 @@ export class ConversationListPage implements OnInit {
 
   private segmentNewConversationAdded(conversation: ConversationModel){
     let user = this.tiledeskAuthService.getCurrentUser()
-    try {
-      window['analytics'].page("Chat List Conversations Page, Agent added to conversation", {});
-    } catch (err) {
-      this.logger.error('Event:Agent added to conversation [page] error', err);
-    }
-
-    try {
-      window['analytics'].identify(user.uid, {
-        name: user.firstname + ' ' + user.lastname,
-        email: user.email,
-        logins: 5,
-      });
-    } catch (err) {
-      this.logger.error('Event:Agent added to conversation [identify] error', err);
-    }
-
-    try {
-      window['analytics'].track('Agent added to conversation', {
-        "username": user.firstname + ' ' + user.lastname,
-        "userId": user.uid,
-        "conversation_id": conversation.uid,
-        "channel_type": conversation.channel_type,
-        "conversation_with": conversation.conversation_with,
-        "project_id":(conversation.channel_type !== TYPE_DIRECT)? conversation.uid.split('-')[2]: null,
-        "project_name":(conversation.channel_type !== TYPE_DIRECT && conversation.attributes && conversation.attributes.departmentId)? conversation.attributes.departmentId: null,
-      },
-      {
-        "context": {
-          "groupId": (conversation.channel_type !== TYPE_DIRECT)? conversation.uid.split('-')[2]: null
-        }
-      });
-    } catch (err) {
-      this.logger.error('Event:Agent added to conversation [track] error', err);
-    }
-
-    if(conversation.channel_type !== TYPE_DIRECT){
+    if(window['analytics']){
       try {
-        window['analytics'].group(conversation.uid.split('-')[2], {
-          name: (conversation.attributes && conversation.attributes.project_name)? conversation.attributes.project_name : null,
-          // plan: projectProfileName,
+        window['analytics'].page("Chat List Conversations Page, Agent added to conversation", {});
+      } catch (err) {
+        this.logger.error('Event:Agent added to conversation [page] error', err);
+      }
+  
+      try {
+        window['analytics'].identify(user.uid, {
+          name: user.firstname + ' ' + user.lastname,
+          email: user.email,
+          logins: 5,
         });
       } catch (err) {
-        this.logger.error('Event:Agent added to conversation [group] error', err);
+        this.logger.error('Event:Agent added to conversation [identify] error', err);
+      }
+  
+      try {
+        window['analytics'].track('Agent added to conversation', {
+          "username": user.firstname + ' ' + user.lastname,
+          "userId": user.uid,
+          "conversation_id": conversation.uid,
+          "channel_type": conversation.channel_type,
+          "conversation_with": conversation.conversation_with,
+          "project_id":(conversation.channel_type !== TYPE_DIRECT)? conversation.uid.split('-')[2]: null,
+          "project_name":(conversation.channel_type !== TYPE_DIRECT && conversation.attributes && conversation.attributes.departmentId)? conversation.attributes.departmentId: null,
+        },
+        {
+          "context": {
+            "groupId": (conversation.channel_type !== TYPE_DIRECT)? conversation.uid.split('-')[2]: null
+          }
+        });
+      } catch (err) {
+        this.logger.error('Event:Agent added to conversation [track] error', err);
+      }
+  
+      if(conversation.channel_type !== TYPE_DIRECT){
+        try {
+          window['analytics'].group(conversation.uid.split('-')[2], {
+            name: (conversation.attributes && conversation.attributes.project_name)? conversation.attributes.project_name : null,
+            // plan: projectProfileName,
+          });
+        } catch (err) {
+          this.logger.error('Event:Agent added to conversation [group] error', err);
+        }
+  
       }
-
     }
   }
 

package/src/assets/js/chat21client.js

@@ -1,7 +1,7 @@
 /*
     Chat21Client
 
-    v0.1.11
+    v0.1.12.3
 
     @Author Andrea Sponziello
     (c) Tiledesk 2020
@@ -22,7 +22,7 @@ class Chat21Client {
         this.client = null;
         this.reconnections = 0 // just to check how many reconnections
         this.client_id = this.uuidv4();
-        this.log = options._log ? true : false;
+        this.log = options.log ? true : false;
         if (options && options.MQTTendpoint) {
             if (options.MQTTendpoint.startsWith('/')) {
                 if (this.log) {
@@ -75,7 +75,7 @@ class Chat21Client {
         this.connected = false
     }
 
-    subscribeToMyConversations() { // MESSAGES ETC.
+    subscribeToMyConversations(subscribedCallback) { // MESSAGES ETC.
         // WILDCARS:
         // MQTT: https://www.hivemq.com/blog/mqtt-essentials-part-5-mqtt-topics-best-practices/
         // RABBITMQ: https://www.cloudamqp.com/blog/2015-09-03-part4-rabbitmq-for-beginners-exchanges-routing-keys-bindings.html#topic-exchange
@@ -84,9 +84,13 @@ class Chat21Client {
             console.log("subscribing to:", this.user_id, "topic", this.topic_inbox);
         }
         this.client.subscribe(this.topic_inbox, (err)  => {
+            if (err) {
+                console.error("An error occurred while subscribing user", this.user_id, "on topic:", this.topic_inbox, "Error:", err);
+            }
             if (this.log) {
                 console.log("subscribed to:", this.topic_inbox, " with err", err)
             }
+            subscribedCallback();
         });
     }
 
@@ -127,7 +131,9 @@ class Chat21Client {
         // callback - function (err)
         // console.log("recipient_id:", recipient_id)
         let dest_topic = `apps/${this.appid}/outgoing/users/${this.user_id}/messages/${recipient_id}/outgoing`
-        // console.log("dest_topic:", dest_topic)
+        if (this.log) {
+            console.log("dest_topic:", dest_topic)
+        }
         // let outgoing_message = {
         //     text: text,
         //     type: type,
@@ -529,172 +535,179 @@ class Chat21Client {
         this.onGroupUpdatedCallbacks.delete(handler);
     }
 
-    start() {
+    start(subscribedCallback) {
         if (this.on_message_handler) {
-            console.log("this.on_message_handler already subscribed. Reconnected num", this.reconnections)
+            if (this.log) {
+                console.log("this.on_message_handler already subscribed. Reconnected num", this.reconnections)
+            }
+            callbsubscribedCallbackack();
             return
         }
-        this.subscribeToMyConversations()
-        // no more then one "on_message" handler, thanks.
-        this.on_message_handler = this.client.on('message', (topic, message) => {
-            // console.log("topic:" + topic + "\nmessage payload:" + message)
-            const _topic = this.parseTopic(topic)
-            if (!_topic) {
+        this.subscribeToMyConversations(() => {
+            // no more than one "on_message" handler, thanks.
+            this.on_message_handler = this.client.on('message', (topic, message) => {
                 if (this.log) {
-                    console.log("Invalid message topic:", topic);
+                    console.log("topic:" + topic + "\nmessage payload:" + message)
                 }
-                return;
-            }
-            const conversWith = _topic.conversWith
-            try {
-                const message_json = JSON.parse(message.toString())
-                
+                const _topic = this.parseTopic(topic)
+                if (!_topic) {
+                    if (this.log) {
+                        console.log("Invalid message topic:", topic);
+                    }
+                    return;
+                }
+                const conversWith = _topic.conversWith
+                try {
+                    const message_json = JSON.parse(message.toString())
+                    
 
-                // TEMPORARILY DISABLED, ADDED-CONVERSATIONS ARE OBSERVED BY NEW MESSAGES.
-                // MOVED TO: this.onMessageAddedCallbacks
-                // if (this.onConversationAddedCallbacks) {
-                //     if (topic.includes("/conversations/") && topic.endsWith(_CLIENTADDED)) {
-                //         // map.forEach((value, key, map) =>)
-                //         this.onConversationAddedCallbacks.forEach((callback, handler, map) => {
-                //             callback(message_json, _topic)
-                //         });
-                //     }
-                // }
+                    // TEMPORARILY DISABLED, ADDED-CONVERSATIONS ARE OBSERVED BY NEW MESSAGES.
+                    // MOVED TO: this.onMessageAddedCallbacks
+                    // if (this.onConversationAddedCallbacks) {
+                    //     if (topic.includes("/conversations/") && topic.endsWith(_CLIENTADDED)) {
+                    //         // map.forEach((value, key, map) =>)
+                    //         this.onConversationAddedCallbacks.forEach((callback, handler, map) => {
+                    //             callback(message_json, _topic)
+                    //         });
+                    //     }
+                    // }
 
-                if (this.onConversationUpdatedCallbacks) {
-                    // example topic: apps.tilechat.users.ME.conversations.CONVERS-WITH.clientdeleted
-                    if (topic.includes("/conversations/") && topic.endsWith(_CLIENTUPDATED)) {
-                        if (this.log) {
-                            console.log("conversation updated! /conversations/, topic:", topic)
+                    if (this.onConversationUpdatedCallbacks) {
+                        // example topic: apps.tilechat.users.ME.conversations.CONVERS-WITH.clientdeleted
+                        if (topic.includes("/conversations/") && topic.endsWith(_CLIENTUPDATED)) {
+                            if (this.log) {
+                                console.log("conversation updated! /conversations/, topic:", topic)
+                            }
+                            // map.forEach((value, key, map) =>)
+                            this.onConversationUpdatedCallbacks.forEach((callback, handler, map) => {
+                                callback(JSON.parse(message.toString()), _topic)
+                            });
                         }
-                        // map.forEach((value, key, map) =>)
-                        this.onConversationUpdatedCallbacks.forEach((callback, handler, map) => {
-                            callback(JSON.parse(message.toString()), _topic)
-                        });
                     }
-                }
 
-                if (this.onConversationDeletedCallbacks) {
-                    if (topic.includes("/conversations/") && topic.endsWith(_CLIENTDELETED)) {
-                        // map.forEach((value, key, map) =>)
-                        if (this.log) {
-                            console.log("conversation deleted! /conversations/, topic:", topic, message.toString() );
+                    if (this.onConversationDeletedCallbacks) {
+                        if (topic.includes("/conversations/") && topic.endsWith(_CLIENTDELETED)) {
+                            // map.forEach((value, key, map) =>)
+                            if (this.log) {
+                                console.log("conversation deleted! /conversations/, topic:", topic, message.toString() );
+                            }
+                            this.onConversationDeletedCallbacks.forEach((callback, handler, map) => {
+                                callback(JSON.parse(message.toString()), _topic)
+                            });
                         }
-                        this.onConversationDeletedCallbacks.forEach((callback, handler, map) => {
-                            callback(JSON.parse(message.toString()), _topic)
-                        });
                     }
-                }
 
-                if (this.onArchivedConversationAddedCallbacks) {
-                    if (topic.includes("/archived_conversations/") && topic.endsWith(_CLIENTADDED)) {
-                        // map.forEach((value, key, map) =>)
-                        this.onArchivedConversationAddedCallbacks.forEach((callback, handler, map) => {
-                            callback(JSON.parse(message.toString()), _topic)
-                        });
+                    if (this.onArchivedConversationAddedCallbacks) {
+                        if (topic.includes("/archived_conversations/") && topic.endsWith(_CLIENTADDED)) {
+                            // map.forEach((value, key, map) =>)
+                            this.onArchivedConversationAddedCallbacks.forEach((callback, handler, map) => {
+                                callback(JSON.parse(message.toString()), _topic)
+                            });
+                        }
                     }
-                }
 
-                if (this.onArchivedConversationDeletedCallbacks) {
-                    if (topic.includes("/archived_conversations/") && topic.endsWith(_CLIENTDELETED)) {
-                        // map.forEach((value, key, map) =>)
-                        this.onArchivedConversationDeletedCallbacks.forEach((callback, handler, map) => {
-                            callback(JSON.parse(message.toString()), _topic)
-                        });
+                    if (this.onArchivedConversationDeletedCallbacks) {
+                        if (topic.includes("/archived_conversations/") && topic.endsWith(_CLIENTDELETED)) {
+                            // map.forEach((value, key, map) =>)
+                            this.onArchivedConversationDeletedCallbacks.forEach((callback, handler, map) => {
+                                callback(JSON.parse(message.toString()), _topic)
+                            });
+                        }
                     }
-                }
 
-                // *********************************************************
-                // This snippet is important to get all messages and notify
-                // conversation > added (to create a conversation entry)
-                // *********************************************************
-                // if (this.onMessageAddedCallbacks) {
-                //     console.log("ttttttttt")
-                if (topic.includes("/messages/") && topic.endsWith(_CLIENTADDED)) {
-                    if (this.onMessageAddedCallbacks) {
-                        this.onMessageAddedCallbacks.forEach((callback, handler, map) => {
-                            callback(JSON.parse(message.toString()), _topic)
-                        });
-                    }
-                    // Observing conversations added from messages
-                    // console.log("Observing conversations added from messages", message_json);
-                    // if (this.onConversationAddedCallbacks) {
-                    // console.log("callbacks ok........");
-                    let update_conversation = true;
-                    if (message_json.attributes && message_json.attributes.updateconversation == false) {
-                        update_conversation = false
-                    }
-                    // console.log("update_conversation........", update_conversation);
-                    if (update_conversation && this.onConversationAddedCallbacks) {
-                        this.onConversationAddedCallbacks.forEach((callback, handler, map) => {
-                            message_json.is_new = true;
-                            const message_for_conv_string = JSON.stringify(message_json);
-                            callback(JSON.parse(message_for_conv_string), _topic)
-                        });
+                    // *********************************************************
+                    // This snippet is important to get all messages and notify
+                    // conversation > added (to create a conversation entry)
+                    // *********************************************************
+                    // if (this.onMessageAddedCallbacks) {
+                    //     console.log("ttttttttt")
+                    if (topic.includes("/messages/") && topic.endsWith(_CLIENTADDED)) {
+                        if (this.onMessageAddedCallbacks) {
+                            this.onMessageAddedCallbacks.forEach((callback, handler, map) => {
+                                callback(JSON.parse(message.toString()), _topic)
+                            });
+                        }
+                        // Observing conversations added from messages
+                        // console.log("Observing conversations added from messages", message_json);
+                        // if (this.onConversationAddedCallbacks) {
+                        let update_conversation = true;
+                        
+                        if (message_json.attributes && message_json.attributes.updateconversation == false) {
+                            update_conversation = false
+                        }
+                        if (update_conversation && this.onConversationAddedCallbacks) {
+                            this.onConversationAddedCallbacks.forEach((callback, handler, map) => {
+                                message_json.is_new = true;
+                                const message_for_conv_string = JSON.stringify(message_json);
+                                callback(JSON.parse(message_for_conv_string), _topic)
+                            });
+                        }
+                        // }
                     }
                     // }
-                }
-                // }
 
-                if (this.onMessageUpdatedCallbacks) {
-                    if (topic.includes("/messages/") && topic.endsWith(_CLIENTUPDATED)) {
-                        this.onMessageUpdatedCallbacks.forEach((callback, handler, map) => {
-                            callback(JSON.parse(message.toString()), _topic)
-                        });
+                    if (this.onMessageUpdatedCallbacks) {
+                        if (topic.includes("/messages/") && topic.endsWith(_CLIENTUPDATED)) {
+                            this.onMessageUpdatedCallbacks.forEach((callback, handler, map) => {
+                                callback(JSON.parse(message.toString()), _topic)
+                            });
+                        }
                     }
-                }
 
-                if (this.onGroupUpdatedCallbacks) {
-                    if (topic.includes("/groups/") && topic.endsWith(_CLIENTUPDATED)) {
-                        this.onGroupUpdatedCallbacks.forEach((callback, handler, map) => {
-                            callback(JSON.parse(message.toString()), _topic)
-                        });
+                    if (this.onGroupUpdatedCallbacks) {
+                        if (topic.includes("/groups/") && topic.endsWith(_CLIENTUPDATED)) {
+                            this.onGroupUpdatedCallbacks.forEach((callback, handler, map) => {
+                                callback(JSON.parse(message.toString()), _topic)
+                            });
+                        }
                     }
-                }
 
-                // // ******* NEW!!
-                this.callbackHandlers.forEach((value, key, map) => {
-                    const callback_obj = value
-                    // callback_obj = {
-                    //     "type": "onMessageUpdatedForConversation",
-                    //     "conversWith": conversWith,
-                    //     "callback": callback
-                    // }
-                    const type = callback_obj.type
-                    if (topic.includes("/messages/") && topic.endsWith(_CLIENTADDED)) {
-                        if (this.log) { console.log("/messages/_CLIENTADDED") }
-                        if (type === CALLBACK_TYPE_ON_MESSAGE_ADDED_FOR_CONVERSATION) {
-                            if (conversWith === callback_obj.conversWith) {
-                                if (this.log) { console.log("/messages/_CLIENTADDED on: ", conversWith)}
-                                callback_obj.callback(JSON.parse(message.toString()), _topic)
+                    // // ******* NEW!!
+                    this.callbackHandlers.forEach((value, key, map) => {
+                        const callback_obj = value
+                        // callback_obj = {
+                        //     "type": "onMessageUpdatedForConversation",
+                        //     "conversWith": conversWith,
+                        //     "callback": callback
+                        // }
+                        const type = callback_obj.type
+                        if (topic.includes("/messages/") && topic.endsWith(_CLIENTADDED)) {
+                            if (this.log) { console.log("/messages/_CLIENTADDED") }
+                            if (type === CALLBACK_TYPE_ON_MESSAGE_ADDED_FOR_CONVERSATION) {
+                                if (conversWith === callback_obj.conversWith) {
+                                    if (this.log) { console.log("/messages/_CLIENTADDED on: ", conversWith)}
+                                    callback_obj.callback(JSON.parse(message.toString()), _topic)
+                                }
                             }
                         }
-                    }
-                    if (topic.includes("/messages/") && topic.endsWith(_CLIENTUPDATED)) {
-                        if (this.log) {console.log("/messages/_CLIENTUPDATED")}
-                        if (type === CALLBACK_TYPE_ON_MESSAGE_UPDATED_FOR_CONVERSATION) {
-                            if (conversWith === callback_obj.conversWith) {
-                                console.log("/messages/_CLIENTUPDATED on: ", conversWith)
-                                callback_obj.callback(JSON.parse(message.toString()), _topic)
+                        if (topic.includes("/messages/") && topic.endsWith(_CLIENTUPDATED)) {
+                            if (this.log) {console.log("/messages/_CLIENTUPDATED")}
+                            if (type === CALLBACK_TYPE_ON_MESSAGE_UPDATED_FOR_CONVERSATION) {
+                                if (conversWith === callback_obj.conversWith) {
+                                    if (this.log) {console.log("/messages/_CLIENTUPDATED on: ", conversWith);}
+                                    callback_obj.callback(JSON.parse(message.toString()), _topic)
+                                }
                             }
                         }
-                    }
-                })
-                
-                // if (topic.includes("/messages/") && topic.endsWith(_CLIENTUPDATED)) {
-                //     this.onMessageUpdatedInConversationCallbacks.forEach((obj, handler, map) => {
-                //         if (conversWith === obj.conversWith) {
-                //             callback(message_json, _topic)
-                //         }
-                //     });
-                // }
-                
+                    })
+                    
+                    // if (topic.includes("/messages/") && topic.endsWith(_CLIENTUPDATED)) {
+                    //     this.onMessageUpdatedInConversationCallbacks.forEach((obj, handler, map) => {
+                    //         if (conversWith === obj.conversWith) {
+                    //             callback(message_json, _topic)
+                    //         }
+                    //     });
+                    // }
+                    
 
-            }
-            catch (err) {
-                console.error("ERROR:", err)
-            }
+                }
+                catch (err) {
+                    console.error("ERROR:", err)
+                }
+            })
+            subscribedCallback();
         })
+        
         // console.log("HANDLER_:", this.on_message_handler)
     }
 
@@ -763,14 +776,23 @@ class Chat21Client {
     }
 
     conversationDetail(conversWith, callback) {
+        if (this.log) {
+            console.log("conversationDetail(). searching on user:", this.user_id, " - conversWith:", conversWith)
+        }
         this.crossConversationDetail(conversWith, false, callback);
     }
 
     archivedConversationDetail(conversWith, callback) {
+        if (this.log) {
+            console.log("archivedConversationDetail(). searching on user:", this.user_id, " - conversWith:", conversWith)
+        }
         this.crossConversationDetail(conversWith, true, callback);
     }
 
     crossConversationDetail(conversWith, archived, callback) {
+        if (this.log) {
+            console.log("searching on user:", this.user_id, " - conv of conversWith:", conversWith, " - archived:", archived)
+        }
         let path = "conversations";
         if (archived) {
             path = "archived_conversations"
@@ -778,8 +800,10 @@ class Chat21Client {
         // ex.: http://localhost:8004/tilechat/04-ANDREASPONZIELLO/conversations/CONVERS_WITH
         //const URL = `${this.APIendpoint}/${this.appid}/${this.user_id}/conversations/${conversWith}`
         const URL = `${this.APIendpoint}/${this.appid}/${this.user_id}/${path}/${conversWith}`
-        console.log("getting conversation detail:", URL)
-        console.log("conversWith:", conversWith)
+        if (this.log) {
+            console.log("getting conversation detail:", URL);
+            console.log("conversWith:", conversWith);
+        }
         
         let options = {
             url: URL,
@@ -790,7 +814,9 @@ class Chat21Client {
             method: 'GET'
         }
         Chat21Client.myrequest(options, (err, response, json) => {
-            console.log("JSON...", json)
+            if (this.log) {
+                console.log("JSON...", json);
+            }
             if (json && json.result && Array.isArray(json.result) && json.result.length ==1) {
                 callback(null, json.result[0]);
             }
@@ -964,12 +990,13 @@ class Chat21Client {
         
         this.client.on('connect', // TODO if token is wrong it must reply with an error!
             () => {
-                if (this.log) {console.log("chat client connected...")}
+                if (this.log) {console.log("Chat client connected. User:" + user_id)}
                 if (!this.connected) {
-                    if (this.log) {console.log("Chat client first connection.")}
+                    if (this.log) {console.log("Chat client first connection for:" + user_id)}
                     this.connected = true
-                    this.start()
-                    callback()
+                    this.start( () => {
+                        callback();
+                    });
                 }
             }
         );