RUM-12059: RUM Debug Widget #2943
Conversation
…novation-week-07-04-25 # Conflicts: # features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/Rum.kt # features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/RumFeature.kt # features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/domain/scope/RumActionScope.kt # features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/domain/scope/RumApplicationScope.kt # features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/domain/scope/RumResourceScope.kt # features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/domain/scope/RumSessionScope.kt # features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/domain/scope/RumViewManagerScope.kt # features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/domain/scope/RumViewScope.kt # features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/monitor/DatadogRumMonitor.kt
… documentation missing
…et-activity-configuration RUM-12062: Refactor the configuration and integration of the RUM Debug Widget
kikoveiga
changed the title
RUM-12066: Add light theme colors to the RUM Debug Widget
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| .trackLongTasks(250L) | ||
| .trackNonFatalAnrs(true) | ||
| .setSlowFramesConfiguration(SlowFramesConfiguration.DEFAULT) | ||
| .enableRumDebugWidget(this) |
There was a problem hiding this comment.
The sample app invokes enableRumDebugWidget(this) in the main source set, but the widget module is declared as debugImplementation in sample/kotlin/build.gradle.kts (line 155). Release variants compile the main sources without the debug dependency on their classpath, so enableRumDebugWidget will be an unresolved symbol and release builds of the sample (or any app following the README guidance) will fail at compile time. Consider moving the call into a debug-only source set or adding the dependency for release when the call is present.
Useful? React with 👍 / 👎.
joepeeples
left a comment
joepeeples
left a comment
There was a problem hiding this comment.
Suggested a couple of small style edits, thanks!
kikoveiga
force-pushed
the
tvaleev/feature/innovation-week-07-04-25
branch
from
70e70b0 to
e3e4714
Compare
kikoveiga
force-pushed
the
tvaleev/feature/innovation-week-07-04-25
branch
from
e3e4714 to
b792dbf
Compare
Co-authored-by: Joe Peeples <[email protected]>
kikoveiga
force-pushed
the
tvaleev/feature/innovation-week-07-04-25
branch
from
0665d51 to
31690dd
Compare
What does this PR do?
Adds a new feature RUM Debug Widget, which displays RUM data in real time. It lives inside its own module
features/dd-sdk-android-rum-debug-widget.Motivation
It is useful for customers to understand if our SDK is correctly configured inside their apps and if RUM events are being collected.
Review checklist (to be filled by reviewers)