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

Fix "valueUnpacker is not a worklet" error when Reanimated Babel plugin is missing #5256

Merged
merged 3 commits into from
Oct 25, 2023

Conversation

tomekzaw
Copy link
Member

Summary

This PR fixes a regression in troubleshooting when user forgot to setup Reanimated Babel plugin. Currently, a generic error is shown, instead of a developer-friendly user message with a link to the docs.

Closes #5209.

Before After
Zrzut ekranu 2023-10-17 o 18 20 45 Zrzut ekranu 2023-10-17 o 18 25 23

Test plan

@tomekzaw tomekzaw changed the title Move worklet check from threads.ts to valueUnpacker.ts Fix "valueUnpacker is not a worklet" error when Reanimated Babel plugin is missing Oct 17, 2023
src/reanimated2/valueUnpacker.ts Outdated Show resolved Hide resolved
@tomekzaw tomekzaw added this pull request to the merge queue Oct 25, 2023
Merged via the queue into main with commit 7ce3e3e Oct 25, 2023
7 checks passed
@tomekzaw tomekzaw deleted the @tomekzaw/fix-value-unpacker-error branch October 25, 2023 16:18
bartlomiejbloniarz pushed a commit that referenced this pull request Oct 27, 2023
…ugin is missing (#5256)

<!-- Thanks for submitting a pull request! We appreciate you spending
the time to work on these changes. Please follow the template so that
the reviewers can easily understand what the code changes affect. -->

## Summary

This PR fixes a regression in troubleshooting when user forgot to setup
Reanimated Babel plugin. Currently, a generic error is shown, instead of
a developer-friendly user message with a link to the docs.

Closes #5209.

| Before | After |
|:-:|:-:|
| <img width="507" alt="Zrzut ekranu 2023-10-17 o 18 20 45"
src="https://github.com/software-mansion/react-native-reanimated/assets/20516055/635685a0-91ab-435e-bd56-a3f59460ecb4">
| <img width="507" alt="Zrzut ekranu 2023-10-17 o 18 25 23"
src="https://github.com/software-mansion/react-native-reanimated/assets/20516055/23a2cc6d-595c-419d-914e-3637382a4b5f">
|

## Test plan

<!-- Provide a minimal but complete code snippet that can be used to
test out this change along with instructions how to run it and a
description of the expected behavior. -->
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.

Error: [Reanimated] valueUnpacker is not a worklet, js engine: hermes
3 participants