diff options
| author | Aaron Bohy <aab@odoo.com> | 2019-01-09 11:46:48 +0000 |
|---|---|---|
| committer | Aaron Bohy <aab@odoo.com> | 2019-01-14 11:31:55 +0000 |
| commit | 18cb7192b78cca03bf2a0e5d410106c5d502329b (patch) | |
| tree | 59fd7d30a5cd912b6e87b56d97ba774a62c5549c /odoo/addons/test_main_flows | |
| parent | b3f1081082ce33afe2513c2712bd0204fc5d8a22 (diff) | |
[FIX] test_main_flows: remove patches due to tour manager delay
Since the removal of the delay in the tour system, some patches that
had been done to make the tour pass (all the time) can be removed.
Diffstat (limited to 'odoo/addons/test_main_flows')
| -rw-r--r-- | odoo/addons/test_main_flows/static/src/js/tour.js | 17 |
1 files changed, 0 insertions, 17 deletions
diff --git a/odoo/addons/test_main_flows/static/src/js/tour.js b/odoo/addons/test_main_flows/static/src/js/tour.js index 047b0c40543..c49ae1cb4eb 100644 --- a/odoo/addons/test_main_flows/static/src/js/tour.js +++ b/odoo/addons/test_main_flows/static/src/js/tour.js @@ -211,7 +211,6 @@ tour.register('main_flow_tour', { position: 'bottom', }, { trigger: ".breadcrumb-item:first", - extra_trigger: ".o_form_readonly", // FIXME: this is required due to an issue in tour_manager (see [*]) content: _t("Use the breadcrumbs to <b>go back to products</b>."), position: "bottom" }, { @@ -542,12 +541,10 @@ tour.register('main_flow_tour', { trigger: ".o_menu_sections a[data-menu-xmlid='mrp.menu_mrp_manufacturing']", content: _t('Click on Operations menuitem'), position: 'bottom', - edition: 'enterprise', }, { trigger: ".o_menu_sections a[data-menu-xmlid='mrp.menu_mrp_production_action']", content: _t('Open manufacturing orders'), position: 'bottom', - edition: 'enterprise', }, { trigger: '.o_data_row:has(.o_data_cell:contains("the_flow.product")):first', content: _t('Select the generated manufacturing order'), @@ -755,7 +752,6 @@ tour.register('main_flow_tour', { }, { edition: "enterprise", trigger: ".o_selected_row .o_field_widget[name=name]", - extra_trigger: ".o_selected_row .o_field_widget[name=partner_id] .o_external_button", // FIXME: this is required due to an issue in tour_manager (see [*]) content: _t('Let\'s enter a name.'), position: "bottom", run: "text the_flow.statement.line", @@ -781,16 +777,3 @@ tour.register('main_flow_tour', { position: "bottom", }]); }); - -/* - * [*] FIXME: issue in tour_manager: - * The JQuery element of a step is registered as soon as its trigger and - * extra_trigger elements are visible in the DOM, but it's action is delayed - * to handle the 'running_step_delay' option (0 by default, but even with 0, - * the action is executed after finishing the current execution block, which - * may produce a re-rendering of the part of the DOM containing the element). - * When this happens, the action is executed on an jquery element not present - * in the DOM anymore. - * To avoid this, we add an 'extra_trigger' to wait for the updated jquery - * element before activating the step (and thus registering the JQuery element). - */ |
