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

Ensure CTAs don’t appear for view-only users #8174

Closed
1 task done
techanvil opened this issue Jan 25, 2024 · 8 comments
Closed
1 task done

Ensure CTAs don’t appear for view-only users #8174

techanvil opened this issue Jan 25, 2024 · 8 comments
Labels
Module: Analytics Google Analytics module related issues P1 Medium priority Team M Issues for Squad 2 Type: Enhancement Improvement of an existing feature

Comments

@techanvil
Copy link
Collaborator

techanvil commented Jan 25, 2024

Feature Description

Ensure CTAs for setting the feature up and creating resources, and the link to the Settings section in the Selection Panel don’t appear for view-only users.

See dashboard sharing in the design doc.


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

Acceptance criteria

  • When on the view-only dashboard, no Audience Segmentation CTAs should be displayed as the user won’t have permission to action them:
    • The Setup CTA Banner should not be displayed.
    • The Audience Creation Notice in the Selection Panel should not be displayed.
    • The link to the Settings section in the Selection Panel should not be displayed.

Implementation Brief

Test Coverage

  • Update VRTs. No additional tests required.

QA Brief

  • Connect/create a property that requires the Site Kit audiences to be created.
  • Switch to a view only user.
  • The Audience Creation notice should not show in this context.
  • Ensure that the other CTAs mentioned in the ACs do not appear in the view only dashboard as well.

Changelog entry

  • Enhance Audience Segmentation CTAs to not appear on shared dashboard.
@techanvil techanvil added Module: Analytics Google Analytics module related issues P1 Medium priority Type: Enhancement Improvement of an existing feature labels Jan 25, 2024
@hussain-t hussain-t self-assigned this Jan 30, 2024
@techanvil
Copy link
Collaborator Author

Hi @hussain-t, just a heads up that I've tweaked the requirement for this issue to include removing the link to the settings section in the selection panel for view-only users.

@hussain-t hussain-t removed their assignment Feb 26, 2024
@ivonac4 ivonac4 added the Team M Issues for Squad 2 label Apr 9, 2024
@techanvil techanvil assigned techanvil and unassigned techanvil Jul 10, 2024
@eclarke1 eclarke1 added the Next Up Issues to prioritize for definition label Jul 10, 2024
@benbowler benbowler assigned benbowler and unassigned benbowler Jul 10, 2024
@kuasha420 kuasha420 self-assigned this Jul 10, 2024
@kuasha420
Copy link
Contributor

Thank you @benbowler for working on this IB. A couple of points.

  1. Looks like AudienceSegmentationSetupCTAWidget is not being rendered on shared dashboard anyways as it's guarded in assets/js/components/DashboardMainApp.js.
  2. We also should follow similar pattern for AddGroupNotice, by preventing it from rendering in shared dashboard in assets/js/modules/analytics-4/components/audience-segmentation/dashboard/AudienceSelectionPanel/index.js.

Thanks.

@kuasha420 kuasha420 assigned benbowler and unassigned kuasha420 Jul 11, 2024
@benbowler
Copy link
Collaborator

Thanks @kuasha420, updated!

@benbowler benbowler assigned kuasha420 and unassigned benbowler Jul 12, 2024
@kuasha420
Copy link
Contributor

IB ✔️

@kuasha420 kuasha420 removed their assignment Jul 15, 2024
@ivonac4 ivonac4 removed the Next Up Issues to prioritize for definition label Jul 16, 2024
@hussain-t hussain-t self-assigned this Aug 8, 2024
@hussain-t
Copy link
Collaborator

We don’t have an easy way to view the Audience Segmentation widgets until #8130 is implemented. Therefore, I’ve updated the dependency. This task should be worked on once #8130 is completed.

cc: @techanvil @ivonac4

@hussain-t hussain-t removed their assignment Aug 8, 2024
@ivonac4 ivonac4 added the Sp Wk 2 Issues to be completed in the second week of the assigned sprint label Aug 8, 2024
@ankitrox ankitrox assigned ankitrox and unassigned ankitrox Aug 13, 2024
@ivonac4 ivonac4 removed the Sp Wk 2 Issues to be completed in the second week of the assigned sprint label Aug 21, 2024
@benbowler benbowler self-assigned this Aug 27, 2024
@benbowler benbowler removed their assignment Aug 27, 2024
@kuasha420 kuasha420 assigned kuasha420 and unassigned kuasha420 Aug 28, 2024
@mohitwp mohitwp self-assigned this Aug 29, 2024
@wpdarren wpdarren assigned kelvinballoo and unassigned mohitwp Aug 29, 2024
@kelvinballoo
Copy link
Collaborator

QA Update ⚠️

Under the view mode, I was not able to see the tiles.
I can't access the selection panel to verify that the notice doesn't appear.
Am I missing any steps here?

Video is below for reference.

SK.view.only.doesn.t.appear.mov

@benbowler
Copy link
Collaborator

Hey @kelvinballoo, as this ticket is waiting on #8130, it won't be testable until that ticket is merged.

@kelvinballoo
Copy link
Collaborator

QA Update ✅

Thanks @benbowler for the clarification.
Since this is behind a feature flag, I am marking this ticket as Pass and I added a note here for the QAB of #8130 to be updated to include the testing of #8174.

Moving ticket to Approval.

@kelvinballoo kelvinballoo removed their assignment Sep 5, 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 P1 Medium priority Team M Issues for Squad 2 Type: Enhancement Improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

10 participants