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

Adjust analytics opt out page events #825

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bsmartin-ep
Copy link

Page to edit

url:
https://www.va.gov/analytics-opt-out.html

For:
department-of-veterans-affairs/va.gov-team#36585

Origin of request (internal/stakeholder/user feedback)

Analytics & Insights team audit of Domo dashboards

Description of what's needed (edits/link changes/additions)

We have a chicken-and-the-egg situation with the analytics opt out event being fired:

recordEvent({ event: 'analytics-opt-out', 'internal-user': 'true' });

The internal user flag is set to true at the property level so this event itself is not making it to the All view (which is what is exported to BigQuery). Because of this, we can not create a list of GA user IDs and retroactively remove data from internal users. This is causing discrepancies with our Domo dashboards and GA.

I just doubled up the event (without the internal user param), but open to suggestions for a pre-event name.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants