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(www): next-themes version fix (#6207) #6419

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ejzeronimo
Copy link

Fixes site theme flashing on high utilization machines with multiple tabs open.

Issue #6207

Copy link

vercel bot commented Jan 22, 2025

@ejzeronimo is attempting to deploy a commit to the shadcn-pro Team on Vercel.

A member of the Team first needs to authorize it.

@ejzeronimo
Copy link
Author

ejzeronimo commented Feb 10, 2025

Hi @shadcn any requirements I am missing for this to get merged? It's a very small change to fix an edge case, but it would still be nice to know that it has been resolved.

@shadcn
Copy link
Collaborator

shadcn commented Feb 10, 2025

@ejzeronimo Looks good. Can you run pnpm install again and push the updated lock please? I'll merge. Thank you.

@ejzeronimo
Copy link
Author

I already pushed the updated lockfile aswell :)

@shadcn
Copy link
Collaborator

shadcn commented Feb 10, 2025

@ejzeronimo Any chance you can resolve the conflict? If not, I can take a look later.

@ejzeronimo
Copy link
Author

@ejzeronimo Any chance you can resolve the conflict? If not, I can take a look later.

I don't have write access (I am assuming to the source repo) so it won't let me resolve.

@ejzeronimo
Copy link
Author

Any success resolving the conflict @shadcn ?

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.

2 participants