-
Notifications
You must be signed in to change notification settings - Fork 87
Update docs for existing Next.js websites on Netlify to include basic build settings #271
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
Comments
@DaveBitter would love for you to open a PR with what you'd like to see!!! |
That'd be great!
I also had to specify |
works for thousands of sites! if you wanna send over a repo demonstrating that it doesnt work for you, id be happy to dig into it! |
@DaveBitter hey dave are you wanting to open a PR for this? otherwise i can take care of it! |
@lindsaylevine yes it seems that the problem comes from our own website settings. I'll dig into it :) |
I'm so sorry for responding this late! I see that you already merged a change to the docs. Perfect 👌 |
Is your feature request related to a problem? Please describe.
I had an existing site on Netlify that didn't use this awesome plugin yet where I just used Next.js static site generation (SSG). For that, you usually configure two deploy settings in Netlify:
npm run build && npm run export
out
What I didn't realize is that I needed to change the build command to just
npm run build
and leave the publish directory blank. It might be an obvious thing for people, but I couldn't find this documented anywhere.Describe the solution you'd like
I think it would be great to document these two deploy settings in the README under
Installation and Configuration -> UI-based installation for existing Next.js sites
. I'm more than happy to pick this up.Describe alternatives you've considered
An alternative is to close this issue and have it as a reference. I tried searching the issues for this and couldn't find it.
Additional context
n/A
Thanks for this awesome plugin!
The text was updated successfully, but these errors were encountered: