-
Notifications
You must be signed in to change notification settings - Fork 44
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
feat: Capture Page Resource Assets #1257
base: main
Are you sure you want to change the base?
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 #1257 +/- ##
==========================================
- Coverage 89.25% 89.17% -0.08%
==========================================
Files 170 170
Lines 7326 7329 +3
Branches 1476 1480 +4
==========================================
- Hits 6539 6536 -3
- Misses 667 672 +5
- Partials 120 121 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
Add detection for page resource assets which captures data as a BrowserPerformance event. This feature is being shipped in an experimental state and is off by default. To opt in to using this feature, it can be turned on by setting the config value
init.performance.resources.enabled = true
in Copy/Paste or NPM installations.Overview
See this CDD for the base behavior of this feature.
Related Issue(s)
https://new-relic.atlassian.net/browse/NR-330514
Testing
New unit and e2e tests have been added to validate the behavior.