From bc7bcc20b8ff4b1f3a302f13bc028958876d4c4d Mon Sep 17 00:00:00 2001
From: Pranav Raj S <pranav@chatwoot.com>
Date: Fri, 8 Apr 2022 15:52:39 +0530
Subject: [PATCH] Revert "feat: Adds support for draft in conversation reply
 box (#4205)" (#4425)

This reverts commit 5ea043605120677824ff948ea023f03e191473c2.
---
 .../dashboard/components/app/UpdateBanner.vue | 15 ++--
 .../components/widgets/WootWriter/Editor.vue  |  4 +-
 .../widgets/conversation/ReplyBox.vue         | 82 +------------------
 .../dashboard/helper/localStorage.js          | 43 ++++------
 .../dashboard/store/modules/auth.js           |  9 +-
 .../store/modules/conversations/helpers.js    |  7 --
 .../specs/conversations/helpers.spec.js       |  7 --
 app/javascript/dashboard/store/utils/api.js   |  6 --
 app/javascript/shared/helpers/TimeHelpers.js  | 21 -----
 .../shared/helpers/specs/TimeHelpers.spec.js  | 32 --------
 10 files changed, 24 insertions(+), 202 deletions(-)
 delete mode 100644 app/javascript/shared/helpers/TimeHelpers.js
 delete mode 100644 app/javascript/shared/helpers/specs/TimeHelpers.spec.js

diff --git a/app/javascript/dashboard/components/app/UpdateBanner.vue b/app/javascript/dashboard/components/app/UpdateBanner.vue
index 63c7bd457..2e4b00031 100644
--- a/app/javascript/dashboard/components/app/UpdateBanner.vue
+++ b/app/javascript/dashboard/components/app/UpdateBanner.vue
@@ -12,11 +12,12 @@
 </template>
 <script>
 import Banner from 'dashboard/components/ui/Banner.vue';
-import { LocalStorage, LOCAL_STORAGE_KEYS } from '../../helper/localStorage';
+import LocalStorage from '../../helper/localStorage';
 import { mapGetters } from 'vuex';
 import adminMixin from 'dashboard/mixins/isAdmin';
 
 const semver = require('semver');
+const dismissedUpdates = new LocalStorage('dismissedUpdates');
 
 export default {
   components: {
@@ -56,22 +57,16 @@ export default {
   },
   methods: {
     isVersionNotificationDismissed(version) {
-      const dismissedVersions =
-        LocalStorage.get(LOCAL_STORAGE_KEYS.DISMISSED_UPDATES) || [];
-      return dismissedVersions.includes(version);
+      return dismissedUpdates.get().includes(version);
     },
     dismissUpdateBanner() {
-      let updatedDismissedItems =
-        LocalStorage.get(LOCAL_STORAGE_KEYS.DISMISSED_UPDATES) || [];
+      let updatedDismissedItems = dismissedUpdates.get();
       if (updatedDismissedItems instanceof Array) {
         updatedDismissedItems.push(this.latestChatwootVersion);
       } else {
         updatedDismissedItems = [this.latestChatwootVersion];
       }
-      LocalStorage.set(
-        LOCAL_STORAGE_KEYS.DISMISSED_UPDATES,
-        updatedDismissedItems
-      );
+      dismissedUpdates.store(updatedDismissedItems);
       this.latestChatwootVersion = this.globalConfig.appVersion;
     },
   },
diff --git a/app/javascript/dashboard/components/widgets/WootWriter/Editor.vue b/app/javascript/dashboard/components/widgets/WootWriter/Editor.vue
index 45e8fb86a..1ac0ec333 100644
--- a/app/javascript/dashboard/components/widgets/WootWriter/Editor.vue
+++ b/app/javascript/dashboard/components/widgets/WootWriter/Editor.vue
@@ -148,8 +148,8 @@ export default {
     showCannedMenu(updatedValue) {
       this.$emit('toggle-canned-menu', !this.isPrivate && updatedValue);
     },
-    value(newValue = '', oldValue = '') {
-      if (newValue !== oldValue) {
+    value(newValue = '') {
+      if (newValue !== this.lastValue) {
         const { tr } = this.state;
         if (this.isFormatMode) {
           this.state = createState(
diff --git a/app/javascript/dashboard/components/widgets/conversation/ReplyBox.vue b/app/javascript/dashboard/components/widgets/conversation/ReplyBox.vue
index d153b53aa..99321eff8 100644
--- a/app/javascript/dashboard/components/widgets/conversation/ReplyBox.vue
+++ b/app/javascript/dashboard/components/widgets/conversation/ReplyBox.vue
@@ -137,7 +137,6 @@ import messageFormatterMixin from 'shared/mixins/messageFormatterMixin';
 import { checkFileSizeLimit } from 'shared/helpers/FileHelper';
 import { MAXIMUM_FILE_UPLOAD_SIZE } from 'shared/constants/messages';
 import { BUS_EVENTS } from 'shared/constants/busEvents';
-import { debounce } from 'shared/helpers/TimeHelpers';
 
 import {
   isEscape,
@@ -150,8 +149,6 @@ import inboxMixin from 'shared/mixins/inboxMixin';
 import uiSettingsMixin from 'dashboard/mixins/uiSettings';
 import { DirectUpload } from 'activestorage';
 import { frontendURL } from '../../../helper/URLHelper';
-import { LocalStorage, LOCAL_STORAGE_KEYS } from '../../../helper/localStorage';
-import { trimMessage } from '../../../store/modules/conversations/helpers';
 
 export default {
   components: {
@@ -204,7 +201,6 @@ export default {
       hasSlashCommand: false,
       bccEmails: '',
       ccEmails: '',
-      doAutoSaveDraft: () => {},
     };
   },
   computed: {
@@ -410,19 +406,10 @@ export default {
     profilePath() {
       return frontendURL(`accounts/${this.accountId}/profile/settings`);
     },
-    conversationId() {
-      return this.currentChat.id;
-    },
   },
   watch: {
-    currentChat(conversation, oldConversation) {
+    currentChat(conversation) {
       const { can_reply: canReply } = conversation;
-
-      if (oldConversation.id !== conversation.id) {
-        this.setToDraft(oldConversation.id, this.replyType);
-        this.getFromDraft();
-      }
-
       if (this.isOnPrivateNote) {
         return;
       }
@@ -447,87 +434,22 @@ export default {
         this.mentionSearchKey = '';
         this.showMentions = false;
       }
-      this.doAutoSaveDraft();
-    },
-    replyType(updatedReplyType, oldReplyType) {
-      this.setToDraft(this.conversationId, oldReplyType);
-      this.getFromDraft();
     },
   },
 
   mounted() {
-    this.getFromDraft();
     // Donot use the keyboard listener mixin here as the events here are supposed to be
     // working even if input/textarea is focussed.
     document.addEventListener('keydown', this.handleKeyEvents);
     document.addEventListener('paste', this.onPaste);
 
     this.setCCEmailFromLastChat();
-    this.doAutoSaveDraft = debounce(
-      () => {
-        this.saveDraft(this.conversationId, this.replyType);
-      },
-      5000,
-      false
-    );
   },
   destroyed() {
     document.removeEventListener('keydown', this.handleKeyEvents);
     document.removeEventListener('paste', this.onPaste);
   },
   methods: {
-    getSavedDraftMessages() {
-      return LocalStorage.get(LOCAL_STORAGE_KEYS.DRAFT_MESSAGES) || {};
-    },
-    saveDraft(conversationId, replyType) {
-      if (this.message || this.message === '') {
-        const savedDraftMessages = this.getSavedDraftMessages();
-        const key = `draft-${conversationId}-${replyType}`;
-        const draftToSave = trimMessage(this.message || '');
-        const {
-          [key]: currentDraft,
-          ...restOfDraftMessages
-        } = savedDraftMessages;
-
-        const updatedDraftMessages = draftToSave
-          ? {
-              ...restOfDraftMessages,
-              [key]: draftToSave,
-            }
-          : restOfDraftMessages;
-
-        LocalStorage.set(
-          LOCAL_STORAGE_KEYS.DRAFT_MESSAGES,
-          updatedDraftMessages
-        );
-      }
-    },
-    setToDraft(conversationId, replyType) {
-      this.saveDraft(conversationId, replyType);
-      this.message = '';
-    },
-    getFromDraft() {
-      if (this.conversationId) {
-        try {
-          const key = `draft-${this.conversationId}-${this.replyType}`;
-          const savedDraftMessages = this.getSavedDraftMessages();
-          this.message = `${savedDraftMessages[key] || ''}`;
-        } catch (error) {
-          this.message = '';
-        }
-      }
-    },
-    removeFromDraft() {
-      if (this.conversationId) {
-        const key = `draft-${this.conversationId}-${this.replyType}`;
-        const draftMessages = this.getSavedDraftMessages();
-        const { [key]: toBeRemoved, ...updatedDraftMessages } = draftMessages;
-        LocalStorage.set(
-          LOCAL_STORAGE_KEYS.DRAFT_MESSAGES,
-          updatedDraftMessages
-        );
-      }
-    },
     onPaste(e) {
       const data = e.clipboardData.files;
       if (!this.showRichContentEditor && data.length !== 0) {
@@ -615,7 +537,6 @@ export default {
             messagePayload
           );
           bus.$emit(BUS_EVENTS.SCROLL_TO_MESSAGE);
-          this.removeFromDraft();
         } catch (error) {
           const errorMessage =
             error?.response?.data?.error ||
@@ -687,7 +608,6 @@ export default {
     },
     onBlur() {
       this.isFocused = false;
-      this.saveDraft(this.conversationId, this.replyType);
     },
     onFocus() {
       this.isFocused = true;
diff --git a/app/javascript/dashboard/helper/localStorage.js b/app/javascript/dashboard/helper/localStorage.js
index cbc562417..cc963aa48 100644
--- a/app/javascript/dashboard/helper/localStorage.js
+++ b/app/javascript/dashboard/helper/localStorage.js
@@ -1,32 +1,17 @@
-export const LOCAL_STORAGE_KEYS = {
-  DISMISSED_UPDATES: 'dismissedUpdates',
-  DRAFT_MESSAGES: 'draftMessages',
-};
+class LocalStorage {
+  constructor(key) {
+    this.key = key;
+  }
 
-export const LocalStorage = {
-  clearAll() {
-    window.localStorage.clear();
-  },
+  store(allItems) {
+    localStorage.setItem(this.key, JSON.stringify(allItems));
+    localStorage.setItem(this.key + ':ts', Date.now());
+  }
 
-  get(key) {
-    const value = window.localStorage.getItem(key);
-    try {
-      return typeof value === 'string' ? JSON.parse(value) : value;
-    } catch (error) {
-      return value;
-    }
-  },
-  set(key, value) {
-    if (typeof value === 'object') {
-      window.localStorage.setItem(key, JSON.stringify(value));
-    } else {
-      window.localStorage.setItem(key, value);
-    }
-    window.localStorage.setItem(key + ':ts', Date.now());
-  },
+  get() {
+    let stored = localStorage.getItem(this.key);
+    return JSON.parse(stored) || [];
+  }
+}
 
-  remove(key) {
-    window.localStorage.removeItem(key);
-    window.localStorage.removeItem(key + ':ts');
-  },
-};
+export default LocalStorage;
diff --git a/app/javascript/dashboard/store/modules/auth.js b/app/javascript/dashboard/store/modules/auth.js
index f9a45646a..5b2feef11 100644
--- a/app/javascript/dashboard/store/modules/auth.js
+++ b/app/javascript/dashboard/store/modules/auth.js
@@ -5,12 +5,7 @@ import * as types from '../mutation-types';
 import authAPI from '../../api/auth';
 import createAxios from '../../helper/APIHelper';
 import actionCable from '../../helper/actionCable';
-import {
-  setUser,
-  getHeaderExpiry,
-  clearCookiesOnLogout,
-  clearLocalStorageOnLogout,
-} from '../utils/api';
+import { setUser, getHeaderExpiry, clearCookiesOnLogout } from '../utils/api';
 import { getLoginRedirectURL } from '../../helper/URLHelper';
 
 const state = {
@@ -99,9 +94,9 @@ export const actions = {
         .login(credentials)
         .then(response => {
           commit(types.default.SET_CURRENT_USER);
-          clearLocalStorageOnLogout();
           window.axios = createAxios(axios);
           actionCable.init(Vue);
+
           window.location = getLoginRedirectURL(ssoAccountId, response.data);
           resolve();
         })
diff --git a/app/javascript/dashboard/store/modules/conversations/helpers.js b/app/javascript/dashboard/store/modules/conversations/helpers.js
index 1a24cdde5..be136af54 100644
--- a/app/javascript/dashboard/store/modules/conversations/helpers.js
+++ b/app/javascript/dashboard/store/modules/conversations/helpers.js
@@ -35,10 +35,3 @@ export const applyPageFilters = (conversation, filters) => {
 
   return shouldFilter;
 };
-
-export const trimMessage = (content = '', maxLength = 1024) => {
-  if (content.length > maxLength) {
-    return content.substring(0, maxLength);
-  }
-  return content;
-};
diff --git a/app/javascript/dashboard/store/modules/specs/conversations/helpers.spec.js b/app/javascript/dashboard/store/modules/specs/conversations/helpers.spec.js
index 599826a99..af25f5527 100644
--- a/app/javascript/dashboard/store/modules/specs/conversations/helpers.spec.js
+++ b/app/javascript/dashboard/store/modules/specs/conversations/helpers.spec.js
@@ -1,7 +1,6 @@
 import {
   findPendingMessageIndex,
   applyPageFilters,
-  trimMessage,
 } from '../../conversations/helpers';
 
 const conversationList = [
@@ -120,9 +119,3 @@ describe('#applyPageFilters', () => {
     });
   });
 });
-
-describe('#trimMessage', () => {
-  expect(trimMessage('Hello world', 5)).toEqual('Hello');
-  expect(trimMessage('Hello', 5)).toEqual('Hello');
-  expect(trimMessage(undefined, 5)).toEqual('');
-});
diff --git a/app/javascript/dashboard/store/utils/api.js b/app/javascript/dashboard/store/utils/api.js
index 9077a43c4..37fd68c7b 100644
--- a/app/javascript/dashboard/store/utils/api.js
+++ b/app/javascript/dashboard/store/utils/api.js
@@ -9,7 +9,6 @@ import {
   CHATWOOT_RESET,
   CHATWOOT_SET_USER,
 } from '../../helper/scriptHelpers';
-import { LocalStorage, LOCAL_STORAGE_KEYS } from '../../helper/localStorage';
 
 Cookies.defaults = { sameSite: 'Lax' };
 
@@ -44,15 +43,10 @@ export const clearBrowserSessionCookies = () => {
   Cookies.remove('user');
 };
 
-export const clearLocalStorageOnLogout = () => {
-  LocalStorage.remove(LOCAL_STORAGE_KEYS.DRAFT_MESSAGES);
-};
-
 export const clearCookiesOnLogout = () => {
   window.bus.$emit(CHATWOOT_RESET);
   window.bus.$emit(ANALYTICS_RESET);
   clearBrowserSessionCookies();
-  clearLocalStorageOnLogout();
   const globalConfig = window.globalConfig || {};
   const logoutRedirectLink =
     globalConfig.LOGOUT_REDIRECT_LINK || frontendURL('login');
diff --git a/app/javascript/shared/helpers/TimeHelpers.js b/app/javascript/shared/helpers/TimeHelpers.js
deleted file mode 100644
index 8473b0f5c..000000000
--- a/app/javascript/shared/helpers/TimeHelpers.js
+++ /dev/null
@@ -1,21 +0,0 @@
-// Returns a function, that, as long as it continues to be invoked, will not
-// be triggered. The function will be called after it stops being called for
-// N milliseconds. If `immediate` is passed, trigger the function on the
-// leading edge, instead of the trailing.
-
-export const debounce = (func, delay, immediate) => {
-  let timerId;
-  return (...args) => {
-    const boundFunc = func.bind(this, ...args);
-    clearTimeout(timerId);
-    if (immediate && !timerId) {
-      boundFunc();
-    }
-    const calleeFunc = immediate
-      ? () => {
-          timerId = null;
-        }
-      : boundFunc;
-    timerId = setTimeout(calleeFunc, delay);
-  };
-};
diff --git a/app/javascript/shared/helpers/specs/TimeHelpers.spec.js b/app/javascript/shared/helpers/specs/TimeHelpers.spec.js
deleted file mode 100644
index e6a94b090..000000000
--- a/app/javascript/shared/helpers/specs/TimeHelpers.spec.js
+++ /dev/null
@@ -1,32 +0,0 @@
-import { debounce } from '../TimeHelpers';
-// Tell Jest to mock all timeout functions
-jest.useFakeTimers('modern');
-
-describe('debounce', () => {
-  test('execute just once with immediate false', () => {
-    let func = jest.fn();
-    let debouncedFunc = debounce(func, 1000);
-
-    for (let i = 0; i < 100; i += 1) {
-      debouncedFunc();
-    }
-
-    // Fast-forward time
-    jest.runAllTimers();
-
-    expect(func).toBeCalledTimes(1);
-  });
-
-  test('execute just once with immediate true', () => {
-    let func = jest.fn();
-    let debouncedFunc = debounce(func, 1000, true);
-
-    for (let i = 0; i < 100; i += 1) {
-      debouncedFunc();
-    }
-
-    jest.runAllTimers();
-
-    expect(func).toBeCalledTimes(1);
-  });
-});