Skip to content

Commit a67b7dc

Browse files
Merge pull request #620 from magento-epam/MC-36922
MC-36922: [Fix] Track pages where Page builder has been used
2 parents 37e820a + 0dffbef commit a67b7dc

File tree

2 files changed

+13
-10
lines changed

2 files changed

+13
-10
lines changed

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

+7-8
Original file line numberDiff line numberDiff line change
@@ -16,33 +16,32 @@ define(['underscore'], function (_) {
1616
*/
1717

1818
build: function (name, args) {
19-
var arrayName = name.split(':'),
20-
action = '',
19+
var action = '',
2120
eventAttributes = {},
2221
event;
2322

2423
if (_.isUndefined(args)) {
2524
return;
2625
}
2726

28-
switch (arrayName[arrayName.length - 1]) {
29-
case 'duplicateAfter':
27+
switch (name) {
28+
case 'contentType:duplicateAfter':
3029
action = 'duplicate';
3130
break;
3231

33-
case 'removeAfter':
32+
case 'contentType:removeAfter':
3433
action = 'remove';
3534
break;
3635

37-
case 'createAfter':
36+
case 'contentType:createAfter':
3837
action = 'create';
3938
break;
4039

41-
case 'editBefore':
40+
case 'contentType:editBefore':
4241
action = 'edit';
4342
break;
4443

45-
case 'visibilityAfter':
44+
case 'contentType:visibilityAfter':
4645
action = args.visibility ? 'show' : 'hide';
4746
break;
4847

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

+6-2
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,7 +26,10 @@ define(['underscore', 'Magento_PageBuilderAdminAnalytics/js/page-builder/event-b
2526
!_.isUndefined(window.digitalData) &&
2627
!_.isUndefined(window._satellite);
2728

28-
if (name.indexOf('readyAfter') !== -1 && isAdminAnalyticsEnabled) {
29+
if (!hasPageBuilderBeenUsed && name.indexOf('stage:fullScreenModeChangeAfter') !== -1 &&
30+
args.fullScreen && isAdminAnalyticsEnabled
31+
) {
32+
hasPageBuilderBeenUsed = true;
2933
window.digitalData.page.url = window.location.href;
3034
window.digitalData.page.attributes = {
3135
editedWithPageBuilder: 'true'

0 commit comments

Comments
 (0)