When using dashboard sharing, if there are modules shared by different administrators, the "Reset sharing permissions" CTA will only reset permissions for the administrator selecting that option. It won't reset permissions for modules not owned by that administrator. This can cause issues as the other administrator may not longer be active, or their WordPress account was removed, leaving a message to "Contact managing user" when that user no longer exists.
To address this, considering making the "Reset sharing permissions" option applicable to all modules, even those not owned by the user resetting.
Steps to reproduce
Set up SK on a site with only Search Console active from a WordPress administrator account (adminA)
Login with a different WordPress administrator account (adminB). Set up Site Kit as a secondary user and connect the Google Analytics or AdSense module. After doing so, grant sharing permissions to other users via Dashboard Sharing.
Log back into the site from the original administrator account (adminA).
Share the Search Console module (so the option to "Reset Sharing Permissions" appears
Use the CTA to "Reset Sharing Permissions"
Only the Search Console module sharing permissions are reset, and not GA.
Note: A fix here would mean we can also close #9617
Do not alter or remove anything below. The following sections will be managed by moderators only.
jamesozzie
commented
2 weeks ago
Bug Description
When using dashboard sharing, if there are modules shared by different administrators, the "Reset sharing permissions" CTA will only reset permissions for the administrator selecting that option. It won't reset permissions for modules not owned by that administrator. This can cause issues as the other administrator may not longer be active, or their WordPress account was removed, leaving a message to "Contact managing user" when that user no longer exists.
To address this, considering making the "Reset sharing permissions" option applicable to all modules, even those not owned by the user resetting.
Steps to reproduce
Note: A fix here would mean we can also close #9617
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
Implementation Brief
Test Coverage
QA Brief
Changelog entry