Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Merge feature/sr-web-view-support #1959

Merged
merged 30 commits into from
Apr 2, 2024
Merged

Merge feature/sr-web-view-support #1959

merged 30 commits into from
Apr 2, 2024

Conversation

xgouchet
Copy link
Member

@xgouchet xgouchet commented Apr 2, 2024

What does this PR do?

Merge feature/sr-web-view-support into develop

mariusc83 and others added 30 commits January 17, 2024 09:47
@mariusc83
RUM-2484 Provide the SR privacy level through the DatadogEventBridge
44a906a
@mariusc83
RUM-2590 Add recorded WebView scenario into the sample app
bf58d3a
@mariusc83
RUM-2483 Introduce the WebViewWireframeMapper
8d777ab
@mariusc83
RUM-2487 Provide the WebViewRecordConsumer for the JS bridge
7803795
@mariusc83
Do not force new batch when persisting SR records
598cfe2
@mariusc83
RUM-2622 Change the SR requestbody to match the new endpoint payload …
668b59b 
…format
@mariusc83
RUM-2572 WebView - force fullsnapshot in browser sdk when RUM view ch…
6d7088a 
…anged
@mariusc83
Revert "Merge pull request #1807 from DataDog/mconstantin/rum-2572/fo…
13a7578 
…rce-browser-sdk-fullsnapsot-when-context-changed"

This reverts commit de212fc, reversing
changes made to d02623f.
@mariusc83
RUM-2875 Introduce the FeatureContextUpdateLitener API
474b8b0
@mariusc83
Merge pull request #1829 from DataDog/mconstantin/rum-2875/propagate-…
2470aa7 
…feature-context-update-through-msb

RUM-2875 Introduce the FeatureContextUpdateLitener API
@mariusc83
RUM-2485 Provide the parent container information for browser rum events
688c2a7
@mariusc83
Merge pull request #1831 from DataDog/mconstantin/rum-2485/provide-th…
fce374b 
…e-container-data-into-browser-rum-events

RUM-2485 Provide the parent container information for browser rum events
@mariusc83
Merge branch 'develop' into sr-web-view-support/merge/develop
0e6f41a
@mariusc83
Merge pull request #1849 from DataDog/sr-web-view-support/merge/develop
64b5013 
Merge develop branch
@xgouchet
RUM-3281 add getCapabilities to the Webview bridge
e22c43e
@xgouchet
Merge pull request #1871 from DataDog/xgouchet/RUM-3281/webview_get_c…
731551f 
…apabilities

RUM-3281 Implement webview bridge getCapabilities
@xgouchet
RUM-3302 detect full snapshot from webview sr
c3b45d2
@xgouchet
Merge pull request #1908 from DataDog/xgouchet/RUM-3302/handle_has_fu…
dec970a 
…ll_snapshot

RUM-3302 detect full snapshot from webview sr
@xgouchet
Merge branch 'develop' into xgouchet/merge_develop
7b32a80
@xgouchet
Merge pull request #1915 from DataDog/xgouchet/merge_develop
df4c3ca 
Merge develop
@xgouchet
Merge remote-tracking branch 'origin/develop' into xgouchet/merge_dev…
6ca9992 
…elop_sr_webview
@xgouchet
Merge pull request #1917 from DataDog/xgouchet/merge_develop_sr_webview
2027b2d 
Merge develop into sr webview feature branch
@xgouchet
Merge remote-tracking branch 'origin/develop' into xgouchet/merge_dev…
319193a 
…elop_sr_webview_3
@xgouchet
Merge pull request #1922 from DataDog/xgouchet/merge_develop_sr_webvi…
26824c4 
…ew_3

merge develop into feature/sr-webview (3)
@xgouchet
RUM-3532 add webview use cases in sample
ff69432
@xgouchet
RUM-3532 keep webviews hidden in Session Replay
140ed18
@xgouchet
RUM-3532 fix PR comments
764ebdf
@xgouchet
Merge pull request #1949 from DataDog/xgouchet/RUM-3532/keep_webview_…
04186b5 
…wireframe_hidden

RUM-3532 keep webview wireframe hidden
@xgouchet
Merge branch 'develop' into xgouchet/srwv_merge_develop
283c82b
@xgouchet
Merge pull request #1958 from DataDog/xgouchet/srwv_merge_develop
da8c7a4 
Merge develop
@xgouchet xgouchet requested review from a team as code owners April 2, 2024 11:54
@codecov-commenter
Copy link

Codecov Report

Merging #1959 (da8c7a4) into develop (0f0d085) will increase coverage by 0.21%.
Report is 2 commits behind head on develop.
The diff coverage is 88.38%.

Additional details and impacted files 
@@             Coverage Diff             @@
##           develop    #1959      +/-   ##
===========================================
+ Coverage    83.21%   83.42%   +0.21%     
===========================================
  Files          483      489       +6     
  Lines        17584    17848     +264     
  Branches      2605     2661      +56     
===========================================
+ Hits         14632    14889     +257     
+ Misses        2257     2235      -22     
- Partials       695      724      +29
Files Coverage Δ
...lin/com/datadog/android/rum/internal/RumFeature.kt 93.29% <ø> (+0.30%) ⬆️
.../datadog/android/rum/internal/domain/RumContext.kt 100.00% <100.00%> (ø)
.../android/rum/internal/domain/scope/RumViewScope.kt 94.52% <100.00%> (+0.76%) ⬆️
...adog/android/sessionreplay/SessionReplayPrivacy.kt 100.00% <100.00%> (ø)
...oid/sessionreplay/internal/SessionReplayFeature.kt 100.00% <100.00%> (+0.91%) ⬆️
...onreplay/internal/net/SegmentRequestBodyFactory.kt 100.00% <100.00%> (ø)
...essionreplay/internal/net/SegmentRequestFactory.kt 100.00% <100.00%> (ø)
...id/sessionreplay/internal/processor/BoundsUtils.kt 68.18% <100.00%> (+68.18%) ⬆️
...ssionreplay/internal/processor/MobileSegmentExt.kt 100.00% <100.00%> (ø)
...replay/internal/processor/RecordedDataProcessor.kt 96.25% <100.00%> (ø)
... and 20 more

... and 18 files with indirect coverage changes

@xgouchet xgouchet merged commit a107d7c into develop Apr 2, 2024
23 checks passed
@xgouchet xgouchet deleted the feature/sr-web-view-support branch April 2, 2024 13:57
@xgouchet xgouchet added this to the 2.8.x milestone Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xnm 0xnm 0xnm approved these changes

@mariusc83 mariusc83 mariusc83 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.8.x
Development

Successfully merging this pull request may close these issues.
4 participants
@xgouchet @codecov-commenter @mariusc83 @0xnm