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

Inaccurate tooltip on 'Who can manage view access' dropdown #5370

Closed
eclarke1 opened this issue Jun 16, 2022 · 4 comments
Closed

Inaccurate tooltip on 'Who can manage view access' dropdown #5370

eclarke1 opened this issue Jun 16, 2022 · 4 comments
Labels
Good First Issue Good first issue for new engineers P0 High priority Type: Bug Something isn't working

Comments

@eclarke1
Copy link
Collaborator

eclarke1 commented Jun 16, 2022

Bug Description

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

Here's the scenario: admin 1 sets up Site Kit and connects all the modules. In the dashboard sharing settings, they select 'Only me' for search console and analytics under 'who can manage view access' All is good. Then admin 2 logs in and goes to the dashboard. They can see everything as expected, and then click on the 'view only' button. They then sign into Google with their google account. They then go to the dashboard sharing settings and see that they are unable to change the permissions for search console and analytics, which is expected BUT, when you hover the 'managed by' tooltip, the message is confusing IMO.

The tool tip says that 'admin 1 has given permission for all users to all admins (which isn't true) It then says you can change this on the dashboard. Against the module, the correct text is displayed "Contact managing user to manage view access" I think the tooltip is confusing.

Agreed that this message is confusing. Maybe we could change it to something like "Contact [username] to manage view access"?

image.png


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

Acceptance criteria

  • The tooltip message when an admin has set a shared module's "manage view access" to "only me" should be updated to: Contact [username] to change who can manage view access for this module.

Implementation Brief

Test Coverage

  • No new tests are needed for this change.

QA Brief

  • Make sure the dashboardSharing feature flag is turned on from the tester plugin.
  • Open the Dashboard Sharing settings modal.
  • In the "Who can manage view access" column, change the dropdown value to "Only me" for any one or more of the modules.
  • Log into the site as a new admin and log into Site Kit with a Google account.
  • Ensure that the tooltip in the "Who can manage view access" column text is according to the AC for the modules where "Only me" was selected.

Changelog entry

  • Update the tooltip for the "Managed by..." info icon on the Dashboard Sharing modal, when the module management has been set to "Only me".
@eclarke1 eclarke1 added P0 High priority Type: Bug Something isn't working labels Jun 16, 2022
@tofumatt tofumatt assigned tofumatt and unassigned tofumatt Jun 20, 2022
@eugene-manuilov eugene-manuilov self-assigned this Jun 21, 2022
@eugene-manuilov
Copy link
Collaborator

@tofumatt, shouldn't we display that text only if the current sharing management for the module is set to "only me"? In other words, we should keep showing the same %s has connected this and given managing permissions to all admins. You can change who can view this on the dashboard message if the current sharing management is set for all admins, right?

@tofumatt
Copy link
Collaborator

@eugene-manuilov Yes, that's what I wrote in the ACs, but I guess it wasn't clear 😅

The tooltip message when an admin has set a shared module's "manage view access" to "only me" should be updated

I've updated it to be a bit more clear.

@tofumatt tofumatt assigned eugene-manuilov and unassigned tofumatt Jun 21, 2022
@eugene-manuilov
Copy link
Collaborator

Thanks @tofumatt 😊. IB ✔️

@eugene-manuilov eugene-manuilov removed their assignment Jun 22, 2022
@tofumatt tofumatt added the Good First Issue Good first issue for new engineers label Jun 22, 2022
@nfmohit nfmohit self-assigned this Jun 23, 2022
@nfmohit nfmohit removed their assignment Jun 23, 2022
@techanvil techanvil assigned techanvil and unassigned techanvil Jun 23, 2022
@wpdarren wpdarren self-assigned this Jun 23, 2022
@wpdarren
Copy link
Collaborator

QA Update: ✅

Verified:

  • When the "Who can manage view access" column value is set to "Only me" and I login as a second admin and connect my Google account, the tooltip in the "Who can manage view access" column text is according to the AC for the modules where "Only me" was selected.

image

@wpdarren wpdarren 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
Good First Issue Good first issue for new engineers P0 High priority Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants