Skip to content

feat(tracking): impression tracking for home sections#17200

Open
nickskalkin wants to merge 2 commits into
mainfrom
nickskalkin/home-page-rail-impression-tracking
Open

feat(tracking): impression tracking for home sections#17200
nickskalkin wants to merge 2 commits into
mainfrom
nickskalkin/home-page-rail-impression-tracking

Conversation

@nickskalkin

@nickskalkin nickskalkin commented May 12, 2026

Copy link
Copy Markdown
Contributor

The type of this PR is: Feat

This PR solves ONYX-1859

Description

Adds Cohesion RailViewed impression events for every section on the home feed (plus MyBids). An event fires once per mount after the rail stays sufficiently visible for ~1 second, so we capture genuine impressions instead of scroll-by glances.

Update: opened #17206 for item impressions tracking (builds on top of the current PR)

@nickskalkin nickskalkin self-assigned this May 12, 2026
@nickskalkin nickskalkin force-pushed the nickskalkin/home-page-rail-impression-tracking branch from 37e5ac9 to 24c3b9d Compare May 12, 2026 08:24
@relativeci

relativeci Bot commented May 12, 2026

Copy link
Copy Markdown

#8471 Bundle Size — 8.93MiB (+0.1%).

2c01024(current) vs 0a7c789 main#8468(baseline)

Warning

Bundle contains 29 duplicate packages – View duplicate packages

Bundle metrics  Change 4 changes Regression 2 regressions
                 Current
#8471
     Baseline
#8468
Regression  Initial JS 2.89MiB(~+0.01%) 2.89MiB
No change  Initial CSS 0B 0B
No change  Cache Invalidation 97.44% 97.44%
No change  Chunks 111 111
No change  Assets 114 114
Change  Modules 5619(+0.16%) 5610
Regression  Duplicate Modules 659(+0.46%) 656
Change  Duplicate Code 4.99%(+0.2%) 4.98%
No change  Packages 248 248
No change  Duplicate Packages 28 28
Bundle size by type  Change 1 change Regression 1 regression
                 Current
#8471
     Baseline
#8468
Regression  JS 8.7MiB (+0.11%) 8.69MiB
No change  Other 234.14KiB 234.14KiB

Bundle analysis reportBranch nickskalkin/home-page-rail-impre...Project dashboard


Generated by RelativeCIDocumentationReport issue

@nickskalkin nickskalkin force-pushed the nickskalkin/home-page-rail-impression-tracking branch from 24c3b9d to 8e3109b Compare May 12, 2026 08:41
@nickskalkin nickskalkin requested a review from dblandin May 12, 2026 08:41
@nickskalkin nickskalkin marked this pull request as ready for review May 12, 2026 08:41
@nickskalkin nickskalkin force-pushed the nickskalkin/home-page-rail-impression-tracking branch from 393f211 to 8e3109b Compare May 13, 2026 09:22
@dblandin dblandin force-pushed the nickskalkin/home-page-rail-impression-tracking branch from 8e3109b to 9124a25 Compare May 21, 2026 15:37
@dblandin dblandin force-pushed the nickskalkin/home-page-rail-impression-tracking branch from ea1a4b3 to 2c01024 Compare May 26, 2026 13:01
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.

1 participant