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

Optimize sunset banner re-appears on Dashboard after dismissal #7138

Closed
wpdarren opened this issue Jun 6, 2023 · 4 comments
Closed

Optimize sunset banner re-appears on Dashboard after dismissal #7138

wpdarren opened this issue Jun 6, 2023 · 4 comments
Labels
Good First Issue Good first issue for new engineers Module: Optimize Google Optimize module related issues P1 Medium priority Type: Bug Something isn't working

Comments

@wpdarren
Copy link
Collaborator

wpdarren commented Jun 6, 2023

Bug Description

The Optimize sunset banner appears on the dashboard even though it has been dismissed.

It only appears for a split second, and it is difficult to see even with slow 3G. Thought it was worth raising. I was able to recreate it with GA4Reporting enabled and disabled.

Screen.Capture.on.2023-06-05.at.18-21-29.mp4

Steps to reproduce

  1. Set up Analytics and Optimize
  2. Go to the main dashboard and notice the Optimize sunset banner
  3. Dismiss the banner notification.
  4. Refresh the dashboard. When I initially tested this the banner didn't appear, so you might want to leave the dashboard (go to settings) and then come back and you'll see the banner very quickly load as per screencast above.

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

Acceptance criteria

  • The Optimize removal banner should not appear if it has been dismissed

Implementation Brief

  • Update the condition to return early in the OptimizeRemovalNotification to return if isDismissed is not strictly false as this can be undefined while dismissed items have not been resolved yet which is why the banner shows briefly (this is only observable when the client-side dismissed item is not present in session storage – i.e. the non-persistent dismissal)

Test Coverage

  • No changes needed

QA Brief

  • Ensure that the Google Optimize module is enabled, on a fresh setup of Google Site Kit.
    • If using an existing install where the banner has already been dismissed, the tester may remove the user meta key named wp_googlesitekitpersistent_dismissed_items inside of the wp_usermeta table of the WordPress database, applicable to their user ID. Although this is the least preferred method, it will clear any item dismissal states to date and unlock testing of this fix.
  • Validate that once Optimize is enabled, the Optimize sunset banner appears within the Site Kit dashboard.
  • Validate that, once dismissed, the Optimize sunset banner no longer appears.
    • This can be validated using slow network simulation within Google Chrome as well, i.e fast 3G or slow 3G

Changelog entry

  • Fix the Optimize sunset banner re-appearing issue.
@wpdarren wpdarren added Module: Optimize Google Optimize module related issues Type: Bug Something isn't working labels Jun 6, 2023
@aaemnnosttv aaemnnosttv self-assigned this Jun 6, 2023
@aaemnnosttv aaemnnosttv added the P1 Medium priority label Jun 6, 2023
@aaemnnosttv aaemnnosttv removed their assignment Jun 6, 2023
@aaemnnosttv aaemnnosttv added the Good First Issue Good first issue for new engineers label Jun 7, 2023
@techanvil techanvil self-assigned this Jun 7, 2023
@techanvil
Copy link
Collaborator

IB ✅

@techanvil techanvil removed their assignment Jun 7, 2023
@chirag1718
Copy link

Hello I find this interesting and I would like to resolve this issue.

@10upsimon 10upsimon self-assigned this Jun 23, 2023
@bethanylang
Copy link
Collaborator

Hi @chirag1718! Someone on our internal team has already picked this up, but please do feel free to take a look at other issues if you'd like. :)

@wpdarren
Copy link
Collaborator Author

QA Update: ✅

Verified:

  • Once Optimize is enabled, the Optimize sunset banner appears within the Site Kit dashboard.
  • Once dismissed, the Optimize sunset banner no longer appears.
    • I validated by refreshing the Site Kit dashboard and the banner does not appear.
    • I also validated by navigating away from the dashboard, ie. going to settings and revisiting the dashboard.

@wpdarren wpdarren removed their assignment Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue Good first issue for new engineers Module: Optimize Google Optimize module related issues P1 Medium priority Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants