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

Add more type safety for Interfaces and components that use theme, themes and resolvedTheme #151

Closed
wants to merge 2 commits into from

Conversation

crammag
Copy link

@crammag crammag commented Nov 10, 2022

First iteration of what we mentioned on the issue #149, maybe we can keep working on it from here.

If it looks fine for you I could add some examples on the readme file or at somewhere else.

@vercel
Copy link

vercel bot commented Nov 10, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
next-themes-basic ❌ Failed (Inspect) Jul 14, 2023 4:59pm
next-themes-tailwind ❌ Failed (Inspect) Jul 14, 2023 4:59pm

@crammag
Copy link
Author

crammag commented Nov 10, 2022

Not quite sure why is failing the node version only on one deployment 😢, I suppose is something that should be changed on vercel, under the project settings

@trm217
Copy link
Collaborator

trm217 commented May 9, 2024

Very outdated at this point. We will look for stronger typing in the v1 release

@trm217 trm217 closed this May 9, 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.

3 participants