-
Notifications
You must be signed in to change notification settings - Fork 5
chore: add MCP email migration tutorial #1062
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
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub.
|
(Aside that doesn't need to be addressed in this PR but I had a hard time finding this page initially in the preview) I agree with the sidebar conversation in Slack. I think having "MCP" listed first while we sort this out helps, but I would like to see the whole title ideally, so I do think we should consider a different style for this section in the docs. |
<Step title="Analyze your existing email templates"> | ||
Before creating any resources in Knock, you'll need to analyze your existing email templates to identify patterns and shared components. Copy the prompt below and provide it to your LLM along with access to your template files. | ||
|
||
**Prompt:** |
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.
You may have already tested this, but I'm curious to see what it would look like with the prompt held under an accordion to save some visual space for a quick scan/overview of the steps themselves. Then I can open up the prompt accordion when I'm doing the migration. Wondering what others think on this as well!
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.
I didn't test but I did consider it. Main reason I left it out is because I figure that with this particular guide, most people wouldn't actually be reading the content of the prompts themselves but adding the accordion means an extra click to get to the prompt so that you can copy it. I agree it isn't great that it takes up so much space on the page, but thought it was a worthwhile compromise for ease of use. wdyt?
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.
@rachael-t I tried this out locally, what do you think? https://www.loom.com/share/3675a611f855479bbea55aafcdf6a4b5
Description
This PR introduces a new tutorial to walk a user through using the Knock MCP server to migrate their existing email templates into Knock.
https://docs-epj9hbwmr-knocklabs.vercel.app/tutorials/migrate-email-with-mcp-server
Todos