From 563ac8b0b71325d0fbc29c6e764ca04547d3e33a Mon Sep 17 00:00:00 2001
From: Ivan Yelizariev <yelizariev@it-projects.info>
Date: Tue, 26 Apr 2016 11:13:10 +0500
Subject: [PATCH 01/10] [FIX] don't repeat Tour after updating

---
 res_partner_mails_count/templates.xml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/res_partner_mails_count/templates.xml b/res_partner_mails_count/templates.xml
index 0541d56..0015e30 100644
--- a/res_partner_mails_count/templates.xml
+++ b/res_partner_mails_count/templates.xml
@@ -51,6 +51,8 @@
             <field name="url" eval="'/web?debug=1&amp;res_partner_mails_count=tutorial#id='+str(ref('base.partner_root'))+'&amp;view_type=form&amp;model=res.partner&amp;/#tutorial_extra.mails_count_tour=true'"/>
             <field name="target">self</field>
         </record>
+    </data>
+    <data noupdate="1">
         <record id="base.open_menu" model="ir.actions.todo">
             <field name="action_id" ref="res_partner_mails_count_tutorial"/>
             <field name="state">open</field>

From 60b1bb64ec3c0ad6fa4c3180be0bc9a2c83eb490 Mon Sep 17 00:00:00 2001
From: Ivan Yelizariev <yelizariev@it-projects.info>
Date: Tue, 26 Apr 2016 11:18:04 +0500
Subject: [PATCH 02/10] [FIX] noupdate must be applied for todo only

---
 res_partner_mails_count/templates.xml | 2 --
 1 file changed, 2 deletions(-)

diff --git a/res_partner_mails_count/templates.xml b/res_partner_mails_count/templates.xml
index 41e59b0..c705f6a 100644
--- a/res_partner_mails_count/templates.xml
+++ b/res_partner_mails_count/templates.xml
@@ -30,8 +30,6 @@
                 </div>
             </field>
         </record>
-    </data>
-    <data noupdate="true">
         <record id="res_partner_mails_count_tutorial" model="ir.actions.act_url">
             <field name="name">res_partner_mails_count Tutorial</field>
             <field name="url" eval="'/web?debug=1&amp;res_partner_mails_count=tutorial#id='+str(ref('base.partner_root'))+'&amp;view_type=form&amp;model=res.partner&amp;/#tutorial_extra.mails_count_tour=true'"/>

From c0396830ce384fa02383085d6ce7d94f7d88230a Mon Sep 17 00:00:00 2001
From: Ivan Yelizariev <yelizariev@it-projects.info>
Date: Tue, 26 Apr 2016 11:23:21 +0500
Subject: [PATCH 03/10] [FIX] don't switch to debug in tutor

---
 res_partner_mails_count/templates.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/res_partner_mails_count/templates.xml b/res_partner_mails_count/templates.xml
index 0015e30..9910d6c 100644
--- a/res_partner_mails_count/templates.xml
+++ b/res_partner_mails_count/templates.xml
@@ -48,7 +48,7 @@
         </record>
         <record id="res_partner_mails_count_tutorial" model="ir.actions.act_url">
             <field name="name">res_partner_mails_count Tutorial</field>
-            <field name="url" eval="'/web?debug=1&amp;res_partner_mails_count=tutorial#id='+str(ref('base.partner_root'))+'&amp;view_type=form&amp;model=res.partner&amp;/#tutorial_extra.mails_count_tour=true'"/>
+            <field name="url" eval="'/web?res_partner_mails_count=tutorial#id='+str(ref('base.partner_root'))+'&amp;view_type=form&amp;model=res.partner&amp;/#tutorial_extra.mails_count_tour=true'"/>
             <field name="target">self</field>
         </record>
     </data>

From 81843d71e98a46265b9d1e6ea6c4ed80955d870c Mon Sep 17 00:00:00 2001
From: Ivan Yelizariev <yelizariev@it-projects.info>
Date: Tue, 26 Apr 2016 11:26:05 +0500
Subject: [PATCH 04/10] [DOC] change text in step

---
 .../static/src/js/res_partner_mails_count_tour.js            | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
index b64e84b..ead7743 100644
--- a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
+++ b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
@@ -19,13 +19,12 @@
                 title:     _t("New fields"),
                 content:   _t("Here is new fields with mails counters. Press one of it."),
                 element:   '.mails_to',
-                
             },
             {
                 waitNot:   '.mails_to:visible',
-                title:     _t("Send message from here"),
+                title:     _t("Check messages or send new one"),
                 placement: 'left',
-                content:   _t("Now you can see corresponding mails. You can send mail to this partner right from here. Press <em>'Send a mesage'</em>."),
+                content:   _t("Click <em>'Send a mesage'</em> to continue the Tour"),
                 element:   '.oe_mail_wall .oe_msg.oe_msg_composer_compact>div>.oe_compose_post',
             },
             {

From 1ef37eb351e26cf6f2e6260768569b0ab13c2c92 Mon Sep 17 00:00:00 2001
From: Ivan Yelizariev <yelizariev@it-projects.info>
Date: Tue, 26 Apr 2016 11:32:24 +0500
Subject: [PATCH 05/10] [DOC] Tour: delete waitFor and increase wait in last
 step

It seems that waitFor+wait doesn't work as expected.
"wait" is count down before start check waitFor condition, while the
idea was opposite -- wait when page is loaded and then count down 2 sec
---
 .../static/src/js/res_partner_mails_count_tour.js              | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
index 960e805..c821231 100644
--- a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
+++ b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
@@ -22,8 +22,7 @@ odoo.define('res_partner_mails_count.res_partner_mails_count_tour', function (re
                 waitFor:   '.mails_to:visible',
             },
             {
-                wait:   '2000',
-                waitFor:   '.o_channel_name.mail_archives:visible',
+                wait:   '8000',
                 title:     _t("That's it"),
                 content:   _t("Enjoy your day! <br/> <br/><a href='https://www.it-projects.info/apps' target='_blank'>IT-Projects LLC</a> team "),
                 popover:   { next: _t("Close Tutorial") },

From 52b61e5e6872cc7f02f1d5441cf675a794820094 Mon Sep 17 00:00:00 2001
From: Ivan Yelizariev <yelizariev@it-projects.info>
Date: Tue, 26 Apr 2016 12:02:26 +0500
Subject: [PATCH 06/10] [FIX] waitFor on last step is needed for test mode

---
 .../static/src/js/res_partner_mails_count_tour.js                | 1 +
 1 file changed, 1 insertion(+)

diff --git a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
index c821231..5387fd6 100644
--- a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
+++ b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
@@ -23,6 +23,7 @@ odoo.define('res_partner_mails_count.res_partner_mails_count_tour', function (re
             },
             {
                 wait:   '8000',
+                waitFor:   '.o_channel_name.mail_archives:visible'
                 title:     _t("That's it"),
                 content:   _t("Enjoy your day! <br/> <br/><a href='https://www.it-projects.info/apps' target='_blank'>IT-Projects LLC</a> team "),
                 popover:   { next: _t("Close Tutorial") },

From cd8d9b07257eec9e06042e203c836d2c3165a8c5 Mon Sep 17 00:00:00 2001
From: Ivan Yelizariev <yelizariev@it-projects.info>
Date: Tue, 26 Apr 2016 12:35:13 +0500
Subject: [PATCH 07/10] [FIX] missed comma

---
 .../static/src/js/res_partner_mails_count_tour.js               | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
index 5387fd6..10cff70 100644
--- a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
+++ b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
@@ -23,7 +23,7 @@ odoo.define('res_partner_mails_count.res_partner_mails_count_tour', function (re
             },
             {
                 wait:   '8000',
-                waitFor:   '.o_channel_name.mail_archives:visible'
+                waitFor:   '.o_channel_name.mail_archives:visible',
                 title:     _t("That's it"),
                 content:   _t("Enjoy your day! <br/> <br/><a href='https://www.it-projects.info/apps' target='_blank'>IT-Projects LLC</a> team "),
                 popover:   { next: _t("Close Tutorial") },

From ba48e29803192daeb6c79d2ce266362bcded1600 Mon Sep 17 00:00:00 2001
From: Ivan Yelizariev <yelizariev@it-projects.info>
Date: Tue, 26 Apr 2016 12:43:12 +0500
Subject: [PATCH 08/10] [IMP] new step in tour

It also fix problem with last timeouted step
---
 .../static/src/js/res_partner_mails_count_tour.js            | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
index 10cff70..0f5abdf 100644
--- a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
+++ b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
@@ -21,6 +21,11 @@ odoo.define('res_partner_mails_count.res_partner_mails_count_tour', function (re
                 element:   '.mails_to',
                 waitFor:   '.mails_to:visible',
             },
+            {
+                title:     _t("Messages are found!"),
+                placement: 'left',
+                element:   '.o_mail_thread',
+            },
             {
                 wait:   '8000',
                 waitFor:   '.o_channel_name.mail_archives:visible',

From d2a6373f1e5969a53a9472c2e68aef575ba1babc Mon Sep 17 00:00:00 2001
From: Ivan Yelizariev <yelizariev@it-projects.info>
Date: Tue, 26 Apr 2016 12:53:10 +0500
Subject: [PATCH 09/10] [FIX] waitNot is necessary, because o_mail_thread is
 already presented in partner page

---
 .../static/src/js/res_partner_mails_count_tour.js                | 1 +
 1 file changed, 1 insertion(+)

diff --git a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
index 0f5abdf..015f4de 100644
--- a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
+++ b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
@@ -24,6 +24,7 @@ odoo.define('res_partner_mails_count.res_partner_mails_count_tour', function (re
             {
                 title:     _t("Messages are found!"),
                 placement: 'left',
+                waitNot:   '.mails_to:visible',
                 element:   '.o_mail_thread',
             },
             {

From e2f097a45821ae8a72bbc6ca72674a8dca4fc15b Mon Sep 17 00:00:00 2001
From: Ivan Yelizariev <yelizariev@it-projects.info>
Date: Tue, 26 Apr 2016 13:01:40 +0500
Subject: [PATCH 10/10] [IMP] descrease delay time + new placement option

---
 .../static/src/js/res_partner_mails_count_tour.js             | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
index 015f4de..1a12578 100644
--- a/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
+++ b/res_partner_mails_count/static/src/js/res_partner_mails_count_tour.js
@@ -23,12 +23,12 @@ odoo.define('res_partner_mails_count.res_partner_mails_count_tour', function (re
             },
             {
                 title:     _t("Messages are found!"),
-                placement: 'left',
+                placement: 'top',
                 waitNot:   '.mails_to:visible',
                 element:   '.o_mail_thread',
             },
             {
-                wait:   '8000',
+                wait:   '5000',
                 waitFor:   '.o_channel_name.mail_archives:visible',
                 title:     _t("That's it"),
                 content:   _t("Enjoy your day! <br/> <br/><a href='https://www.it-projects.info/apps' target='_blank'>IT-Projects LLC</a> team "),