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

react-native example: update to react-native 0.75.3 boilerplate code #292

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

perf2711
Copy link
Contributor

This PR updates react-native example to [email protected].
The code was created using npx @react-native-community/cli@latest init rnbtexample.

@perf2711 perf2711 added the enhancement New feature or request label Sep 26, 2024
@perf2711 perf2711 self-assigned this Sep 26, 2024
Copy link
Collaborator

@konraddysput konraddysput left a comment

Choose a reason for hiding this comment

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

  • wrong application name
  • Did you run the app natively on the phone? I think due to the new namespace and lack of changes in the javascript binding, we're not able to trigger an error
  • We're missing java unhandled exception generator (it was removed)
  • we need to be sure source maps are being generated on the android and ios. the xcode solution task doesn't have source map generation script

examples/sdk/reactNative/ios/rnbtexample/AppDelegate.mm Outdated Show resolved Hide resolved
examples/sdk/reactNative/ios/rnbtexample/Info.plist Outdated Show resolved Hide resolved
@@ -0,0 +1,37 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Collaborator

Choose a reason for hiding this comment

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

examples/sdk/reactNative/package.json Outdated Show resolved Hide resolved
@perf2711 perf2711 force-pushed the feature/react-native-example-update branch from 970c2ec to d1e9d5e Compare September 30, 2024 13:51
@perf2711 perf2711 force-pushed the feature/react-native-example-update branch from d1e9d5e to ec9ade7 Compare September 30, 2024 15:35
Copy link
Collaborator

@konraddysput konraddysput left a comment

Choose a reason for hiding this comment

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

If we want to land this change we need to also fix the problem with crash support on the newest gradle. Please wait on other changes before landing it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants