Skip to content

[RUM-9801] Fix main thread initialization warning for RCTDatadogWebViewManager#866

Merged
cdn34dd merged 1 commit into
developfrom
carlosnogueira/RUM-9801/webview-main-thread-init
May 6, 2025
Merged

[RUM-9801] Fix main thread initialization warning for RCTDatadogWebViewManager#866
cdn34dd merged 1 commit into
developfrom
carlosnogueira/RUM-9801/webview-main-thread-init

Conversation

@cdn34dd
Copy link
Copy Markdown
Contributor

@cdn34dd cdn34dd commented May 5, 2025

What does this PR do?

FIxes the following warning message some people have been seeing:

WARN Module RCTDatadogWebViewManager requires main queue setup since it overrides init but doesn't implement requiresMainQueueSetup. In a future release React Native will default to initializing all native modules on a background thread unless explicitly opted-out of.

@cdn34dd cdn34dd requested a review from a team as a code owner May 5, 2025 10:11
@datadog-datadog-prod-us1
Copy link
Copy Markdown

Datadog Report

Branch report: carlosnogueira/RUM-9801/webview-main-thread-init
Commit report: e287f7c
Test service: dd-sdk-reactnative

✅ 0 Failed, 669 Passed, 1 Skipped, 4.07s Total Time

Copy link
Copy Markdown
Contributor

@sbarrio sbarrio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat 👍

@cdn34dd cdn34dd merged commit e287f7c into develop May 6, 2025
8 checks passed
@cdn34dd cdn34dd deleted the carlosnogueira/RUM-9801/webview-main-thread-init branch May 6, 2025 15:11
@sbarrio sbarrio mentioned this pull request May 14, 2025
4 tasks
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