Skip to content

DOCS-3301 add documentation to track background events#924

Merged
alai97 merged 3 commits into
masterfrom
Priyanshi/background_events_android
May 31, 2022
Merged

DOCS-3301 add documentation to track background events#924
alai97 merged 3 commits into
masterfrom
Priyanshi/background_events_android

Conversation

@priyanshi-gupta

Copy link
Copy Markdown
Contributor

What does this PR do?

A brief description of the change being made with this pull request.

Motivation

What inspired you to submit this pull request?

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)

@priyanshi-gupta
priyanshi-gupta requested review from a team as code owners May 4, 2022 09:04
Comment thread docs/rum_getting_started.md Outdated

{{< tabs >}}
{{% tab "Java" %}}
```swift

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
```swift
```java

Do we also need different tabs considering the call will look the same for both Kotlin and Java?

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.

+1, if we are adding more tabs, please add this to the end after Line 340:

{{% /tab %}}
{{< /tabs >}}

Comment thread docs/rum_getting_started.md Outdated
@codecov-commenter

codecov-commenter commented May 4, 2022

Copy link
Copy Markdown

Codecov Report

Merging #924 (97596f4) into master (709e70c) will decrease coverage by 0.03%.
The diff coverage is n/a.

❗ Current head 97596f4 differs from pull request most recent head 8767c83. Consider uploading reports for the commit 8767c83 to get more accurate results

@@            Coverage Diff             @@
##           master     #924      +/-   ##
==========================================
- Coverage   83.05%   83.01%   -0.03%     
==========================================
  Files         267      267              
  Lines        9049     9049              
  Branches     1453     1453              
==========================================
- Hits         7515     7512       -3     
- Misses       1139     1140       +1     
- Partials      395      397       +2     
Impacted Files Coverage Δ
...g/internal/domain/event/DdSpanToSpanEventMapper.kt 96.08% <0.00%> (-3.92%) ⬇️
.../android/rum/internal/monitor/DatadogRumMonitor.kt 94.48% <0.00%> (-0.61%) ⬇️
...android/rum/internal/ndk/DatadogNdkCrashHandler.kt 87.93% <0.00%> (ø)

@kayayarai
kayayarai requested a review from alai97 May 4, 2022 14:59
Comment thread docs/rum_getting_started.md Outdated
Comment thread docs/rum_getting_started.md Outdated

{{< tabs >}}
{{% tab "Java" %}}
```swift

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.

+1, if we are adding more tabs, please add this to the end after Line 340:

{{% /tab %}}
{{< /tabs >}}

@alai97 alai97 changed the title add documentation to track background events DOCS-3301 add documentation to track background events May 9, 2022
@alai97 alai97 added the documentation Improvements or additions to documentation label May 9, 2022
alai97 and others added 2 commits May 9, 2022 14:47
@priyanshi-gupta
priyanshi-gupta requested review from 0xnm and alai97 May 27, 2022 10:36
@alai97
alai97 merged commit 017a332 into master May 31, 2022
@alai97
alai97 deleted the Priyanshi/background_events_android branch May 31, 2022 16:19
@xgouchet xgouchet added this to the 1.14.0 milestone Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size-tiny

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants