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 279 forks source link

Fix glitches in the Audience Segmentation Setup CTA Banner #9231

Open techanvil opened 3 weeks ago

techanvil commented 3 weeks ago

Bug Description

There are a couple of glitches relating to the Audience Segmentation Setup CTA Banner.

  1. The "Don't show again" dismissal CTA flashes in place of the "Maybe later" CTA.
  2. The Setup CTA Banner shows again when the No Audiences Banner is visible.

These should be small fixes and can be addressed in this single issue.

Steps to reproduce

The "Don't show again" dismissal CTA flashes in place of the "Maybe later" CTA.

  1. Set up Site Kit with the audienceSegmentation feature flag enabled and Analytics connected.
  2. Refresh the dashboard to show the Setup CTA Banner.
  3. The dismissal CTA will briefly show the "Don't show again" text in place of "Maybe later".
  4. Run through the setup process and the same glitch will occur while the "Enabling groups" CTA is showing.

The Setup CTA Banner shows again when the No Audiences Banner is visible.

  1. Set up Site Kit with the audienceSegmentation feature flag enabled and Analytics connected.
  2. Set up the Audience Segmentation feature with the "new visitors" and "returning visitors" audiences selected.
  3. Archive the audiences in Analytics (or hide them using the Tester plugin).
  4. Refresh the dashboard and open the Selection Panel to trigger an audience resync.
  5. See the Setup CTA Banner unexpectedly return, while the No Audiences Banner is displayed as expected.

Screenshots

The "Don't show again" dismissal CTA flashes in place of the "Maybe later" CTA.

https://github.com/user-attachments/assets/1a79dabc-9986-4341-86a0-82e3db1bba76

The Setup CTA Banner shows again when the No Audiences Banner is visible.

enable_groups_with_no_audiences

Additional Context


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 2 weeks ago

IB :white_check_mark:

kelvinballoo commented 1 week ago

QA Update ✅

This was verified good. Moving ticket to Approval.

https://github.com/user-attachments/assets/4a507f51-0fa8-46af-8aa7-560770382c24