Incorrect dimension filter applied in various components that uses adSourceName
dimension
#8658
Labels
Module: AdSense
Google AdSense module related issues
Module: Analytics
Google Analytics module related issues
P0
High priority
Squad 1 (Team S)
Issues for Squad 1
Type: Bug
Something isn't working
Bug Description
In the following components, the
dimensionFilters
is applied in the report args incorrectly, resulting them to be no-op.AnalyticsAndAdSenseAccountsDetectedAsLinkedOverlayNotification
DashboardTopEarningPagesWidgetGA4
TopEarningContentWidget
The dimension filters should be placed inside the
dimensionFilters
property in the report args object. Also for a little bit of simplification, the logic can be rewritten to be the following shorthand syntax, since the filter is a exact match string filter covered byGoogle\Site_Kit\Modules\Analytics_4\Report\Request::parse_dimension_filter
:Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
AnalyticsAndAdSenseAccountsDetectedAsLinkedOverlayNotification
,DashboardTopEarningPagesWidgetGA4
andTopEarningContentWidget
should be applied correctly.Implementation Brief
assets/js/components/OverlayNotification/AnalyticsAndAdSenseAccountsDetectedAsLinkedOverlayNotification.js
,assets/js/modules/adsense/components/dashboard/DashboardTopEarningPagesWidgetGA4.js
andassets/js/modules/adsense/components/widgets/TopEarningContentWidget.js
file:filter
property from the report options.dimensionFilters
object to the report option, and add theadSourceName
property withGoogle AdSense account (${ adSenseAccountID })
value inside thedimensionFilters
object.Test Coverage
QA Brief
QAing the changes here specifically would require a site that would render the Top Earning Pages Widget and Top Earning Content Widget on the Site Kit dashboard, then testing the output before/after this change. That's probably not realistic, but checking for the filters being correct is enough really, so best to just test that the components are still rendering correctly without errors, and the new filters are working without issue.
Changelog entry
adSourceName
dimension to use the correct dimension filter.The text was updated successfully, but these errors were encountered: