-
Notifications
You must be signed in to change notification settings - Fork 95
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
Synchronously call function #10
Comments
Could you please share other details in bug issue template for your example?
|
Screens tsx
app.tsx
|
Are we going to go one by one? Which versions are you using? |
oo sorry :(
|
And the Hold Menu version? Is it v0.0.8? This might because of your Reanimated setup as well. Be sure Reanimated it is working fine. |
yes, v0.0.8, Reanimated I've been using it for a long time, but let me check again. |
I see, I have to reproduce problem to solve and will try. Any help would be great to reproduce it. |
Reanimated no problem, add react-native-hold-menu some problem |
https://github.com/software-mansion/react-native-reanimated/releases/tag/2.0.0 Can it test this with the stable version just released? I wonder if the library does not work steadily |
I have not tested with stable version yet but was working with rc-3 that last version before the stable. No breaking changes after rc-3 (maybe this). Will try it today with 2.0.0. |
@enesozturk Can it run ? I have the same error. I use react-native init to create a project and follow react-native-unimodules installing. Do I need try create-react-native-app to create a project ? |
Hi @booyeu, I am trying to fix another more important issue for now, after that this must be solved with new release. It is important which React Native and Reanimated version you are using. You can check the expo project or clone this repo and run the example project. |
@enesozturk thank you very much ^^ . And I want to know dose it also need expo in the new release ? My app will be bigger when I choice add react-native-unimodules to enable expo function |
You don't have to use Expo. You can use bare RN setup with unimodules. It is needed for blurview and haptic feedback. We may make it optional further releases. |
@enesozturk ok ok , thank you . I really really like this project and appreciate . And want to know maybe when it can use in [email protected] and [email protected] stable version ? |
Not sure. Thanks for let me know the versions. I will check it 👍🏽 |
I fix the problem by set _leftOrRight from object to value, and edit "..._leftOrRight" to "left:_leftOrRight" |
There must be something missing. _leftOrRight is returning an |
and it seems your doc write a wrong props . holditems your doc shows styles , but it is containerStyles in codes~~ |
Oh I see, will fix it 👍🏽 |
That is weird. No idea why. Maybe it is related the same issue. Text is passing through Shared Value. That might breake something. |
I am experiencing this exact same error, any fix for it yet? |
As @booyeu pointed out the solution, that works because object spread is not supported in worklets, its a known problem, as mentioned in the reanimated docs here https://docs.swmansion.com/react-native-reanimated/docs/next/#known-problems-and-limitations |
Maybe you guys could release this to npm? We're having to point to the develop branch because the npm version is broken and I don't think it's going to work for anyone. |
For anyone encountering this issue, the simplest thing I found is to fork the repo, build the develop branch, remove lib from the .gitignore and commit it, then point to your fork in the package.json |
Having the exact same issue here too. |
Not working for me |
Originally posted by @productfrontenddeveloper in #9 (comment)
The text was updated successfully, but these errors were encountered: