Closed 10upsimon closed 1 month ago
- Said widget makes use of the
MetricTileTable
component to facilitate layout of said top 3 pages, matching the design
@10upsimon, AC looks good but this point isn't needed, I'll remove it and move this ticket to IB.
IB ✔️
@tofumatt I removed one point from IB, based on latest development, we will not need to use showWidget
check in case you already included it, it can be removed
Top pages driving leads
widget is not available under KMW panel if conversionReporting
feature flag is not enabled.Top pages driving leads
widget is showing zero data if data is not available for selected property or selected date range.
Feature Description
One of the identified metric tiles for the ACR epic is
Top pages driving leads
. Details of this metric tile are as follows:contact
andsubmit_lead_form
events, if present, otherwise, it will use the data for events that are present.submit_lead_form
,contact
screenPageViews
,generate_lead
,contact
andsubmit_lead_form
events count,pagePath
Figma Design
Top pages driving leads within ACR metrics section of design
Widget Preview
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
Top pages driving leads
ACR widget is introduced to the existing list of widgets available within KMWgenerate_lead
,submit_lead_form
andcontact
event count based on thepagePath
, with page titles and URLs listed on the left, and total event counts to the rightconversionReporting
feature flag is not enabled, until such time that the feature flag is removed in a future issueImplementation Brief
assets/js/googlesitekit/datastore/user/constants.js
KM_ANALYTICS_TOP_PAGES_DRIVING_LEADS
, holding a stringkmAnalyticsTopPagesDrivingLeads
assets/js/components/KeyMetrics/key-metrics-widgets.js
shouldDisplayWidgetWithConversionEvent
, that will use the selector implemented in #9132 to check ifthis.requiredConversionEventName
is present in the returned array, alongside the presence ofconversionReporting
feature flag. It can work similarly likeshouldDisplayWidgetWithCustomDimensions
KM_ANALYTICS_TOP_PAGES_DRIVING_LEADS
title
description
andtooltip
requiredConversionEventName
, and assign it an array value containing following event names:submit_lead_form
,contact
andgenerate_lead
displayInList
pass the helper functionshouldDisplayWidgetWithConversionEvent
assets/js/modules/analytics-4/components/widgets/TopPagesDrivingLeadsWidget.js
assets/js/modules/analytics-4/components/widgets/PopularContentWidget.js
for example, as a starting pointreportOptions
:pagePath
andeventName
for dimensionseventCount
metricdimensionFilters
foreventName
dimension. UseinListFilter
forfilterType
and for thevalue
include the detected event name by cross-referencing therecentEvents
inline data returned from selector implemented in #9132 with -['submit_lead_form', 'contact', 'generate_lead']
. In case bothcontact
andsubmit_lead_form
are detected, use onlysubmit_lead_form
, to cover the cases where users have both Popup Maker and Contact Form 7 plugins installed, as both events would be tracked, making them duplicate.inListFilter
filter used inassets/js/modules/analytics-4/components/widgets/TopRecentTrendingPagesWidget.js
pagePath
and count from returned values ofeventCount
metricTest Coverage
QA Brief
Top pages driving leads
widget is not availableconversionReporting
feature flagTop pages driving leads
tileTop pages driving leads
widget is showing zero data (since widget will only have data on property that has ACR data)Changelog entry