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

Solid wrapper #674

Merged
Merged

Conversation

LiamMartens
Copy link
Contributor

@LiamMartens LiamMartens commented Dec 31, 2023

Adds a package for Solid-JS support. Also includes docs for it.

@davidjerleke davidjerleke added feature request New feature or request solid Issue is related to SolidJS labels Dec 31, 2023
@davidjerleke davidjerleke linked an issue Dec 31, 2023 that may be closed by this pull request
@davidjerleke davidjerleke changed the title feat: Add solid support Solid wrapper Dec 31, 2023
@davidjerleke davidjerleke self-assigned this Jan 5, 2024
@davidjerleke
Copy link
Owner

@LiamMartens I have a little left to test on this PR and I want to let you know that this is great! I will make some small adjustments to align some things with the other existing wrappers like React, Vue and Svelte.

Soon Embla will have a SolidJS wrapper published 🎉. I want to put your name on the SolidJS wrapper if that’s ok?

@davidjerleke davidjerleke force-pushed the feature/solid-support branch 3 times, most recently from 9a6e598 to 1d0ba4b Compare January 6, 2024 20:43
Co-authored-by: David Jerleke <david.jerleke@gmail.com>
Co-authored-by: Liam Martens <hi@liammartens.com>
@davidjerleke davidjerleke force-pushed the feature/solid-support branch from 1d0ba4b to 32176d5 Compare January 6, 2024 20:53
@davidjerleke davidjerleke removed their assignment Jan 6, 2024
@davidjerleke davidjerleke self-requested a review January 6, 2024 20:54
Copy link
Owner

@davidjerleke davidjerleke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fantastic ⭐. Thank you @LiamMartens!

@davidjerleke davidjerleke merged commit 2c24621 into davidjerleke:master Jan 6, 2024
@davidjerleke davidjerleke added the resolved This issue is resolved label Jan 6, 2024
@LiamMartens
Copy link
Contributor Author

Awesome thanks for having a look and merging! @davidjerleke

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request resolved This issue is resolved solid Issue is related to SolidJS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for solid-js
2 participants