Hide Navigation Tabs if Single connected Account

[Bonapara]

We should hide the navigation tabs that allow switching between different email/calendar accounts when the user has only connected one account:

[greptile-apps]

Modify SettingsAccountsMessageChannelsContainer to conditionally render TabList only if there are multiple email accounts.

export const SettingsAccountsMessageChannelsContainer = () => {
  // ... existing code ...

  if (!messageChannels.length) {
    return <SettingsAccountsListEmptyStateCard />;
  }

  const hasMultipleAccounts = accounts.length > 1;

  return (
    <>
      {hasMultipleAccounts && (
        <TabList
          tabListId={SETTINGS_ACCOUNT_MESSAGE_CHANNELS_TAB_LIST_COMPONENT_ID}
          tabs={tabs}
        />
      )}
      {messageChannels.map((messageChannel) => (
        <React.Fragment key={messageChannel.id}>
          {messageChannel.id === activeTabId && (
            <SettingsAccountsMessageChannelDetails
              messageChannel={messageChannel}
            />
          )}
        </React.Fragment>
      ))}
    </>
  );
};

References

/packages/twenty-front/src/modules/settings/accounts/components/SettingsAccountsMessageChannelsContainer.tsx
/packages/twenty-front/src/modules/ui/layout/tab/components/TabList.tsx

About Greptile

This response provides a starting point for your research, not a precise solution.

Help us improve! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.

Ask Greptile · Edit Issue Bot Settings

[adithej]

Hi, I am interested in working on this, please assign me

[Bonapara]

Sure @adithej, thanks for contributing!

[ViditJain123]

I have created a pull request addressing this issue here

[prateekj117]

@charlesBochet CMIIR, but I think we can close this one as the related MR is already merged.

[adithej]

Yes , it's been merged , refer : #6274