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
Fix incomplete UI runtime initialization #5505
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🫡
Sorry for bothering, but is there any chance this will be released soon? |
@Nodonisko I'm afraid we don't have enough capacity to publish a stable release this week but we hope to merge this PR this week and it will be available in |
Thanks, nightly is fine :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couldn't find anything to request changes 😭
Hello guys, how to install 3.7.0-nightly via npm? |
|
Hi @tomekzaw, any plan to release it on stable ? |
@huextrat We don't have exact plans when to release 3.7.0 but maybe in 1-2 weeks? |
Voor issue: #104540 ...willen we de fix doorvoeren voor: software-mansion/react-native-reanimated#5505 (ook software-mansion/react-native-reanimated#5421) ...en die is opgelost in: https://github.com/software-mansion/react-native-reanimated/releases/tag/3.7.0
Summary
Fixes #5421, fixes #5331.
This PR removes
installValueUnpacker
method which was called on the JS thread during the creation ofNativeReanimatedModule
but operated directly on the UI runtime. Because of this, UI runtime was already used from JS thread so any use from the UI runtime (e.g. Reanimated event handlers) would cause a crash due to a failed check inReanimatedReentrancyCheck::before
. Note that the check also occurred in release mode due to improperly setNDEBUG
flag.Test plan