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

644 bring shadcn to develop #645

Merged
merged 5 commits into from
Jul 8, 2024
Merged

Conversation

camilovegag
Copy link
Collaborator

@camilovegag camilovegag commented Jul 4, 2024

Closes: #644

Adds tailwind and capability to use shadcn components in develop branch.

This is based on edge esmeralda branch files, added react types to berlin package because shadcn components were warning about:

The inferred type cannot be named without a reference to '.pnpm/@[email protected]/node_modules/@types/react'. This is likely not portable. A type annotation is necessary.ts(2742)

Copy link

vercel bot commented Jul 4, 2024

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

Name Status Preview Comments Updated (UTC)
forum-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 4, 2024 11:41pm

@camilovegag camilovegag requested a review from diegoalzate July 4, 2024 04:30
@camilovegag camilovegag self-assigned this Jul 4, 2024
@camilovegag camilovegag added improvement Improvement of an existing feature multi-event-support labels Jul 4, 2024
@camilovegag camilovegag linked an issue Jul 4, 2024 that may be closed by this pull request
@diegoalzate
Copy link
Contributor

lets redo this because i think edge-esmeralda might be wrong in some ways as all the dependencies seem to be needed in production but according to these docs they can be dev dependencies: https://ui.shadcn.com/docs/installation/vite

@camilovegag camilovegag merged commit 663a919 into develop Jul 8, 2024
4 checks passed
@camilovegag camilovegag deleted the 644-bring-shadcn-to-develop branch July 8, 2024 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement of an existing feature multi-event-support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bring shadcn to develop
2 participants