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

Goals CTA widget appears on shared dashboard #5351

Closed
eclarke1 opened this issue Jun 15, 2022 · 6 comments
Closed

Goals CTA widget appears on shared dashboard #5351

eclarke1 opened this issue Jun 15, 2022 · 6 comments
Labels
P0 High priority Type: Bug Something isn't working

Comments

@eclarke1
Copy link
Collaborator

eclarke1 commented Jun 15, 2022

Bug Description

Bug bash issue: https://app.asana.com/0/1202258919887896/1202419766205815 please refer to Asana issue for background

When the non-authenticated user (editor) has access to Analytics, the Goals CTA appears on the shared dashboard. It's my understanding that this should not appear.

image.png


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

Acceptance criteria

  • The Analytics "Goals" CTA should not appear in any shared dashboard context; it should only appear when the user viewing the dashboard has authenticated with Site Kit

Implementation Brief

  • Using assets/js/modules/search-console/components/dashboard/SearchFunnelWidget/Overview.js,
    • Render CreateGoalCTA or CTA if the user has been authenticated by querying the core/user data store via the isAuthenticated selector.

Test Coverage

  • No new tests to be added.

QA Brief

  • Ensure that the Goals CTA doesn't appear for non-authenticated users with view-only access to Analytics as mentioned in the AC.
  • Ensure that it appears for the authenticated users.

Changelog entry

  • Do not show "Create Goals" widget on shared dashboard.
@eclarke1 eclarke1 added P0 High priority Type: Bug Something isn't working labels Jun 15, 2022
@tofumatt tofumatt assigned tofumatt and unassigned tofumatt Jun 16, 2022
@aaemnnosttv
Copy link
Collaborator

  • it should only appear when the user viewing the dashboard has authenticated with Site Kit and has access to the Analytics account

@tofumatt we don't currently have a way to check if the user has that kind of access to the account/property. It may be possible to request through the management API but this would substantially increase the effort needed for this change and we don't do it now so I'm going to remove that detail. We can always revisit it later in a follow-up 👍

@asvinb asvinb assigned asvinb and unassigned asvinb Jun 17, 2022
@tofumatt tofumatt self-assigned this Jun 20, 2022
@tofumatt
Copy link
Collaborator

Ah, fair enough on the access bit. In that case, looks good!

IB ✅

@tofumatt tofumatt removed their assignment Jun 20, 2022
@hussain-t hussain-t self-assigned this Jun 21, 2022
@hussain-t hussain-t removed their assignment Jun 21, 2022
@techanvil techanvil assigned techanvil and hussain-t and unassigned techanvil Jun 21, 2022
@hussain-t hussain-t assigned techanvil and unassigned hussain-t Jun 21, 2022
@techanvil techanvil removed their assignment Jun 22, 2022
@techanvil
Copy link
Collaborator

Hi @hussain-t, please could you add a QA Brief for this issue?

@hussain-t
Copy link
Collaborator

OMG, how did I miss that! Thanks for catching it, @techanvil 👍

@hussain-t hussain-t assigned techanvil and unassigned hussain-t Jun 22, 2022
@techanvil
Copy link
Collaborator

Thanks @hussain-t, that's great. Moving back over to QA :)

@techanvil techanvil removed their assignment Jun 22, 2022
@mohitwp mohitwp self-assigned this Jun 22, 2022
@mohitwp
Copy link
Collaborator

mohitwp commented Jun 23, 2022

QA Update ✅

Verified 👍

  • Verified for admin user when admin user not signed in and access dashboard in view only mode.
  • Verified for other non authenticated users.
  • Now 'Goals CTA widget' is not appearing for users having only 'View only' access.

Admin user 'View only' dashboard

image

Editor User

image

@mohitwp mohitwp removed their assignment Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 High priority Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

8 participants