search

google / site-kit-wp

Site Kit is a one-stop solution for WordPress users to use everything Google has to offer to make them successful on the web.
https://sitekit.withgoogle.com
Apache License 2.0
1.23k stars 287 forks source link

Update metrics used in GA4 All Traffic Widget for consistency #7263

Open aaemnnosttv opened 1 year ago

aaemnnosttv commented 1 year ago

Bug Description

In #6639 we changed the source links in GA4 dashboard widgets to point to the correct locations in the GA4 reporting interfaces on the service. In doing so, a few of the metrics that are used within the All Traffic Widget were changed to use the incorrect users metric activeUsers instead of totalUsers. This is likely due to an oversight when trying to match the destination URL since GA4 uses activeUsers by default on the service.

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

Implementation Brief

Test Coverage

QA Brief

Changelog entry

techanvil commented 1 year ago

It's worth noting that the GA4 Analytics UI on analytics.google.com that we link to seems to be hardwired to use activeUsers, as changing the metric to totalUsers in the resulting Analytics URL doesn't change the figures displayed in the UI.

As a sanity check, I ran a couple of reports using the API to verify the data displayed in the UI matched a report using activeUsers while a similar report using totalUsers had different values.

aaemnnosttv commented 1 year ago

@techanvil the AC here probably depend on the answer we get to our internal query about these metrics so assigning this to you for now.

ivonac4 commented 9 months ago

@techanvil I see there was some kind of dependency here, are you able to work on this now?

techanvil commented 5 months ago

@ivonac4, sorry for the slow reply here.

This issue relates to the stalled issue #7214 and the related discussion in Slack. We'll have to pick this conversation up again to progress this issue, however this was a conversation with Mariya so if we want to revisit this while she's away, we'll need to get Andrey up to speed on the context here. For now I've unassigned the issue and moved it to Stalled as I don't expect us to get to it in the near future.