Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix example projects #776

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

cdn34dd
Copy link

@cdn34dd cdn34dd commented Jan 16, 2025

What does this PR do?

Fix both example and example-new-architecture and update projects to latest supported react-native version.

- Upgrade react-native to version 0.74.6
- Update project dependencies
- Fix metro & babel configs
- Upgrade gradle version
- Increase minSdkVersion & ndkVersion
- Update 'WixApp' component to access 'Navigation' object from
'@datadog/mobile-react-native-navigation'
…rade

- Upgrade react-native to version 0.74.6
- Update project dependencies
- Fix metro config
- Upgrade gradle version
- Increase minSdkVersion & ndkVersion
- Remove flipper initialization & refactor 'MainApplication.kt'
- Rename 'getBundleURL' to 'bundleURL' in 'AppDelegate.mm'
@cdn34dd cdn34dd requested a review from a team as a code owner January 16, 2025 10:33
example/metro.config.js Show resolved Hide resolved
example/metro.config.js Show resolved Hide resolved
Comment on lines +15 to +20
const viewPredicate: ViewNamePredicate = function customViewNamePredicate(
_event: ComponentDidAppearEvent,
trackedName: string
) {
return 'Custom RNN ' + trackedName;
};

Choose a reason for hiding this comment

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

Code Quality Violation

Function name 'customViewNamePredicate' should match name 'viewPredicate'. (...read more)

Use the same name between your function declaration and the variable or property to which it is assigned.

View in Datadog  Leave us feedback  Documentation

@cdn34dd cdn34dd marked this pull request as draft January 16, 2025 10:33
@datadog-datadog-prod-us1
Copy link

Datadog Report

Branch report: carlosnogueira/RUM-8045/fix-example-projects
Commit report: b1f9c39
Test service: dd-sdk-reactnative

✅ 0 Failed, 630 Passed, 1 Skipped, 12.04s Total Time

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.

1 participant