Skip to content

Move recent docs edits which were done in Android SDK repo#18936

Merged
rtrieu merged 8 commits into
masterfrom
nogorodnikov/move-edits-from-android-sdk-repo
Jul 21, 2023
Merged

Move recent docs edits which were done in Android SDK repo#18936
rtrieu merged 8 commits into
masterfrom
nogorodnikov/move-edits-from-android-sdk-repo

Conversation

@0xnm

@0xnm 0xnm commented Jul 13, 2023

Copy link
Copy Markdown
Member

What does this PR do?

I noticed that some edit for Logs and Tracing documentation were done in Android SDK repo since these docs were actually moved to this repo.

Android SDK repo should have these files removed, so I moved these edit commit-by-commit here (you can check commit history here).

Also I noticed that some docs still reference the docs in the Android SDK repo, so I removed such references.

Motivation

Additional Notes


Reviewer checklist

  • Review the changed files.
  • Review the URLs listed in the Preview section.
  • Check images for PII
  • Review any mentions of "Contact Datadog support" for internal support documentation.

@0xnm
0xnm requested review from a team as code owners July 13, 2023 10:08
@git-thuerk-done git-thuerk-done added the editorial review Waiting on a more in-depth review label Jul 13, 2023
@git-thuerk-done

Copy link
Copy Markdown
Contributor

Thanks! I have created Docs card for someone to review this further:
https://datadoghq.atlassian.net/browse/DOCS-5878

@rtrieu rtrieu 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.

LGTM!

@rtrieu
rtrieu merged commit 42527e7 into master Jul 21, 2023
@rtrieu
rtrieu deleted the nogorodnikov/move-edits-from-android-sdk-repo branch July 21, 2023 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial review Waiting on a more in-depth review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants