Skip to content

[DOCS-13867] Fix DatadogProviderConfiguration constructor in Expo setup code sample#35855

Open
iadjivon wants to merge 1 commit intomasterfrom
ida.adjivon/DOCS-13867-rum-react-native-code-change
Open

[DOCS-13867] Fix DatadogProviderConfiguration constructor in Expo setup code sample#35855
iadjivon wants to merge 1 commit intomasterfrom
ida.adjivon/DOCS-13867-rum-react-native-code-change

Conversation

@iadjivon
Copy link
Copy Markdown
Contributor

@iadjivon iadjivon commented Apr 8, 2026

What does this PR do? What is the motivation?

Fixes DOCS-13867

The Expo setup code sample for DatadogProviderConfiguration used invalid JavaScript syntax — configuration properties (site, service, verbosity, rumConfiguration, etc.) were passed as flat positional arguments instead of being wrapped in an options object as the constructor expects.

Constructor signature (from SDK source):

constructor(clientToken, env, trackingConsent, options)

Changes:

  • Add trackingConsent as the 3rd argument
  • Wrap site, service, verbosity, rumConfiguration, logsConfiguration, and traceConfiguration inside an options object (4th argument)

Verified against:

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

🤖 Generated with Claude Code

@iadjivon iadjivon requested review from a team as code owners April 8, 2026 19:31
@iadjivon iadjivon added the WORK IN PROGRESS No review needed, it's a wip ;) label Apr 8, 2026
@github-actions github-actions bot added Architecture Everything related to the Doc backend cdocs labels Apr 8, 2026
The code sample used invalid JavaScript syntax, passing configuration
properties as flat positional arguments instead of wrapping them in an
options object as the SDK constructor expects.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@iadjivon iadjivon force-pushed the ida.adjivon/DOCS-13867-rum-react-native-code-change branch from 5c740c5 to 20fe493 Compare April 8, 2026 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Architecture Everything related to the Doc backend cdocs WORK IN PROGRESS No review needed, it's a wip ;)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant