[IMP] website: remove extra step of banner tour

Now that we will write tour by theme, this part is useless.
We just keep the old steps that will be not covered by future theme tour.

task-2172208

3 files changed, 7 insertions, 34 deletions

diff --git a/addons/website/static/src/js/tours/banner.js b/addons/website/static/src/js/tours/homepage.js
index 112c4c64b6f..f7da2401ed1 100644
--- a/addons/website/static/src/js/tours/banner.js
+++ b/addons/website/static/src/js/tours/homepage.js
@@ -1,4 +1,4 @@
-odoo.define("website.tour.banner", function (require) {
+odoo.define("website.tour.homepage", function (require) {
 "use strict";
 
 var core = require("web.core");
@@ -6,7 +6,7 @@ var tour = require("web_tour.tour");
 
 var _t = core._t;
 
-tour.register("banner", {
+tour.register("homepage", {
     url: "/",
 }, [{
     trigger: "a[data-action=edit]",
@@ -82,26 +82,3 @@ tour.register("banner", {
     position: "bottom",
 }]);
 });
-
-//==============================================================================
-
-odoo.define("website.tour.contact", function (require) {
-"use strict";
-
-var core = require("web.core");
-var tour = require("web_tour.tour");
-var _t = core._t;
-
-tour.register("contact", {
-    url: "/page/contactus",
-}, [{
-    trigger: "li#customize-menu",
-    content: _t("<b>Install a contact form</b> to improve this page."),
-    extra_trigger: "#o_contact_mail",
-    position: "bottom",
-}, {
-    trigger: "li#install_apps",
-    content: _t("<b>Install new apps</b> to get more features. Let's install the <i>'Contact form'</i> app."),
-    position: "bottom",
-}]);
-});
diff --git a/addons/website/tests/test_ui.py b/addons/website/tests/test_ui.py
index 88643ec593a..9dab4e6583b 100644
--- a/addons/website/tests/test_ui.py
+++ b/addons/website/tests/test_ui.py
@@ -110,10 +110,7 @@ class TestUiTranslate(odoo.tests.HttpCase):
 @odoo.tests.common.tagged('post_install', '-at_install')
 class TestUi(odoo.tests.HttpCase):
 
-    def test_01_admin_tour_banner(self):
-        self.start_tour("/", 'banner', login='admin', step_delay=100)
-
-    def test_02_restricted_editor(self):
+    def test_01_restricted_editor(self):
         self.restricted_editor = self.env['res.users'].create({
             'name': 'Restricted Editor',
             'login': 'restricted',
@@ -125,10 +122,10 @@ class TestUi(odoo.tests.HttpCase):
         })
         self.start_tour("/", 'restricted_editor', login='restricted')
 
-    def test_03_backend_dashboard(self):
+    def test_02_backend_dashboard(self):
         self.start_tour("/", 'backend_dashboard', login='admin')
 
-    def test_04_website_navbar_menu(self):
+    def test_03_website_navbar_menu(self):
         website = self.env['website'].search([], limit=1)
         self.env['website.menu'].create({
             'name': 'Test Tour Menu',
@@ -139,7 +136,7 @@ class TestUi(odoo.tests.HttpCase):
         })
         self.start_tour("/", 'website_navbar_menu')
 
-    def test_05_specific_website_editor(self):
+    def test_04_specific_website_editor(self):
         website_default = self.env['website'].search([], limit=1)
         new_website = self.env['website'].create({'name': 'New Website'})
         website_editor_assets_view = self.env.ref('website.assets_wysiwyg')
diff --git a/addons/website/views/assets.xml b/addons/website/views/assets.xml
index 726328e49f4..8562ec5e7c1 100644
--- a/addons/website/views/assets.xml
+++ b/addons/website/views/assets.xml
@@ -160,8 +160,7 @@
     <script type="text/javascript" src="/website/static/src/js/menu/new_content.js"/>
     <script type="text/javascript" src="/website/static/src/js/menu/seo.js"/>
     <script type="text/javascript" src="/website/static/src/js/menu/translate.js"/>
-
-    <script type="text/javascript" src="/website/static/src/js/tours/banner.js"/>
+    <script type="text/javascript" src="/website/static/src/js/tours/homepage.js"/>
 
     <script type="text/javascript" src="/website/static/src/js/widgets/ace.js"/>
 </template>