Skip to content

Conversation

@sbarrio
Copy link
Contributor

@sbarrio sbarrio commented Dec 18, 2025

What does this PR do?

This PR modifies the configuration object types so the following properties are now moved into the rumConfiguration object:

resourceTraceSampleRate 
nativeLongTaskThreshold
nativeCrashReportingEnabled
useAccesibilityLabel

It also exposes optional configuration objects for Rum, Logs and Trace on DatadogProviderConfiguration, allowing for easier configuration without the need to set and create separate configuration objects and having to set each property one by one.

Motivation

We need to make sure that domain related properties are properly placed under the right configuration for their target module. We also need to make configuration as easy as possible while retaining flexibility.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests
  • 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)
  • If this PR is auto-generated, please make sure also to manually update the code related to the change

@sbarrio sbarrio self-assigned this Dec 18, 2025
@sbarrio sbarrio marked this pull request as ready for review December 19, 2025 01:11
@sbarrio sbarrio requested a review from a team as a code owner December 19, 2025 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants