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

Updated and fixed the build for latest dependencies #89

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

scipilot
Copy link

FYI. I found the current project wouldn't install as-is (npm couldn't resolve the dependencies originally specified), and so I spent some time getting it up and running again.

This compiles and runs OK on MacOS 12.2.1, with XCode 13.4, npm v8.5.5, node v16.15.0

  • Updated npm modules to React Native 0.70.0 and matching sub-dependencies.
  • Increased minimum iOS platform to 13 to solve requirements from pods (mainly React.RCTWebSocket)
  • Fixed the signature change to RCTBundleURLProvider, and a podfile path syntax change.
  • Fixed TS strict "error" warnings of unused React imports.

…ies.

Increased minimum iOS platform to 13 to solve requirements from pods (mainly React.RCTWebSocket)

Fixed the signature change to RCTBundleURLProvider, and podfile path syntax change.

Fixed TS strict "error" warnings of unused React imports.
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