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

As a second admin, banners still show even after disabling the feature #9481

Open
1 task
wpdarren opened this issue Oct 9, 2024 · 4 comments
Open
1 task
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 9, 2024

Bug Description

Created from bug bash. Asana ticket can be found here.

As a second admin, banners still show even after disabling the feature.

Image

Steps to reproduce

  1. Setup AS as an admin
  2. Switch to a secondary admin account, got to the settings and disable the feature
  3. When you go back to the dashboard you still see the setup banner and subtle success notification even though the widget area is now hidden.

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

Acceptance criteria

Implementation Brief

  • In the AudienceSegmentationIntroductoryOverlayNotification component, retrieve the isAudienceSegmentationWidgetHidden value. Add a check for isAudienceSegmentationWidgetHidden === false to the condition that determines if the notification should be displayed:

const shouldShowNotification =
isDismissed === false &&
shouldShowAudienceSegmentationIntroductoryOverlay &&
DASHBOARD_TYPE_MAIN === dashboardType;

Test Coverage

  • Add a test case to the JS tests for the component, and fix its story/VRT.

QA Brief

  • Follow the steps to reproduce, and confirm the Intro Popup/Banner does not appear.
  • Toggle the feature back on, and return to the dashboard. The Intro Popup/Banner should reappear.

Note that, as per the AC, the Setup Success Banner will be addressed separately via #9482.

Changelog entry

  • Fix an issue that caused the visitor groups overlay notification to appear unexpectedly.
@wpdarren wpdarren added Module: Analytics Google Analytics module related issues P0 High priority Type: Bug Something isn't working labels Oct 9, 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
@nfmohit nfmohit self-assigned this Oct 14, 2024
@nfmohit
Copy link
Collaborator

nfmohit commented Oct 14, 2024

IB ✅

@nfmohit nfmohit removed their assignment Oct 14, 2024
@techanvil techanvil self-assigned this Oct 16, 2024
@techanvil techanvil removed their assignment Oct 17, 2024
@nfmohit nfmohit assigned nfmohit and unassigned nfmohit Oct 17, 2024
@mohitwp mohitwp self-assigned this Oct 21, 2024
@mohitwp
Copy link
Collaborator

mohitwp commented Oct 21, 2024

QA Update ⚠

Question

@techanvil I've noticed that the banner and widgets still appear on the view-only dashboard, even after both the secondary and primary admins disable the feature. Is this expected behaviour? My understanding is that if the primary admin who shared the modules disables the feature, the banner should no longer appear on the view-only dashboard.

Here’s the sequence I observed:

  1. The primary admin sets up and shares the Analytics module with the secondary admin and the view-only dashboard.
  2. The secondary admin disables the feature under settings.
  3. The banner stops appearing on the main dashboard for the secondary admin.
  4. The primary admin also disables the feature under settings.
  5. When logging in as a view-only user, the banner and widgets still show on the view-only dashboard.
  6. Could you please share your thoughts on this?
Recording.1533.mp4

PASS CASE

Recording.1532.mp4

@techanvil
Copy link
Collaborator

Hi @mohitwp, thanks for checking in on this.

The "Display visitor groups in the dashboard" setting is per-user, not a global setting, so the behaviour you are seeing is as expected.

In fact, at the moment it's not possible for a view-only user to toggle the setting at all, as they don't have access to the Settings screen, this is a known shortcoming in the UX that we will address at some point but it's as expected for the time being.

@techanvil techanvil removed their assignment Oct 21, 2024
@mohitwp
Copy link
Collaborator

mohitwp commented Oct 21, 2024

QA Update ✅

Recording.1532.mp4

@mohitwp mohitwp removed their assignment Oct 21, 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

5 participants