Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Partial data badges showing incorrectly when property is in partial data state #9474

Closed
1 task done
wpdarren opened this issue Oct 8, 2024 · 5 comments
Closed
1 task done
Labels
Module: Analytics Google Analytics module related issues P0 High priority Team M Issues for Squad 2 Type: Bug Something isn't working

Comments

@wpdarren
Copy link
Collaborator

wpdarren commented Oct 8, 2024

Bug Description

Reported during the bug bash. Asana ticket here

When the GA property is in the partial data state neither the audience or "Top content" partial data badge should display in the audience tiles.

However, the audience partial data badge briefly appears while the property partial data state is being determined, and the "Top content" partial data badge appears and stays in the tile.

image


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

Acceptance criteria

  • Neither of the partial data badges in an audience tile should be displayed until the partial data state for the currently connected Analytics property, the audience, and the googlesitekit_post_type custom dimension have been determined.
    • This is to ensure that neither of the badges will briefly show up before being removed once the state has been been fully resolved.
  • Neither of the partial data badges in an audience tile should be displayed if the property is in the partial data state.

Implementation Brief

  • In assets/js/modules/analytics-4/components/audience-segmentation/dashboard/AudienceTilesWidget/AudienceTile/index.js
    • While determining whether the audience is in partial data state, check if isPropertyPartialData is undefined, return false within useInViewSelect if it is undefined or true.
    • While determining whether top content is in partial data state, isTopContentPartialData, check if isPropertyPartialData is undefined, return false within useInViewSelect if it is undefined or true.

Test Coverage

  • Add the JS test for ensuring Partial Data badge doesn't show for audience tile or top content when analytics property is in partial data state.

QA Brief

  • Set up Site Kit and connect Analytics using a property which is in the partial data state. It's preferable for the property to be in the partial data state over the past 28 days, if it's only partial over the past 90 days it should be possible to quickly change the selected date range when the page loads to test the issue.
  • The property needs to have at least one user-defined audience with non-zero data over the past 90 days already set up.
  • Click on Enable groups to set up Audience Segmentation.
  • Verify the partial data badges at the top of the user-defined audiences don't briefly appear.
  • Verify the partial data badges for the "Top content" metric area don't appear at all.

Changelog entry

  • Ensure partial data badges display correctly when the connected Analytics property is in the partial data state.
@wpdarren wpdarren added Type: Bug Something isn't working P0 High priority Module: Analytics Google Analytics module related issues labels Oct 8, 2024
@ivonac4 ivonac4 added the Team M Issues for Squad 2 label Oct 10, 2024
@techanvil techanvil assigned techanvil and unassigned techanvil Oct 10, 2024
@ankitrox ankitrox assigned ankitrox and unassigned ankitrox Oct 14, 2024
@techanvil techanvil self-assigned this Oct 14, 2024
@techanvil
Copy link
Collaborator

IB ✅

@kelvinballoo
Copy link
Collaborator

QA Update ⚠

Hi @techanvil , I tested this and the results are not as expected.
This is the way I am testing:

  • Set up SK and to create a new analytics property with a user-defined audience

  • I am setting up the Tester plugin with random data and partial data state for 28 days + AS flag on. Image below is the partial state config.

    Image

  • I enabled the groups. Initially, there was only one tile (user-defined one) that appeared without 'Partial data' badge. I then added one more tile to it and then both tiles had the partial data badge at the top content area. Video is attached for reference.

    Image

    9474.-.Issue.with.partial.data.mov

QUESTION:
It looks like an issue because we are expecting the 'Partial Data' badge not to appear at the Top content area. Let me know your thoughts on this.
The good thing is that the flicker/brief appearance at the top is no longer there.

@techanvil
Copy link
Collaborator

Hi @kelvinballoo, thanks for spotting this! We someone managed to miss that one. I've created a followup PR to address the issue: #9554

@techanvil techanvil removed their assignment Oct 24, 2024
@hussain-t hussain-t self-assigned this Oct 24, 2024
@hussain-t
Copy link
Collaborator

Thanks for the follow-up fix, @techanvil.

Sending it back to you for another pass, @kelvinballoo.

@hussain-t hussain-t assigned kelvinballoo and unassigned hussain-t Oct 24, 2024
@kelvinballoo
Copy link
Collaborator

QA Update ✅

Verified good as follows.
Moving ticket to approval.

  • SK was set up and used the tester plugin to set it to partial data state.

  • Analytics property connected had at least one user-defined audience with non-zero data.

  • When clicking on Enable groups to set up Audience Segmentation:

    • The partial data badges at the top of the user-defined audiences do not briefly appear. ✅
    • The partial data badges for the "Top content" metric area don't appear at all. ✅
    9474.-.partial.data.not.showing.720p.mov

@kelvinballoo kelvinballoo removed their assignment Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Module: Analytics Google Analytics module related issues P0 High priority Team M Issues for Squad 2 Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants