-
Notifications
You must be signed in to change notification settings - Fork 295
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
Comments
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. |
Thank you @benbowler for working on this IB. A couple of points.
Thanks. |
Thanks @kuasha420, updated! |
IB ✔️ |
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 |
QA Update
|
Hey @kelvinballoo, as this ticket is waiting on #8130, it won't be testable until that ticket is merged. |
QA Update ✅Thanks @benbowler for the clarification. Moving ticket to Approval. |
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
Implementation Brief
assets/js/modules/analytics-4/components/audience-segmentation/dashboard/AudienceSelectionPanel/index.js
, use theuseViewOnly
hook to only render theAudienceCreationNotice
outside of the view only context.AudienceSegmentationSetupCTAWidget.js
is already hidden in the view only context.Test Coverage
QA Brief
Changelog entry
The text was updated successfully, but these errors were encountered: