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

GA4 dashboard survey invokes duplicate trigger #6839

Closed
aaemnnosttv opened this issue Apr 7, 2023 · 3 comments
Closed

GA4 dashboard survey invokes duplicate trigger #6839

aaemnnosttv opened this issue Apr 7, 2023 · 3 comments
Labels
invalid Reason for closed bug issues Module: Analytics Google Analytics module related issues Type: Bug Something isn't working

Comments

@aaemnnosttv
Copy link
Collaborator

aaemnnosttv commented Apr 7, 2023

Bug Description

The view_ga4_dashboard trigger added in #6574 is resulting in a duplicate trigger request being made, which (as can be seen in the screencast below) also results in a duplicate survey timeout being sent.

Steps to reproduce

  • Clear timeout for view_ga4_dashboard survey if present
  • View GA4 dashboard
  • See duplicate request made

Screenshots

Kapture.2023-04-07.at.17.44.55.mp4

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

Acceptance criteria

  • When GA4 Reporting is enabled and the Site Kit dashboard is viewed, the survey trigger request for view_ga4_dashboard should only be made 1 time, not 2 (or more) times.

Implementation Brief

Test Coverage

QA Brief

Changelog entry

@aaemnnosttv aaemnnosttv added Type: Bug Something isn't working P1 Medium priority labels Apr 7, 2023
@mxbclang mxbclang added the Module: Analytics Google Analytics module related issues label Apr 13, 2023
@mxbclang mxbclang added P1 Med and removed P1 Med labels Apr 26, 2023
@tofumatt tofumatt assigned tofumatt and unassigned tofumatt May 5, 2023
@mxbclang mxbclang removed the P1 Low label Jul 17, 2023
@techanvil techanvil added the Good First Issue Good first issue for new engineers label Aug 31, 2023
@zutigrm zutigrm self-assigned this Sep 5, 2023
@zutigrm
Copy link
Collaborator

zutigrm commented Sep 5, 2023

@aaemnnosttv This doesn't seem to be an issue, the survey requests that are sent are actually having different data.

  • data: {triggerID: "view_dashboard"}
  • data: {triggerID: "view_ga4_dashboard"}

Same goes for the timeout request. So in the curent state there is no duplicate request, view_ga4_dashboard survey is actually called only once.

@tofumatt
Copy link
Collaborator

tofumatt commented Sep 5, 2023

(See discussion in Slack about this one: https://10up.slack.com/archives/CBKKQEBR9/p1693918024612529?thread_ts=1693907454.790689&cid=CBKKQEBR9)

@aaemnnosttv I think this can be closed as it's fine to have both surveys trigger, given they're for different things, right? It seems we don't even have a view_ga4_dashboard trigger, but we might want one for the future.

Just confirming with you @aaemnnosttv that it's okay to have both triggers request? If so I think we can close this.

@tofumatt tofumatt assigned aaemnnosttv and unassigned zutigrm Sep 5, 2023
@aaemnnosttv
Copy link
Collaborator Author

Sounds good. These are indeed separate and fine to keep. Thanks all!

@aaemnnosttv aaemnnosttv closed this as not planned Won't fix, can't repro, duplicate, stale Sep 14, 2023
@aaemnnosttv aaemnnosttv removed their assignment Sep 14, 2023
@aaemnnosttv aaemnnosttv added invalid Reason for closed bug issues and removed P1 Medium priority Good First Issue Good first issue for new engineers labels Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid Reason for closed bug issues Module: Analytics Google Analytics module related issues Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants