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

chore(testapp): bump RNTA to latest 3.x #705

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

Conversation

gabrieldonadel
Copy link

Summary

In order to test bridgeless mode compatibility using react-native-test-app you must be use version 3.1.0 or greater, so this PR bumps RNTA to latest 3.x. Changes were based on react-native-menu/menu#712

  • Removed react-native-test-app from root level package.json, given that it's not necessary
  • Run yarn configure-test-app --force; To update and realign the files
  • Verify the changes and remove anything extra that shouldn't be there

Test Plan

 yarn bootstrap
 yarn example start
 yarn example android
 yarn example ios

What's required for testing (prerequisites)?

An iOS device or Simulator and an Android device or Emulator

image

Compatibility

OS Implemented
iOS
Android

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I updated the typed files (TS and Flow)

@vonovak
Copy link
Collaborator

vonovak commented Mar 6, 2024

Hello @gabrieldonadel thanks for the PR!
Let me note that I've done the same kind of changes already as part of #603

In fact, they are already published as part of a full rewrite, albeit not in OSS. You can read more about it here https://react-native-documents.github.io/

I'm not sure yet about the maintenance of this module from my side, maybe I'll leave it up for the community.

I'm planning to record some video about the new package and then document it in this repo in the next few days. So, um, I'll review this PR if you want but down the line... would you be interested in maintaining it?

@gabrieldonadel
Copy link
Author

Hi @vonovak, we're making an effort at Expo along with Meta and the React Native community to make sure that the most popular libraries, including this one, work with the new architecture on 0.74 using bridgeless mode.

From my testing, everything will work out of the box after facebook/react-native#43351 gets included in 0.74 RC3. The only other change that might be necessary is replacing onCatalystInstanceDestroy with invalidate

Regarding the status of this library, I guess we could update the readme file, explaining that this library will no longer be maintained and that new features will be implemented in https://react-native-documents.github.io/?

@vonovak
Copy link
Collaborator

vonovak commented Apr 1, 2024

@gabrieldonadel thank you for your PR! When I was testing it, I noticed one thing (maybe that's my mistake so please let me know) - when I make changes to the TS files inside src, those changes are not picked up by metro.

This is because the example project has react-native-document-picker in its own node_modules as a dependency.

The developer experience I'm aiming for is that when I make changes in src I want to se them right away. Did I forget some part of the setup? (something with @rnx-kit/metro-config might be at play here)

Thank you

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