-
Notifications
You must be signed in to change notification settings - Fork 69
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
Add existing GA events to New Homepage #12065
Comments
Per Dave sync today: not soft-launch blocking for turning on the modal, but is hard-launch blocking for cutting over to the new homepage / taking down the old. |
Search AC not valid, the rest may still be and need work. Rolling into next sprint. |
@jtmst, did you get to the bottom of this? It looks above like your PR just for a search change was Closed rather than Merged 2 weeks ago. But where is the rest of your code? |
@wesrowe department-of-veterans-affairs/content-build#1429 Lost track of this PR around on site and it didnt actually get merged, its in now. |
@jtmst, I just tested these fixes in staging. Most ACs are validated in GA interface (a few looked good in AdSwerve but there was no data in GA; likely because no one clicked in staging). However, one group of events was (mostly) missing from AdSwerve:
|
PR merged, in Review/QA to verify after deploy. |
The above were solved around 1/31, forgot to close ticket. Closing. |
NOTE: This is a temporary fix; #11965 tracks the desired long-term fix
Description
In #11467 we added homepage-specific events to the New Home Page; however: they can't be tracked as written due to limitations in Google Tag Manager (GTM). A custom implementation is required on GTM. (Platform Analytics team has been asked to do it.)
As a temporary measure, we want to add GA events that work within the existing GTM scheme:
New events detailed in this Mural area
Screenshot of mural:
Acceptance Criteria
ec: Interactions ea: Default Button CTA - [button text] - [optional: button color]
Search input in-page gets same GA event as header search, e.g.No-op: Turned out to be already presentec: Search ea: Search Results Returned ~ Type Ahead Enabled ~ All VA.gov ~ [search term input] el: view_search_results
ec: Interactions ea: Navigation - Zone One - ->[optional: path of destination] - [required: link text] - [optional/not defined]
ec: Interactions ea: Navigation - Zone One - ->[optional: path of destination] - [required: link text] - [optional/not defined]
ec: Interactions ea: Default Button CTA - [button text] - [optional: button color]
CMS Team
Please check the team(s) that will do this work.
Program
Platform CMS Team
Sitewide Crew
⭐️ Sitewide CMS
⭐️ Public Websites
⭐️ Facilities
⭐️ User support
The text was updated successfully, but these errors were encountered: