-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
new architecture not working #45
Comments
I've created a PR that enables compatibility with the new architecture - #44 It's just a couple of lines so you can use |
@hrastnik thanks ok let me try |
Are you sure it's a problem with react-native-volume-manager? |
yes 100% sure bro i think there is some issue related to new arch |
I dont think that #44 is enough to fix for new arch. We're also using views, so more changes are involved. |
Ok so i want this feature urgent so i will raise the new arch support patch |
Maybe we can sponsor @WoLewicki and he might be able to help |
@hirbod i also migrated react native sound , voice and splash screen so let me first try my self |
Let me know if you need any help 😄 |
I've just built 2 completely new RN apps. One with React Native 0.75 where the default is the legacy architecture, and one with React Native 0.76 where the default is the new architecture with TurboModules. I've added react-native-volume-manager to both versions and I've applied my patch to the version with React Native 0.76. Here's the video of both apps running and behaving the same. Both apps were build in production mode. I'm not sure if I'm missing something, but my patch seems to work at least for me. react-native-volume-manager.1.mp4 |
Did you install pods with |
Yes, I did! It works without issue. |
Let me test again with fresh project |
What about Android? |
Android works as well! |
@hirbod Here is proof its not working with new arch also i publish this repro example in my git if you have time then check Steps
Package JSON
volumeManager.mp4 |
Please let me know if it's working for you. If not, I would be happy to raise a patch. |
working fine in android |
@SolankiYogesh Did you apply the patch from my PR? Note that my PR is not merged yet so you'll have to manually open node modules and apply the changes. The change is only 3 lines of code so it's simple to apply |
finally its working thanks bruh @hirbod closing ... |
The text was updated successfully, but these errors were encountered: