[FIX] mail: discuss: unpin channel fixes

FIX: use the correct API of mail.channel.py to unfollow a channel
i.e.: `execute_command_leave`

FIX: unpinning a channel has immediate effect on the interface
chatWindows are closed, and the inbox is displayed

FIX: unpinning a channel destroys corresponding chatWindows across tabs
It also unpins the channel in other tabs

FIX: python sends is_pinned as part of the channel_info
both at init messaging and in the main bus notification
That way, we can keep track of the synchronization of the pin status
between JS and python

task-2275000
task-2267941

closes odoo/odoo#53703

X-original-commit: d4fcc64acb2fc0c6fd9df505380f665abd8b39c4
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>

1 files changed, 2 insertions, 4 deletions

diff --git a/addons/im_livechat/static/src/models/messaging_initializer/messaging_initializer.js b/addons/im_livechat/static/src/models/messaging_initializer/messaging_initializer.js
index ab9fc72193c..5f356f8146d 100644
--- a/addons/im_livechat/static/src/models/messaging_initializer/messaging_initializer.js
+++ b/addons/im_livechat/static/src/models/messaging_initializer/messaging_initializer.js
@@ -17,11 +17,9 @@ registerInstancePatchModel('mail.messaging_initializer', 'im_livechat/static/src
         this._super(initMessagingData);
         const { channel_livechat = [] } = initMessagingData;
         for (const data of channel_livechat) {
-            this.env.models['mail.thread'].insert(Object.assign(
-                {},
+            this.env.models['mail.thread'].insert(
                 this.env.models['mail.thread'].convertData(data),
-                { isPinned: true }
-            ));
+            );
         }
     },
 });