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
General fixes #1128
General fixes #1128
Conversation
const isAppLine = (line: string) => line.includes(options.appDirectory); | ||
const isAppLine = (line: string) => | ||
line.includes(options.rootDirectory) && | ||
!line.includes('node_modules'); |
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.
Does it not work with just the node_modules
check?
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.
It... probably does 🤔
Edit: I forgot why I did it but now I remember -- it should work only with the node_modules check in user apps but not in the monorepo where part of the stack comes from packages/hydrogen
😅
You’re seeing this because you don’t have a home route | ||
in your project yet. <br /> | ||
Run <code>h2 generate route home</code> to create your home route. | ||
Learn more about | ||
{` `} | ||
<CreateRoutesLink /> |
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.
So much better
* Remove log related to .shopify in .gitignore * Update list prompt and cleanup * Throw error in HR when storeDomain is falsy * Trim stack trace for every known error, not only GraphQL * Show server.ts in stack trace and ensure something is shown * Fix tests * Update HR error messages * Improve virtual-route information when store is linked * Use mock.shop in hello-world * Changesets
A few minor fixes, check commits 1 by 1 since they are independent.
The main change is starting to use mock.shop in
hello-world
. I think this makes sense because that's the template that new storefronts created from Admin will use. This way, it's more similar to skeleton.