Skip to content

Commit

Permalink
Merged PR 48338: Remove chat teams button from settings
Browse files Browse the repository at this point in the history
## What's being changed

The UI of the Chat Settings page - we removed the 'Configure Chat Teams' link.

## Why it's being changed

This button was a bit misleading because it landed the merchant/user directly in the general user management area of Dotdigital. It pertained to Chat only in so far as it allowed the enabling of user permissions for Chat; linking to the Chat team customisation area wasn't possible before because this was dependent on a team being created in the first place.

In any case we're now replacing it with a link to our Help Centre that explains everything.

## How to review / test this change

- Clear cache
- View link
- Visit link

Related work items: #227329
  • Loading branch information
sta1r committed Oct 16, 2023
1 parent c6205a9 commit 0bb7441
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 35 deletions.
16 changes: 0 additions & 16 deletions Block/Adminhtml/Config/Settings/ConfigureTeamsButton.php

This file was deleted.

12 changes: 0 additions & 12 deletions Model/Config.php
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ class Config
public const XML_PATH_LIVECHAT_API_HOST = 'chat_api_credentials/settings/api_host';
public const XML_PATH_LIVECHAT_API_TOKEN = 'chat_api_credentials/credentials/api_token';
public const CHAT_PORTAL_URL = 'WebChat';
public const CHAT_CONFIGURE_TEAM_PATH = 'team/users/all';
public const CHAT_CONFIGURE_WIDGET_PATH = 'account/chat-settings';
public const MAGENTO_PROFILE_CALLBACK_ROUTE = 'ec_chat/profile?isAjax=true';

Expand Down Expand Up @@ -201,17 +200,6 @@ public function getChatPortalUrl(): string
return $this->emailConfig->getRegionAwarePortalUrl() . self::CHAT_PORTAL_URL;
}

/**
* Get configuration chat team url
*
* @return string
* @throws LocalizedException
*/
public function getConfigureChatTeamUrl(): string
{
return $this->emailConfig->getRegionAwarePortalUrl() . self::CHAT_CONFIGURE_TEAM_PATH;
}

/**
* Get configuration chat widget url
*
Expand Down
8 changes: 1 addition & 7 deletions etc/adminhtml/system.xml
Original file line number Diff line number Diff line change
Expand Up @@ -22,13 +22,7 @@
<field id="enabled">1</field>
</depends>
<frontend_model>Dotdigitalgroup\Chat\Block\Adminhtml\Config\Settings\ConfigureWidgetButton</frontend_model>
</field>
<field id="configure_chat_teams" translate="label" sortOrder="13" showInDefault="1" showInWebsite="1" showInStore="0">
<label>Configure Chat Teams</label>
<depends>
<field id="enabled">1</field>
</depends>
<frontend_model>Dotdigitalgroup\Chat\Block\Adminhtml\Config\Settings\ConfigureTeamsButton</frontend_model>
<comment><![CDATA[Learn how to set up Chat by reading our <a href="https://support.dotdigital.com/en/articles/8409160-set-up-chat-in-magento-open-source-and-adobe-commerce" target="_blank">Help Centre</a> articles.]]></comment>
</field>
</group>
</section>
Expand Down

0 comments on commit 0bb7441

Please sign in to comment.