Skip to content

fix: getCurrentSessionId returns correct value#1817

Merged
fuzzybinary merged 1 commit into
developfrom
jward/RUM-1962-session-id-acessor
Jan 10, 2024
Merged

fix: getCurrentSessionId returns correct value#1817
fuzzybinary merged 1 commit into
developfrom
jward/RUM-1962-session-id-acessor

Conversation

@fuzzybinary

Copy link
Copy Markdown
Member

What does this PR do?

Fix getCurrentSessionId to actually return the correct session id for the currently active session.

refs: RUM-1962

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@fuzzybinary
fuzzybinary requested review from a team as code owners January 10, 2024 16:35
@fuzzybinary
fuzzybinary force-pushed the jward/RUM-1962-session-id-acessor branch from 77c852d to 34221a2 Compare January 10, 2024 16:57
@fuzzybinary
fuzzybinary force-pushed the jward/RUM-1962-session-id-acessor branch from 34221a2 to c0b2f35 Compare January 10, 2024 17:02
@codecov-commenter

codecov-commenter commented Jan 10, 2024

Copy link
Copy Markdown

Codecov Report

Merging #1817 (c0b2f35) into develop (0f3e356) will increase coverage by 0.08%.
The diff coverage is 50.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1817      +/-   ##
===========================================
+ Coverage    83.38%   83.46%   +0.08%     
===========================================
  Files          466      466              
  Lines        16388    16370      -18     
  Branches      2456     2461       +5     
===========================================
- Hits         13665    13663       -2     
+ Misses        2045     2031      -14     
+ Partials       678      676       -2     
Files Coverage Δ
.../android/rum/internal/monitor/DatadogRumMonitor.kt 84.81% <50.00%> (-0.14%) ⬇️

... and 23 files with indirect coverage changes

@fuzzybinary
fuzzybinary merged commit 723fbc8 into develop Jan 10, 2024
@fuzzybinary
fuzzybinary deleted the jward/RUM-1962-session-id-acessor branch January 10, 2024 19:15
@xgouchet xgouchet added this to the 2.5.0 milestone Feb 19, 2024
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.

4 participants