-
Notifications
You must be signed in to change notification settings - Fork 7
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: improving enhancer loading timing #175
Conversation
This PR will trigger no release when merged. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #175 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 1 1
Lines 66 69 +3
=========================================
+ Hits 66 69 +3 ☔ View full report in Codecov by Sentry. |
# [2.2.0](v2.1.4...v2.2.0) (2024-08-13) ### Bug Fixes * **404:** send referrer ([84a6556](84a6556)) * release process ([#191](#191)) ([f40a949](f40a949)) ### Features * **bundle:** release a standalone 404.js that can be used to track error pages ([33d70b5](33d70b5)) * improve errors capture ([#189](#189)) ([d7448eb](d7448eb)) * improving enhancer loading timing ([#175](#175)) ([d9c690e](d9c690e)) * trigger release ([7d8f4e0](7d8f4e0))
🎉 This PR is included in version 2.2.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This seems to be messing with adobe/helix-rum-enhancer#256 need to understand why |
Faster loading of rum-enhancer in the default behavior means that the URL manipulation needed for the test is coming too late. Fixed in adobe/helix-rum-enhancer@bd2eae3 |
Introduces a new capability: let the consumer define when to load the
enhancer
. To do so:window.hlx.RUM_MANUAL_ENHANCE = true;
sampleRUM();
sampleRUM.enhance();