Skip to content

Add setVitalsUpdateFrequency to the doc#1015

Merged
plousada merged 2 commits into
masterfrom
plousada/add-set-mobile-vitals-freq-doc
Aug 23, 2022
Merged

Add setVitalsUpdateFrequency to the doc#1015
plousada merged 2 commits into
masterfrom
plousada/add-set-mobile-vitals-freq-doc

Conversation

@plousada

Copy link
Copy Markdown
Member

Add setVitalsUpdateFrequency function to the advanced configuration doc.

What does this PR do?

Adds setVitalsUpdateFrequency function and description to the advanced configuration doc.

Motivation

Parity with iOS doc.

Additional Notes

Anything else we should know when reviewing?

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)

Add setVitalsUpdateFrequency function to the advanced configuration doc.
@plousada
plousada requested review from a team as code owners August 18, 2022 14:25
Comment thread docs/configure_rum_android_sdk.md Outdated

@alai97 alai97 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, one copy suggestion!

@codecov-commenter

codecov-commenter commented Aug 19, 2022

Copy link
Copy Markdown

Codecov Report

Merging #1015 (6625e4c) into master (8a558ac) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1015      +/-   ##
==========================================
- Coverage   83.10%   83.06%   -0.04%     
==========================================
  Files         267      267              
  Lines        9079     9079              
  Branches     1461     1461              
==========================================
- Hits         7545     7541       -4     
- Misses       1135     1138       +3     
- Partials      399      400       +1     
Impacted Files Coverage Δ
...android/log/internal/logger/TelemetryLogHandler.kt 75.00% <0.00%> (-25.00%) ⬇️
...ndroid/core/internal/persistence/file/EventMeta.kt 70.00% <0.00%> (-10.00%) ⬇️
...id/webview/internal/rum/WebViewRumEventConsumer.kt 76.74% <0.00%> (-9.30%) ⬇️
...rc/main/java/com/datadog/opentracing/DDTracer.java 56.07% <0.00%> (+0.42%) ⬆️
...al/persistence/file/batch/BatchFileOrchestrator.kt 94.95% <0.00%> (+2.02%) ⬆️

@plousada
plousada merged commit bfe7af2 into master Aug 23, 2022
@plousada
plousada deleted the plousada/add-set-mobile-vitals-freq-doc branch August 23, 2022 09:57
@xgouchet xgouchet added this to the 1.15.0 milestone Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants