-
Notifications
You must be signed in to change notification settings - Fork 246
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
Update links to docs after successful project creation via CLI #810
Conversation
This comment has been minimized.
This comment has been minimized.
@@ -234,7 +234,9 @@ export async function runInit( | |||
)} to start your local development server and start building`.value, | |||
].filter((step): step is string => Boolean(step)), | |||
reference: [ | |||
'Building with Hydrogen: https://shopify.dev/docs/custom-storefronts/hydrogen/building/begin-development', | |||
'Getting started with Hydrogen: https://shopify.dev/docs/custom-storefronts/hydrogen/building/begin-development', | |||
'Hydrogen project structure: https://shopify.dev/docs/custom-storefronts/hydrogen/project-structure', |
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.
This one doesn't show up in your screenshot; does it get filtered out, or is the screenshot outdated?
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.
Ah, good catch. I've updated the screenshot now!
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.
Ah nice thanks. Looks great! 🚢
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.
Added a changeset too 👍🏻
Oh does this need a changeset? Probably I would think? @gfscott |
🚢 |
WHY are these changes introduced?
Fixes #524
This PR adds a couple more links to documentation when scaffolding out a new project via CLI.
Checklist
[ ] I've added a changeset if this PR contains user-facing or noteworthy changes[ ] I've added tests to cover my changes[ ] I've added or updated the documentation