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

Prevent error duplication in Analytics setup & settings #7276

Closed
mohitwp opened this issue Jul 10, 2023 · 4 comments
Closed

Prevent error duplication in Analytics setup & settings #7276

mohitwp opened this issue Jul 10, 2023 · 4 comments
Labels
Module: Analytics Google Analytics module related issues P2 Low priority Type: Bug Something isn't working

Comments

@mohitwp
Copy link
Collaborator

mohitwp commented Jul 10, 2023

Bug Description

Duplicate errors appearing on Analytics setup & settings page when user a/c which have only 'View' access try to set up analytics.

Steps to reproduce

  • Start set up for Analytics without having granted the edit scope
  • Choose an account that the user only has Viewer-level permissions for
  • Choose "Set up a new property".
  • Submit the form
  • When prompted, grant the additional scope to create the property
  • See duplicate errors on settings page.

Screenshots

image

Recording.443.mp4

Additional Context

  • PHP Version:
  • OS: [e.g. iOS]
  • Browser: [e.g. chrome, safari]
  • Plugin Version: [e.g. 22]
  • Device: [e.g. iPhone6]

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

Acceptance criteria

  • Since Universal Analytics has been removed from Site Kit's UI, when creating a new GA4 Property, the UA Insufficient Permissions error should not be rendered.

Implementation Brief

  • In assets/js/modules/analytics/components/setup/SetupForm.js:
    • Remove rendering the StoreErrorNotices component for the analytics data store. This should be a safe fix since we do not allow creating or editing UA properties anymore.

Test Coverage

  • No new tests required.

QA Brief

  • Follow the steps to reproduce.
  • Verify that the error notices for UA are not displayed.

Changelog entry

  • Update Analytics setup and settings views to avoid showing duplicate errors.
@mohitwp mohitwp added Module: Analytics Google Analytics module related issues Type: Bug Something isn't working labels Jul 10, 2023
@aaemnnosttv aaemnnosttv added the P2 Low priority label Jul 11, 2023
@nfmohit nfmohit changed the title Duplicate errors appearing on settings page when user a/c which have only 'View' access try to set up analytics. Prevent error duplication in Analytics setup & settings Oct 23, 2023
@nfmohit
Copy link
Collaborator

nfmohit commented Oct 23, 2023

Just a heads-up that this happens in both Analytics setup & settings screens, and the solution would be quite similar I believe. As a result, I have slightly updated the title and description to refer that.

@jimmymadon
Copy link
Collaborator

jimmymadon commented Dec 3, 2023

Can confirm that this happens on the latest develop branch too, despite removing UA from the UI - but only for the Setup Form (and not for the Settings Form anymore since we have removed the usage of assets/js/modules/analytics/components/settings/SettingsControls.js

Screenshot 2023-12-04 at 00 03 24

@jimmymadon jimmymadon removed their assignment Dec 4, 2023
@techanvil techanvil self-assigned this Dec 5, 2023
@techanvil
Copy link
Collaborator

AC ✅
IB ✅

Nice one @jimmymadon :)

@techanvil techanvil removed their assignment Dec 5, 2023
@hussain-t hussain-t self-assigned this Jan 8, 2024
@hussain-t hussain-t removed their assignment Jan 8, 2024
eugene-manuilov added a commit that referenced this issue Jan 12, 2024
@eugene-manuilov eugene-manuilov removed their assignment Jan 12, 2024
@mohitwp mohitwp self-assigned this Jan 16, 2024
@mohitwp
Copy link
Collaborator Author

mohitwp commented Jan 17, 2024

QA Update ✅

  • Tested on dev environment.
  • Verified that on dev branch error duplication error is now resolved.
  • Verified that on dev branch UA error is not showing.
Recording.739.mp4

image

@mohitwp mohitwp removed their assignment Jan 17, 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 P2 Low priority Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

8 participants