-
Notifications
You must be signed in to change notification settings - Fork 133
docs: Add quick start prompt to the docs #1734
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment bugbot run to trigger another review on this PR
|
Preview for this PR was built for commit |
|
As I understand it is supposed to be a prompt for LLMs (or rather Agent since it needs to execute actions) right? Why not show it to people and offer copy abilitiy? To offer them just a copy button with CTA feels like not the best UX |
|
To be honest, I got the inspiration in the Clerk's doc as I quite liked it: https://clerk.com/docs/quickstarts/nextjs. If I understand your suggestion, you would show the whole prompt (for example with "accordion" - https://ui.shadcn.com/docs/components/accordion, or other UI element), let users read the prompt, and let them decide if they want to copy it, or not? |
|
Yeah exactly :) I definitely see the value, it just feels wrong to give the user ability to copy something without previously see it/read it. |
|
Yeah, I think that is a good point. I will redesign the component. |
1649f30 to
be1c2cf
Compare
8cf0cd8 to
4344bce
Compare
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
You can check it @TC-MO. Do you know, if it is possible to add analytics (i.e. to measure click events on the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks nice ,and thank you for adding the ability to see the prompt ;) 🚀
sources/platform/actors/development/quick_start/start_locally.md
Outdated
Show resolved
Hide resolved
|
As for the analytics, I think this is a request for @jbartadev |
|
Preview for this PR was built for commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Thanks for the approve, @TC-MO. The next steps:
Therefore, I will wait with merge. :) Thanks! |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
@TC-MO just FYI, I added the analytics, and tested the prompt with several models. I will also post this for feedback in |
|
Closing, will merge it as part of #1756 |
The PR adds a button to copy a prompt for quick start.
Important: