Skip to content

[DO NOT MERGE] add shadcn nextjs quickstart option #2506

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[DO NOT MERGE] add shadcn nextjs quickstart option #2506

wants to merge 2 commits into from

Conversation

alexcarpenter
Copy link
Member

@alexcarpenter alexcarpenter commented Aug 15, 2025 •
edited by alexisintech
Loading

🔎 Previews:

What does this solve?

  • Introduce manual and shadcn cli nextjs quickstart options via tabs

⚠️ Depends on https://github.com/clerk/clerk/pull/1437

What changed?

Checklist

  • I have clicked on "Files changed" and performed a thorough self-review
  • All existing checks pass

@alexcarpenter alexcarpenter requested a review from a team as a code owner August 15, 2025 20:10
@vercel Vercel
Copy link

vercel bot commented Aug 15, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
clerk-docs Ready Ready Preview Aug 15, 2025 8:14pm

@github-actions GitHub Actions
Copy link
Contributor

⚠️ Changes found in the following quickstarts:

  • nextjs.mdx

⚠️ Please update the corresponding quickstarts in the Dashboard

@github-actions GitHub Actions
Copy link
Contributor

⚠️ Changes found in the following quickstarts:

  • nextjs.mdx

⚠️ Please update the corresponding quickstarts in the Dashboard

@alexisintech
Copy link
Member

@mwickett wanna get your eyes on this to make sure its cool !

@mwickett
Copy link
Member

Once this merges, we'll be good to go here. I just want to make sure we have o11y on shadcn usage before we launch it on the quickstart.

alexisintech
alexisintech approved these changes Aug 27, 2025
View reviewed changes
Copy link
Member

@alexisintech alexisintech left a comment

Choose a reason for hiding this comment

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

cool 😸💖

@alexcarpenter alexcarpenter changed the title add shadcn nextjs quickstart option [DO NOT MERGE] add shadcn nextjs quickstart option Aug 27, 2025
brkalow
brkalow reviewed Aug 27, 2025
View reviewed changes
docs/quickstarts/nextjs.mdx
## Create your first user

<Include src="_partials/nextjs/create-first-user" />

Copy link
Member

Choose a reason for hiding this comment

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

Do we want to include the "It's time to build!" step here like the non-shadcn one?

docs/quickstarts/nextjs.mdx
Comment on lines +411 to +412
- [Install Clerk with shadcn/ui CLI](/docs/references/nextjs/shadcn)
- Learn how to install Clerk with the shadcn/ui CLI.
Copy link
Member

Choose a reason for hiding this comment

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

We can improve this copy. This makes it seem like another quickstart approach. Should next steps be more fleshed out like the other quickstart?

@alexcarpenter
Copy link
Member Author

Note

Note this is also blocked by this open PR from @NWylynko atm https://github.com/clerk/clerk/pull/1437

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brkalow brkalow brkalow left review comments

@alexisintech alexisintech alexisintech approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@alexcarpenter @alexisintech @mwickett @brkalow