-
Notifications
You must be signed in to change notification settings - Fork 50
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
fix: Obtain FirstInteraction directly from performance API #1410
Conversation
Asset Size Report
Merging this pull request will result in the following asset size changes:
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1410 +/- ##
==========================================
+ Coverage 88.66% 88.70% +0.04%
==========================================
Files 169 170 +1
Lines 7411 7413 +2
Branches 1504 1505 +1
==========================================
+ Hits 6571 6576 +5
+ Misses 733 728 -5
- Partials 107 109 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Co-authored-by: ptang-nr <[email protected]>
To alleviate the potential performance impact of creating new performance observers to support legacy FirstInteraction timings, a change has been made to query the performance API directly for the presence of first-input data without the use of an observer at logical checkpoints.
Overview
Related Issue(s)
https://new-relic.atlassian.net/browse/NR-365650
Testing
New tests and test fixes for e2e and unit