Merge branch 'develop' into fix/article-view

When moving form using Gmail Legacy auth to using OAuth, we need the
email address that will be used to connect. This is because we need to
store this email address in the cache and reuse when we get the callback
to find the associated inbox.

However there are cases where the imap login might be
`support@company.com` and the email used to communicate will be
`contact@company.com` (Probably an alias) In that case, we need to send
the correct email address to Chatwoot when re-authenticating

At the moment, we used the inbox email. This PR adds a check that
defaults to to `imap_login` if that is available and imap is enabled

This PR also fixes an unrelated problem where the email inbox creation
flow was not working

---

Tested it, it is working correctly

![CleanShot 2024-10-09 at 14 23
47@2x](https://github.com/user-attachments/assets/0e2cb6c8-1224-4b45-b34a-7b19611249bc)

app/controllers/google/callbacks_controller.rb

@@ -1,6 +1,15 @@
 class Google::CallbacksController < OauthCallbackController
   include GoogleConcern
 
+  def find_channel_by_email
+    # find by imap_login first, and then by email
+    # this ensures the legacy users can migrate correctly even if inbox email address doesn't match
+    imap_channel = Channel::Email.find_by(imap_login: users_data['email'], account: account)
+    return imap_channel if imap_channel
+
+    Channel::Email.find_by(email: users_data['email'], account: account)
+  end
+
   private
 
   def provider_name

app/controllers/oauth_callback_controller.rb

@@ -25,7 +25,7 @@ class OauthCallbackController < ApplicationController
   end
 
   def find_or_create_inbox
-    channel_email = Channel::Email.find_by(email: users_data['email'], account: account)
+    channel_email = find_channel_by_email
     # we need this value to know where to redirect on sucessful processing of the callback
     channel_exists = channel_email.present?
 
@@ -39,6 +39,10 @@ class OauthCallbackController < ApplicationController
     [channel_email.inbox, channel_exists]
   end
 
+  def find_channel_by_email
+    Channel::Email.find_by(email: users_data['email'], account: account)
+  end
+
   def update_channel(channel_email)
     channel_email.update!({
                             imap_login: users_data['email'], imap_address: imap_address,

app/javascript/dashboard/App.vue

@@ -13,6 +13,7 @@ import { useStore } from 'dashboard/composables/store';
 import WootSnackbarBox from './components/SnackbarContainer.vue';
 import { setColorTheme } from './helper/themeHelper';
 import { isOnOnboardingView } from 'v3/helpers/RouteHelper';
+import { useAccount } from 'dashboard/composables/useAccount';
 import {
   registerSubscription,
   verifyServiceWorkerExistence,
@@ -35,8 +36,9 @@ export default {
   setup() {
     const router = useRouter();
     const store = useStore();
+    const { accountId } = useAccount();
 
-    return { router, store };
+    return { router, store, currentAccountId: accountId };
   },
   data() {
     return {
@@ -52,11 +54,7 @@ export default {
       currentUser: 'getCurrentUser',
       authUIFlags: 'getAuthUIFlags',
       accountUIFlags: 'accounts/getUIFlags',
-      // currentAccountId: 'getCurrentAccountId',
     }),
-    currentAccountId() {
-      return this.$route.params.accountId;
-    },
     hasAccounts() {
       const { accounts = [] } = this.currentUser || {};
       return accounts.length > 0;
@@ -72,23 +70,19 @@ export default {
         this.showAddAccountModal = true;
       }
     },
-    currentAccountId() {
-      console.log('THIS CALL', this.currentAccountId);
-      if (this.currentAccountId) {
-        this.initializeAccount();
-      } else {
-        console.log('NO ACCOUNT ID');
-      }
+    currentAccountId: {
+      immediate: true,
+      handler() {
+        if (this.currentAccountId) {
+          this.initializeAccount();
+        }
+      },
     },
   },
-  created() {
-    this.setLocale(window.chatwootConfig.selectedLocale);
-  },
-  async mounted() {
-    this.setLocale(window.chatwootConfig.selectedLocale);
+  mounted() {
     this.initializeColorTheme();
     this.listenToThemeChanges();
-    await this.$store.dispatch('accounts/get');
+    this.setLocale(window.chatwootConfig.selectedLocale);
   },
   unmounted() {
     if (this.reconnectService) {
@@ -104,7 +98,7 @@ export default {
       mql.onchange = e => setColorTheme(e.matches);
     },
     setLocale(locale) {
-      this.$i18n.locale = locale;
+      this.$root.$i18n.locale = locale;
     },
     async initializeAccount() {
       await this.$store.dispatch('accounts/get');

app/javascript/dashboard/components/layout/Sidebar.vue

@@ -2,6 +2,7 @@
 import { mapGetters } from 'vuex';
 import { getSidebarItems } from './config/default-sidebar';
 import { useKeyboardEvents } from 'dashboard/composables/useKeyboardEvents';
+import { useAccount } from 'dashboard/composables/useAccount';
 import { useRoute, useRouter } from 'vue-router';
 
 import PrimarySidebar from './sidebarComponents/Primary.vue';
@@ -33,6 +34,7 @@ export default {
   setup(props, { emit }) {
     const route = useRoute();
     const router = useRouter();
+    const { accountId } = useAccount();
 
     const toggleKeyShortcutModal = () => {
       emit('openKeyShortcutModal');
@@ -72,6 +74,7 @@ export default {
 
     return {
       toggleKeyShortcutModal,
+      accountId,
     };
   },
   data() {
@@ -82,7 +85,6 @@ export default {
 
   computed: {
     ...mapGetters({
-      accountId: 'getCurrentAccountId',
       currentUser: 'getCurrentUser',
       globalConfig: 'globalConfig/get',
       inboxes: 'inboxes/getInboxes',
@@ -120,7 +122,6 @@ export default {
         this.currentUser,
         this.accountId
       );
-
       const menuItems = this.sideMenuConfig.primaryMenu;
       return menuItems.filter(menuItem => {
         const isAvailableForTheUser = hasPermissions(
@@ -131,7 +132,6 @@ export default {
         if (!isAvailableForTheUser) {
           return false;
         }
-
         if (
           menuItem.alwaysVisibleOnChatwootInstances &&
           !this.isACustomBrandedInstance
@@ -139,12 +139,10 @@ export default {
           return true;
         }
         if (menuItem.featureFlag) {
-          const isEnabled = this.isFeatureEnabledonAccount(
+          return this.isFeatureEnabledonAccount(
             this.accountId,
             menuItem.featureFlag
           );
-
-          return isEnabled;
         }
         return true;
       });

app/javascript/dashboard/components/layout/sidebarComponents/Primary.vue

@@ -50,9 +50,6 @@ export default {
       showOptionsMenu: false,
     };
   },
-  mounted() {
-    console.log('MENU ITEMS', this.menuItems);
-  },
   methods: {
     frontendURL,
     toggleOptions() {

app/javascript/dashboard/components/layout/sidebarComponents/PrimaryNavItem.vue

@@ -26,13 +26,6 @@ export default {
       default: false,
     },
   },
-  mounted() {
-    console.log(
-      'MOUNTED: PRIMARY NAV ITEM',
-      this.name,
-      this.$t(`SIDEBAR.${this.name}`)
-    );
-  },
 };
 </script>
 

app/javascript/dashboard/components/widgets/conversation/bubble/Text.vue

@@ -1,5 +1,5 @@
 <script>
-import Letter from 'vue-letter';
+import { Letter } from 'vue-letter';
 import GalleryView from '../components/GalleryView.vue';
 
 export default {

app/javascript/dashboard/composables/spec/useAccount.spec.js

@@ -1,14 +1,15 @@
-import { ref } from 'vue';
-import { describe, it, expect, vi } from 'vitest';
+import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { useAccount } from '../useAccount';
-import { useStoreGetters } from 'dashboard/composables/store';
+import { useRoute } from 'vue-router';
 
-vi.mock('dashboard/composables/store');
+vi.mock('vue-router');
 
 describe('useAccount', () => {
   beforeEach(() => {
-    useStoreGetters.mockReturnValue({
-      getCurrentAccountId: ref(123),
+    useRoute.mockReturnValue({
+      params: {
+        accountId: 123,
+      },
     });
   });
 

app/javascript/dashboard/composables/useAccount.js

@@ -1,18 +1,20 @@
 import { computed } from 'vue';
-import { useStoreGetters } from 'dashboard/composables/store';
+import { useRoute } from 'vue-router';
 
 /**
  * Composable for account-related operations.
  * @returns {Object} An object containing account-related properties and methods.
  */
 export function useAccount() {
-  const getters = useStoreGetters();
-
   /**
    * Computed property for the current account ID.
    * @type {import('vue').ComputedRef<number>}
    */
-  const accountId = computed(() => getters.getCurrentAccountId.value);
+  const route = useRoute();
+
+  const accountId = computed(() => {
+    return Number(route.params.accountId);
+  });
 
   /**
    * Generates an account-scoped URL.

app/javascript/dashboard/routes/dashboard/settings/Wrapper.vue

@@ -29,7 +29,7 @@ const showNewButton = computed(
       button-route="new"
       :icon="icon"
       :header-title="t(headerTitle)"
-      :button-text="headerButtonText ? t(headerButtonText) : ''"
+      :button-text="t(headerButtonText)"
       :show-back-button="showBackButton"
       :back-url="backUrl"
       :show-new-button="showNewButton"

app/javascript/dashboard/routes/dashboard/settings/account/Index.vue

@@ -5,6 +5,7 @@ import { mapGetters } from 'vuex';
 import { useAlert } from 'dashboard/composables';
 import { useUISettings } from 'dashboard/composables/useUISettings';
 import { useConfig } from 'dashboard/composables/useConfig';
+import { useAccount } from 'dashboard/composables/useAccount';
 import { FEATURE_FLAGS } from '../../../../featureFlags';
 import semver from 'semver';
 import { getLanguageDirection } from 'dashboard/components/widgets/conversation/advancedFilterItems/languages';
@@ -13,9 +14,10 @@ export default {
   setup() {
     const { updateUISettings } = useUISettings();
     const { enabledLanguages } = useConfig();
+    const { accountId } = useAccount();
     const v$ = useVuelidate();
 
-    return { updateUISettings, v$, enabledLanguages };
+    return { updateUISettings, v$, enabledLanguages, accountId };
   },
   data() {
     return {
@@ -46,7 +48,6 @@ export default {
       globalConfig: 'globalConfig/get',
       getAccount: 'accounts/getAccount',
       uiFlags: 'accounts/getUIFlags',
-      accountId: 'getCurrentAccountId',
       isFeatureEnabledonAccount: 'accounts/isFeatureEnabledonAccount',
     }),
     showAutoResolutionConfig() {
@@ -95,8 +96,8 @@ export default {
       return this.id.toString();
     },
   },
-  async mounted() {
-    await this.initializeAccount();
+  mounted() {
+    this.initializeAccount();
   },
   methods: {
     async initializeAccount() {
@@ -112,7 +113,7 @@ export default {
           latest_chatwoot_version: latestChatwootVersion,
         } = this.getAccount(this.accountId);
 
-        this.$i18n.locale = locale;
+        this.$root.$i18n.locale = locale;
         this.name = name;
         this.locale = locale;
         this.id = id;
@@ -140,7 +141,7 @@ export default {
           support_email: this.supportEmail,
           auto_resolve_duration: this.autoResolveDuration,
         });
-        this.$i18n.locale = this.locale;
+        this.$root.$i18n.locale = this.locale;
         this.getAccount(this.id).locale = this.locale;
         this.updateDirectionView(this.locale);
         useAlert(this.$t('GENERAL_SETTINGS.UPDATE.SUCCESS'));

app/javascript/dashboard/routes/dashboard/settings/inbox/channels/Email.vue

@@ -47,7 +47,7 @@ const emailProviderList = computed(() => {
 
 function onClick(emailProvider) {
   if (emailProvider.isEnabled) {
-    this.provider = emailProvider.key;
+    provider.value = emailProvider.key;
   }
 }
 </script>

app/javascript/dashboard/routes/dashboard/settings/inbox/channels/google/Reauthorize.vue

@@ -1,5 +1,5 @@
 <script setup>
-import { ref } from 'vue';
+import { ref, computed } from 'vue';
 import InboxReconnectionRequired from '../../components/InboxReconnectionRequired.vue';
 import googleClient from 'dashboard/api/channel/googleClient';
 
@@ -17,11 +17,18 @@ const { t } = useI18n();
 
 const isRequestingAuthorization = ref(false);
 
+const inboxEmail = computed(() => {
+  if (props.inbox.imap_login && props.inbox.imap_enabled) {
+    return props.inbox.imap_login;
+  }
+  return props.inbox.email;
+});
+
 async function requestAuthorization() {
   try {
     isRequestingAuthorization.value = true;
     const response = await googleClient.generateAuthorization({
-      email: props.inbox.email,
+      email: inboxEmail.value,
     });
 
     const {

app/javascript/dashboard/store/modules/accounts.js

@@ -43,11 +43,7 @@ export const getters = {
     return diffDays <= TRIAL_PERIOD_DAYS;
   },
   isFeatureEnabledonAccount: $state => (id, featureName) => {
-    console.log('----------------------------------------------------');
-    console.log('CHECK ENABLED for', featureName, id);
-    console.log($state, findRecordById($state, id));
     const { features = {} } = findRecordById($state, id);
-    console.log('FEATURES', features, features[featureName]);
     return features[featureName] || false;
   },
 };

app/javascript/entrypoints/dashboard.js

@@ -10,7 +10,6 @@ import Multiselect from 'vue-multiselect';
 import { plugin, defaultConfig } from '@formkit/vue';
 import WootSwitch from 'components/ui/Switch.vue';
 import WootWizard from 'components/ui/Wizard.vue';
-import { sync } from 'vuex-router-sync';
 import FloatingVue from 'floating-vue';
 import WootUiKit from 'dashboard/components';
 import App from 'dashboard/App.vue';
@@ -18,6 +17,7 @@ import i18nMessages from 'dashboard/i18n';
 import createAxios from 'dashboard/helper/APIHelper';
 
 import commonHelpers, { isJSONValid } from 'dashboard/helper/commons';
+import { sync } from 'vuex-router-sync';
 import router, { initalizeRouter } from 'dashboard/routes';
 import store from 'dashboard/store';
 import constants from 'dashboard/constants/globals';
@@ -39,10 +39,11 @@ import 'floating-vue/dist/style.css';
 const i18n = createI18n({
   legacy: false, // https://github.com/intlify/vue-i18n/issues/1902
   locale: 'en',
-  fallbackLocale: 'en',
   messages: i18nMessages,
 });
 
+sync(store, router);
+
 const app = createApp(App);
 app.use(i18n);
 app.use(store);
@@ -98,11 +99,9 @@ app.component('fluent-icon', FluentIcon);
 app.directive('resize', vResizeObserver);
 app.directive('on-clickaway', onClickaway);
 
-sync(store, router);
 // load common helpers into js
 commonHelpers();
 window.WOOT_STORE = store;
-window.dashboardApp = app;
 window.WootConstants = constants;
 window.axios = createAxios(axios);
 // [VITE] Disabled this we don't need it, we can use `useEmitter` directly

app/javascript/entrypoints/widget.js

@@ -25,7 +25,6 @@ import {
 const i18n = createI18n({
   legacy: false, // https://github.com/intlify/vue-i18n/issues/1902
   locale: 'en',
-  fallbackLocale: 'en',
   messages: i18nMessages,
 });
 

app/javascript/survey/views/Response.vue

@@ -120,7 +120,7 @@ export default {
       }
     },
     setLocale(locale) {
-      this.$i18n.locale = locale || 'en';
+      this.$root.$i18n.locale = locale || 'en';
     },
   },
 };

app/javascript/v3/App.vue

@@ -31,7 +31,7 @@ export default {
       };
     },
     setLocale(locale) {
-      this.$i18n.locale = locale;
+      this.$root.$i18n.locale = locale;
     },
   },
 };

app/javascript/widget/App.vue

@@ -66,33 +66,6 @@ export default {
   mounted() {
     const { websiteToken, locale, widgetColor } = window.chatwootWebChannel;
     this.setLocale(locale);
-    setTimeout(() => {
-      console.log('SET LOCALE AGAIN 100');
-      this.setLocale(locale);
-    }, 100);
-    setTimeout(() => {
-      console.log('SET LOCALE AGAIN 200');
-      this.setLocale(locale);
-    }, 200);
-
-    setTimeout(() => {
-      console.log('SET LOCALE AGAIN 300');
-      this.setLocale(locale);
-    }, 300);
-
-    setTimeout(() => {
-      console.log('SET LOCALE AGAIN 400');
-      this.setLocale(locale);
-    }, 400);
-
-    setTimeout(() => {
-      console.log('SET LOCALE AGAIN 500');
-      this.setLocale(locale);
-    }, 500);
-    setTimeout(() => {
-      console.log('SET LOCALE YET AGAIN');
-      this.setLocale(locale);
-    }, 5000);
     this.setWidgetColor(widgetColor);
     setHeader(window.authToken);
     if (this.isIFrame) {
@@ -148,7 +121,6 @@ export default {
     },
     setLocale(localeWithVariation) {
       if (!localeWithVariation) return;
-      console.log('SETTING LOCALE HAHAHAHAHA!', localeWithVariation);
       const { enabledLanguages } = window.chatwootWebChannel;
       const localeWithoutVariation = localeWithVariation.split('_')[0];
       const hasLocaleWithoutVariation = enabledLanguages.some(
@@ -159,9 +131,9 @@ export default {
       );
 
       if (hasLocaleWithVariation) {
-        this.$i18n.locale = localeWithVariation;
+        this.$root.$i18n.locale = localeWithVariation;
       } else if (hasLocaleWithoutVariation) {
-        this.$i18n.locale = localeWithoutVariation;
+        this.$root.$i18n.locale = localeWithoutVariation;
       }
     },
     registerUnreadEvents() {

app/javascript/widget/components/HeaderActions.vue

@@ -60,7 +60,12 @@ export default {
         chatwootWebChannel: { websiteToken },
         authToken,
       } = window;
-      popoutChatWindow(origin, websiteToken, this.$i18n.locale, authToken);
+      popoutChatWindow(
+        origin,
+        websiteToken,
+        this.$root.$i18n.locale,
+        authToken
+      );
     },
     closeWindow() {
       if (IFrameHelper.isIFrame()) {

app/javascript/widget/components/PreChat/Form.vue

@@ -37,7 +37,7 @@ export default {
   },
   data() {
     return {
-      locale: this.$i18n.locale,
+      locale: this.$root.$i18n.locale,
       hasErrorInPhoneInput: false,
       message: '',
       formValues: {},

app/javascript/widget/components/template/EmailInput.vue

@@ -75,7 +75,7 @@ export default {
           messageId: this.messageId,
         });
       } catch (error) {
-        console.log(error);
+        // Ignore error
       } finally {
         this.isUpdating = false;
       }

app/javascript/widget/components/template/IntegrationCard.vue

@@ -41,7 +41,7 @@ export default {
           );
         this.dyteAuthToken = authToken;
       } catch (error) {
-        console.log(error);
+        // Ignore Error for now
       } finally {
         this.isLoading = false;
       }

app/javascript/widget/router.js

@@ -34,8 +34,9 @@ export default createRouter({
           component: () => import('./views/Messages.vue'),
         },
         {
-          path: '/article',
+          path: '/article/:link',
           name: 'article-viewer',
+          props: true,
           component: () => import('./views/ArticleViewer.vue'),
         },
       ],

app/javascript/widget/store/modules/contacts.js

@@ -31,7 +31,7 @@ export const actions = {
       const { data } = await ContactsAPI.get();
       commit(SET_CURRENT_USER, data);
     } catch (error) {
-      console.log(error);
+      // Ignore error
     }
   },
   update: async ({ dispatch }, { user }) => {
@@ -39,7 +39,7 @@ export const actions = {
       await ContactsAPI.update(user);
       dispatch('get');
     } catch (error) {
-      console.log(error);
+      // Ignore error
     }
   },
   setUser: async ({ dispatch }, { identifier, user: userObject }) => {
@@ -91,14 +91,14 @@ export const actions = {
     try {
       await ContactsAPI.setCustomAttributes(customAttributes);
     } catch (error) {
-      console.log(error);
+      // Ignore error
     }
   },
   deleteCustomAttribute: async (_, customAttribute) => {
     try {
       await ContactsAPI.deleteCustomAttribute(customAttribute);
     } catch (error) {
-      console.log(error);
+      // Ignore error
     }
   },
 };

app/javascript/widget/store/modules/conversation/actions.js

@@ -25,7 +25,7 @@ export const actions = {
       // Emit event to notify that conversation is created and show the chat screen
       emitter.emit(ON_CONVERSATION_CREATED);
     } catch (error) {
-      console.log(error);
+      // Ignore error
     } finally {
       commit('setConversationUIFlag', { isCreating: false });
     }
@@ -133,7 +133,7 @@ export const actions = {
       commit('conversation/setMetaUserLastSeenAt', lastSeen, { root: true });
       commit('setMissingMessagesInConversation', updatedConversation);
     } catch (error) {
-      console.log(error);
+      // IgnoreError
     }
   },
 
@@ -158,7 +158,7 @@ export const actions = {
     try {
       await toggleTyping(data);
     } catch (error) {
-      console.log(error);
+      // IgnoreError
     }
   },
 
@@ -172,7 +172,7 @@ export const actions = {
       commit('setMetaUserLastSeenAt', lastSeen);
       await setUserLastSeenAt({ lastSeen });
     } catch (error) {
-      console.log(error);
+      // IgnoreError
     }
   },
 
@@ -184,7 +184,7 @@ export const actions = {
     try {
       await setCustomAttributes(customAttributes);
     } catch (error) {
-      console.log(error);
+      // IgnoreError
     }
   },
 
@@ -192,7 +192,7 @@ export const actions = {
     try {
       await deleteCustomAttribute(customAttribute);
     } catch (error) {
-      console.log(error);
+      // IgnoreError
     }
   },
 };

app/javascript/widget/store/modules/conversationAttributes.js

@@ -22,7 +22,7 @@ export const actions = {
       commit(SET_CONVERSATION_ATTRIBUTES, data);
       commit('conversation/setMetaUserLastSeenAt', lastSeen, { root: true });
     } catch (error) {
-      console.log(error);
+      // Ignore error
     }
   },
   update({ commit }, data) {

app/javascript/widget/store/modules/conversationLabels.js

@@ -9,14 +9,14 @@ export const actions = {
     try {
       await conversationLabels.create(label);
     } catch (error) {
-      console.log(error);
+      // Ignore error
     }
   },
   destroy: async (_, label) => {
     try {
       await conversationLabels.destroy(label);
     } catch (error) {
-      console.log(error);
+      // Ignore error
     }
   },
 };

app/javascript/widget/store/modules/events.js

@@ -5,7 +5,7 @@ const actions = {
     try {
       await events.create(name);
     } catch (error) {
-      console.log(error);
+      // Ignore error
     }
   },
 };

app/javascript/widget/store/modules/message.js

@@ -38,7 +38,7 @@ export const actions = {
       );
       dispatch('contacts/get', {}, { root: true });
     } catch (error) {
-      console.log(error);
+      // Ignore error
     }
     commit('toggleUpdateStatus', false);
   },

app/javascript/widget/views/ArticleViewer.vue

@@ -1,16 +1,16 @@
-<script>
+<script setup>
 import IframeLoader from 'shared/components/IframeLoader.vue';
 
-export default {
-  name: 'ArticleViewer',
-  components: {
-    IframeLoader,
+defineProps({
+  link: {
+    type: String,
+    required: true,
   },
-};
+});
 </script>
 
 <template>
   <div class="bg-white h-full">
-    <IframeLoader :url="$route.query.link" />
+    <IframeLoader :url="link" />
   </div>
 </template>

app/javascript/widget/views/Home.vue

@@ -79,7 +79,7 @@ export default {
       }
       this.$router.push({
         name: 'article-viewer',
-        query: { link: linkToOpen },
+        params: { link: linkToOpen },
       });
     },
     viewAllArticles() {

spec/controllers/google/callbacks_controller_spec.rb

@@ -39,8 +39,9 @@ RSpec.describe 'Google::CallbacksController', type: :request do
       expect(Redis::Alfred.get(cache_key)).to be_nil
     end
 
-    it 'creates updates inbox channel config if inbox exists and authentication is successful' do
-      inbox = create(:channel_email, account: account, email: email)&.inbox
+    it 'updates inbox channel config if inbox exists with imap_login and authentication is successful' do
+      channel_email = create(:channel_email, account: account, imap_login: email)
+      inbox = channel_email.inbox
       expect(inbox.channel.provider_config).to eq({})
 
       stub_request(:post, 'https://accounts.google.com/o/oauth2/token')
@@ -50,7 +51,7 @@ RSpec.describe 'Google::CallbacksController', type: :request do
 
       get google_callback_url, params: { code: code }
 
-      expect(response).to redirect_to app_email_inbox_settings_url(account_id: account.id, inbox_id: account.inboxes.last.id)
+      expect(response).to redirect_to app_email_inbox_settings_url(account_id: account.id, inbox_id: inbox.id)
       expect(account.inboxes.count).to be 1
       expect(inbox.channel.reload.provider_config.keys).to include('access_token', 'refresh_token', 'expires_on')
       expect(inbox.channel.reload.provider_config['access_token']).to eq response_body_success[:access_token]

vite.config.ts

@@ -66,7 +66,7 @@ export default defineConfig({
     lib: isLibraryMode
       ? {
           entry: path.resolve(__dirname, './app/javascript/entrypoints/sdk.js'),
-          formats: ['umd'], // UMD format for single file
+          formats: ['cjs'], // CJS format for single file
           name: 'sdk',
         }
       : undefined,