Skip to content

Commit 971434d

Browse files
committed
MC-36922: [Fix required] Track pages where Page builder has been used
- Refactor: Avoid sending editedWithPageBuilder several times for a single page
1 parent 73e12e8 commit 971434d

File tree

1 file changed

+4
-2
lines changed
  • app/code/Magento/PageBuilderAdminAnalytics/view/adminhtml/web/js/page-builder

1 file changed

+4
-2
lines changed

app/code/Magento/PageBuilderAdminAnalytics/view/adminhtml/web/js/page-builder/events-mixin.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,8 @@ define(['underscore', 'Magento_PageBuilderAdminAnalytics/js/page-builder/event-b
1010
return function (target) {
1111
var originalTarget = target.trigger,
1212
isAdminAnalyticsEnabled,
13-
event;
13+
event,
14+
hasPageBuilderBeenUsed = false;
1415

1516
/**
1617
* Invokes custom code to track information regarding Page Builder usage
@@ -25,9 +26,10 @@ define(['underscore', 'Magento_PageBuilderAdminAnalytics/js/page-builder/event-b
2526
!_.isUndefined(window.digitalData) &&
2627
!_.isUndefined(window._satellite);
2728

28-
if (name.indexOf('stage:fullScreenModeChangeAfter') !== -1 &&
29+
if (!hasPageBuilderBeenUsed && name.indexOf('stage:fullScreenModeChangeAfter') !== -1 &&
2930
args.fullScreen && isAdminAnalyticsEnabled
3031
) {
32+
hasPageBuilderBeenUsed = true;
3133
window.digitalData.page.url = window.location.href;
3234
window.digitalData.page.attributes = {
3335
editedWithPageBuilder: 'true'

0 commit comments

Comments
 (0)