-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Update React Native to 0.76.2 #2395
base: main
Are you sure you want to change the base?
Conversation
puckey
commented
Nov 7, 2024
•
edited
Loading
edited
- Merges in chore: improve bridgeless interop layer compatibility #2290
- Updates node to 22 (LTS)
- Updates example app to use react-native 0.76.2
Node 22 has entered LTS as of October 2024
example: fix
pls note #2370 breaks on rn <0.76.1 |
I came here to see what would be the progress for this PR to be merged? Would it fully work with RN 0.76.1 is there any help I can provide to try it out? |
I updated the example app to |
@puckey what would be the best way to try it out? Should I clone the library on this PR's branch and then replace my node_modules/react-native-track-player folder with the content of it? |
u should clone and start testing the example app. if ur absolutely sure to test in ur own app, do note newarch breaks/bugs many libs right now, so if it doesnt work u shouldnt comment on this PR but the respective libs. u should first make sure ur project uses RN 0.76.2. then for RNTP instead of a version, use this PR's SHA. for example https://github.com/lovegaoshi/azusa-player-mobile/blob/e3ce4f61806075ce5d6458748b570e9733c7fc2a/package.json#L113. but use this PR's SHA instead |
thank you @lovegaoshi I will try it asap and get back to you here! |
typescript keep yelling if i use this branch. any idea why? but its work tho |
What error are you seeing? |
I figured that out. Lib is not being downloaded in this branch. so I copy lib from latest react native and paste into node_modules/react-native-track-player folder |
I too had issues with typescript following this advice: #2389 (comment) There is no
Hope this helps, love to see this merged and the whole thing sorted out |
hope this help. use this patch
|
Hello @puckey @lovegaoshi Please let me know how i can help in testing and getting this merged asap. |
buddy its oss and mit. fork it, patch it, and use it urself. |