From d6ac430a0e488c23868b07bb5af4ad9889d9cd59 Mon Sep 17 00:00:00 2001
From: Amine Ben hammou <amine@mailpoet.com>
Date: Fri, 20 Mar 2020 04:27:35 +0100
Subject: [PATCH] Replace old advanced tab with the new one

---
 assets/js/src/old_settings/tabs.js         |  3 +++
 assets/js/src/settings/components/tabs.tsx | 10 +++++-----
 views/old_settings.html                    |  2 +-
 3 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/assets/js/src/old_settings/tabs.js b/assets/js/src/old_settings/tabs.js
index 1226ee06c1..9336511071 100644
--- a/assets/js/src/old_settings/tabs.js
+++ b/assets/js/src/old_settings/tabs.js
@@ -62,6 +62,9 @@ if (jQuery('#mailpoet_settings').length > 0) {
       if (tab === 'signup') {
         window.location.href = '?page=mailpoet-new-settings#/signup';
       }
+      if (tab === 'advanced') {
+        window.location.href = '?page=mailpoet-new-settings#/advanced';
+      }
       // reset all active tabs
       jQuery('.nav-tab-wrapper a').removeClass('nav-tab-active');
 
diff --git a/assets/js/src/settings/components/tabs.tsx b/assets/js/src/settings/components/tabs.tsx
index 1785562450..871dec7104 100644
--- a/assets/js/src/settings/components/tabs.tsx
+++ b/assets/js/src/settings/components/tabs.tsx
@@ -41,13 +41,13 @@ export default () => {
           {t('wooCommerceTab')}
         </TabLink>
       )}
-      <a
-        className="nav-tab"
-        href="?page=mailpoet-settings#advanced"
-        data-automation-id="settings-advanced-tab"
+      <TabLink
+        name="advanced"
+        current={current}
+        automationId="settings-advanced-tab"
       >
         {t('advancedTab')}
-      </a>
+      </TabLink>
       <TabLink
         name="premium"
         current={current}
diff --git a/views/old_settings.html b/views/old_settings.html
index 396eba1c89..7c2627f6d7 100644
--- a/views/old_settings.html
+++ b/views/old_settings.html
@@ -21,7 +21,7 @@
         <% if is_woocommerce_active %>
           <a class="nav-tab" href="?page=mailpoet-new-settings#/woocommerce" data-automation-id="woocommerce_settings_tab"><%= __('WooCommerce') %></a>
         <% endif %>
-        <a class="nav-tab" href="#advanced" data-automation-id="settings-advanced-tab"><%= __('Advanced') %></a>
+        <a class="nav-tab" href="?page=mailpoet-new-settings#/advanced" data-automation-id="settings-advanced-tab"><%= __('Advanced') %></a>
         <a class="nav-tab nav-tab-reload" href="?page=mailpoet-new-settings#premium" data-automation-id="activation_settings_tab"><%= __('Key Activation') %></a>
       </h2>