Migrate RUM SDK integrated libraries pages to Cdocs pattern#36012
Open
Migrate RUM SDK integrated libraries pages to Cdocs pattern#36012
Conversation
Converts all 5 RUM SDK integrated libraries pages (Android, iOS, Flutter,
Kotlin Multiplatform, React Native) to the same Cdocs .mdoc.md structure
used by setup, advanced_configuration, and data_collected pages.
- Extracts content into shared partials under sdk/integrated_libraries/
- Creates individual SDK wrapper pages in application_monitoring/{sdk}/
- Adds unified view at client_sdks/integrated_libraries.mdoc.md with content_filters
- Removes old manually-authored .md files
- Browser, Roku, and Unity excluded (no integrated libraries pages)
- Fixes Vale style warnings (refer to, please, ensures, NPM → npm)
- Fixes incorrect [6] → [11] reference link in Flutter Dio section
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
github-actions
bot
added
Architecture
Contributor
Contributor
|
Created https://datadoghq.atlassian.net/browse/DOCS-14011 for review |
git-thuerk-done
requested changes
Apr 15, 2026
Contributor
git-thuerk-done
left a comment
git-thuerk-done
left a comment
There was a problem hiding this comment.
Hi @rtrieu this is looking good! I mostly have some suggestions for the Browser, Unity, and Roku libraries, let me know if you have any questions!
|
|
||
| Select a platform below to see the libraries available for integration with the Datadog SDK. | ||
|
|
||
| <!-- Browser: no integrated libraries page --> |
Contributor
There was a problem hiding this comment.
Suggested change
| <!-- Browser: no integrated libraries page --> | |
| <!-- Browser: no integrated libraries page --> |
I see you have a comment here as well as for Roku and Unity, however there is still a prompt for the user to select a platform "below" and then just blank space. In addition to removing the "select a platform below" sentence, I would suggest for these to add:
{% if equals($platform, "browser") %}
There are no integrated libraries for the Browser SDK.
{% /if %}
And then the same for Roku and Unity
|
|
||
| ## Overview | ||
|
|
||
| Select a platform below to see the libraries available for integration with the Datadog SDK. |
Contributor
There was a problem hiding this comment.
Suggested change
| Select a platform below to see the libraries available for integration with the Datadog SDK. |
See above comment, we should remove this due to the condition for Browser, Roku ,and Unity not containing any content. Also the filter UI already implies selection.
| ### Setup | ||
|
|
||
| **Note**: This package is an integration for [`react-navigation`][1] library, please make sure you first install and setup the core `mobile-react-native` SDK. | ||
| **Note**: This package is an integration for [`react-navigation`][1] library. Make sure you first install and setup the core `mobile-react-native` SDK. |
Contributor
There was a problem hiding this comment.
Suggested change
| **Note**: This package is an integration for [`react-navigation`][1] library. Make sure you first install and setup the core `mobile-react-native` SDK. | |
| **Note**: This package is an integration for [`react-navigation`][1] library. Make sure you first install and set up the core `mobile-react-native` SDK. |
"setup" here should be "set up" (verb form), or you could use "configure"
| ## React Native Navigation | ||
|
|
||
| **Note**: This package is an integration for `react-native-navigation` library. Please make sure you first install and setup the core `mobile-react-native` SDK. | ||
| **Note**: This package is an integration for `react-native-navigation` library. Make sure you first install and setup the core `mobile-react-native` SDK. |
Contributor
There was a problem hiding this comment.
Suggested change
| **Note**: This package is an integration for `react-native-navigation` library. Make sure you first install and setup the core `mobile-react-native` SDK. | |
| **Note**: This package is an integration for `react-native-navigation` library. Make sure you first install and set up the core `mobile-react-native` SDK. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do? What is the motivation?
Converts RUM SDK integrated libraries pages (Android, Flutter, iOS, Kotlin Multiplatform, React Native) to the Cdocs
.mdoc.mdpattern used by the setup, advanced configuration, and data collected pages.layouts/shortcodes/mdoc/en/sdk/integrated_libraries/integrated_libraries.mdfiles with.mdoc.mdwrapper pagesclient_sdks/integrated_libraries.mdoc.mdwithcontent_filtersfor per-SDK contentprivate: trueMerge instructions
Merge readiness:
Additional notes