Skip to content

feat: Expose library versions used to capture session replay data #809

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

Merged
merged 2 commits into from
Nov 10, 2023

Conversation

metal-messiah
Copy link
Member

Expose library versions used to capture session replay data, to help sync visualizations in the UI Replayer components across version migrations.

Overview

This PR exposes the rrweb lib version used to capture the reply data to a public namespace to be used for UI syncing purposes.

Related Issue(s)

https://new-relic.atlassian.net/browse/NR-178304

Testing

Tests have been updated to account for the change

Copy link

github-actions bot commented Nov 9, 2023

Asset Size Report

Merging this pull request will result in the following asset size changes:

Agent Asset Previous Size New Size Diff
lite loader 30.52 kB / 10.47 kB (gzip) 30.53 kB / 10.47 kB (gzip) 0.03% / 0.02% (gzip)
lite async-chunk 45.45 kB / 15.02 kB (gzip) 45.45 kB / 15.02 kB (gzip) 0% / 0% (gzip)
pro loader 47.92 kB / 15.88 kB (gzip) 47.93 kB / 15.88 kB (gzip) 0.02% / 0.01% (gzip)
pro async-chunk 73.6 kB / 23.44 kB (gzip) 73.6 kB / 23.44 kB (gzip) 0% / -0.01% (gzip)
spa loader 54.38 kB / 17.86 kB (gzip) 54.39 kB / 17.86 kB (gzip) 0.02% / 0.03% (gzip)
spa async-chunk 88.09 kB / 27.8 kB (gzip) 88.08 kB / 27.8 kB (gzip) 0% / -0.01% (gzip)
lite-polyfills loader 121.98 kB / 39.44 kB (gzip) 121.99 kB / 39.44 kB (gzip) 0.01% / 0% (gzip)
lite-polyfills async-chunk 57.81 kB / 17.2 kB (gzip) 57.81 kB / 17.2 kB (gzip) 0% / 0% (gzip)
pro-polyfills loader 141.49 kB / 45.32 kB (gzip) 141.5 kB / 45.32 kB (gzip) 0.01% / 0% (gzip)
pro-polyfills async-chunk 101.34 kB / 27.21 kB (gzip) 101.34 kB / 27.21 kB (gzip) 0% / 0% (gzip)
spa-polyfills loader 149.51 kB / 47.45 kB (gzip) 149.53 kB / 47.46 kB (gzip) 0.01% / 0% (gzip)
spa-polyfills async-chunk 116.5 kB / 31.67 kB (gzip) 116.5 kB / 31.67 kB (gzip) 0% / 0% (gzip)

Copy link

codecov bot commented Nov 9, 2023

Codecov Report

Merging #809 (331982c) into main (92d864c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #809   +/-   ##
=======================================
  Coverage   78.50%   78.50%           
=======================================
  Files         142      142           
  Lines        6256     6256           
  Branches     1217     1217           
=======================================
  Hits         4911     4911           
  Misses       1148     1148           
  Partials      197      197           
Flag Coverage Δ
unit-tests 55.27% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/features/session_replay/aggregate/index.js 94.31% <ø> (ø)
src/loaders/configure/nonce.js 83.33% <100.00%> (ø)

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@metal-messiah metal-messiah merged commit bc275ee into main Nov 10, 2023
@metal-messiah metal-messiah deleted the rrweb-metadata branch November 10, 2023 17:41
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.

2 participants