Add LLM prompts to Next.js and React quickstarts #2261
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Hey, here’s your docs preview: https://clerk.com/docs/pr/2261 |
|
|
alexcarpenter
approved these changes
May 28, 2025
|
Given the removal of the github repos from the top of the pages, can we add the example repo in to the sidebar in the react.js sdk like we have it in the nextjs sidebar items? 
|
|
oh good call, I didn't realize those were there. I'll add React. |
NWylynko
reviewed
May 28, 2025
|
NWylynko
approved these changes
May 28, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔎 Previews:
What does this solve?
We've seen a 4% lift in activation for dev instances where the LLM prompt copy button is clicked, so this adds the same functionality to Docs.
What changed?
Uses the new
LLMPromp(added in https://github.com/clerk/clerk/pull/1202) component to render a banner with a copy button anddisplayTexton the Next.js and React quickstarts.🛹 Talking with @NWylynko we're ok with this rough first pass where the prompt content is included inline as a prop. Goal is to get this shipped ASAP and then once his improvements ship we'll revisit how we handle this.
Note: this removes the from both the Next.js and React quickstarts - looking at PostHog data, the sample repo and the pre-requisites get very low numbers of clicks (6 in the last 90 days for the prereq) so I think it's fine to remove, but just calling that out.
Checklist