(#5278) Blog Analytics Helper#5291
Open
dlescarbeau wants to merge 3 commits into
Open
Conversation
dlescarbeau
marked this pull request as draft
June 30, 2026 18:58
dlescarbeau
changed the title
dlescarbeau
force-pushed
the
ticket/5278-blog-series-list-analytics
branch
from
July 1, 2026 14:38
1f342ad to
76fe1c4
Compare
There was a problem hiding this comment.
Pull request overview
Adds dedicated analytics tracking for NCIDS blog experiences (blog series list, right-rail links, and pager links) to meet the new EDDL event requirements from issues #5278–#5280, while preventing legacy dynamic-list tracking from firing on legacy blog series pages.
Changes:
- Introduces
blog-analytics-helperwith three analytics helpers that emit the required EDDLtrackOtherevents. - Adds Jest tests validating payload shape, truncation, pageType handling, and “only once per click” behavior.
- Updates the blogs entrypoint to initialize the new helpers and adjusts legacy jQuery analytics to skip blog series dynamic-list tracking.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docroot/profiles/custom/cgov_site/themes/custom/ncids_trans/front-end/src/lib/core/analytics/blog-analytics-helper.ts | Implements new EDDL click events for blog series list, right rail, and pager interactions. |
| docroot/profiles/custom/cgov_site/themes/custom/ncids_trans/front-end/src/lib/core/analytics/tests/blog-analytics-helper.test.ts | Adds unit tests covering new tracking behavior and duplicate-handler prevention. |
| docroot/profiles/custom/cgov_site/themes/custom/ncids_trans/front-end/src/entrypoints/blogs/blogs.ts | Wires the new analytics helpers into the blogs bundle on DOMContentLoaded. |
| docroot/profiles/custom/cgov_site/themes/custom/cgov/cgov_common/src/libraries/analytics/analytics.js | Prevents legacy DynamicListItemClick tracking from attaching on legacy blog series pages. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+14
to
+37
| const getBlogTitle = (collectionItem: HTMLElement): string => | ||
| collectionItem | ||
| .querySelector('.usa-collection__heading a') | ||
| ?.textContent?.trim() || '_ERROR_'; | ||
|
|
||
| const blogSeriesListClickHandler = | ||
| (collectionItems: HTMLElement[]) => | ||
| (evt: Event): void => { | ||
| const target = evt.currentTarget as HTMLElement; | ||
| const collectionItem = target.closest( | ||
| COLLECTION_ITEM_SELECTOR | ||
| ) as HTMLElement; | ||
|
|
||
| if (!collectionItem) { | ||
| return; | ||
| } | ||
|
|
||
| const collectionItemIndex = collectionItems.indexOf(collectionItem); | ||
|
|
||
| trackOther(BLOG_SERIES_LIST_EVENT_NAME, BLOG_SERIES_LIST_EVENT_NAME, { | ||
| location: 'Body', | ||
| componentType: 'Blog Series List', | ||
| title: getBlogTitle(collectionItem).slice(0, 50), | ||
| linkArea: 'Title', |
dlescarbeau
force-pushed
the
ticket/5278-blog-series-list-analytics
branch
2 times, most recently
from
July 2, 2026 15:22
1469be3 to
06d2166
Compare
dlescarbeau
marked this pull request as ready for review
July 2, 2026 15:23
Closes #5278 update for 5278 commit updates analytics per copilot feedback
dlescarbeau
force-pushed
the
ticket/5278-blog-series-list-analytics
branch
from
July 2, 2026 15:23
06d2166 to
1b9f733
Compare
ODE DeploymentCode has been deployed to ODE 1152. |
dlescarbeau
requested review from
KateMashkinaNIH,
andyvanavery31,
davisaw and
dev-rana-publicis
July 7, 2026 14:53
|
@dlescarbeau Blog Series List Clicks and Blog Pager Clicks both look good! |
- adds tracking to blog categories - adds tracking to blog archive box Closes #5279
dlescarbeau
force-pushed
the
ticket/5278-blog-series-list-analytics
branch
from
July 7, 2026 17:51
1b9f733 to
e27c2cd
Compare
Contributor
Author
This is being addressed |
andyvanavery31
removed their request for review
July 7, 2026 20:04
KateMashkinaNIH
approved these changes
Jul 8, 2026
dev-rana-publicis
approved these changes
Jul 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #5278
Closes #5279
Closes #5280