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

Support serialization of exotic react compoonent #1490

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

exah
Copy link

@exah exah commented Oct 12, 2024

Right when we attempt passing forwardRef component to addFunctionSerializer (following official guide) we get error:

Invalid exports.

You can only export plain objects, arrays, strings, numbers and null/undefined.

As this type of components internally objects are with render method.

This PR adds support for creating generic styled react components with custom ref using addFunctionSerializer.

Right when we attempt passing `forwardRef` component to `addFunctionSerializer` we get error:

```
Invalid exports.

You can only export plain objects, arrays, strings, numbers and null/undefined.
```

This PR adds support for creating generic `styled` components using `addFunctionSerializer`.
Copy link

changeset-bot bot commented Oct 12, 2024

🦋 Changeset detected

Latest commit: 0a7df64

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
@vanilla-extract/integration Major
@vanilla-extract/esbuild-plugin Patch
@vanilla-extract/jest-transform Patch
@vanilla-extract/parcel-transformer Patch
@vanilla-extract/rollup-plugin Patch
@vanilla-extract/vite-plugin Patch
@vanilla-extract/webpack-plugin Patch
@vanilla-extract/next-plugin Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@exah exah changed the title Fix serialization of exotic react compoonent Support serialization of exotic react compoonent Oct 20, 2024
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.

1 participant