Skip to content
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

Add next.js information in static-site-generators #1944

Closed
wants to merge 1 commit into from

Conversation

aromeoes
Copy link

Describe your changes

Next.js guide was missing, even though it was specified in the top of the site

What issue(s) does this address?

Lack of next.js upload example

Does this update depend on any other PRs?

no

Checklist before requesting a review

  • Passing the beta version of the Check Markdown links for modified files check. Action results can be viewed here.

Checklist before merging

  • Passing all required checks (The beta Check Markdown links for modified files check is not required)

Copy link

welcome bot commented Oct 24, 2024

Thank you for submitting this PR!
A maintainer will be here shortly to review it.
We are super grateful, but we are also overloaded! Help us by making sure that:

  • The context for this PR is clear, with relevant discussion, decisions and stakeholders linked/mentioned.
  • Your contribution itself is clear (grammar and spelling checked, any code blocks checked and commented) and in its best form. Follow the docs contribution guidelines if they apply.

Getting other community members to do a review would be great help too on complex PRs (you can ask in the chats/forums). If you are unsure about something, just leave us a comment.
Next steps:

  • A maintainer will triage and assign priority to this PR, commenting on any missing things and potentially assigning a reviewer for high priority items.
  • The PR gets reviews, discussed and approvals as needed.
  • The PR is merged by maintainers when it has been approved and comments addressed.

We currently aim to provide initial feedback/triaging within two business days. Please keep an eye on any labelling actions, as these will indicate priorities and status of your contribution.
We are very grateful for your contribution!


## Next.js

Refer to Next.js's [output](https://nextjs.org/docs/app/api-reference/next-config-js/output) guide to install Ruby and Jekyll.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Refer to Next.js's [output](https://nextjs.org/docs/app/api-reference/next-config-js/output) guide to install Ruby and Jekyll.
Refer to Next.js's [output](https://nextjs.org/docs/app/api-reference/next-config-js/output) guide to configure next.js to configure static output.


Use the following commands to compile and then output your application into static html
```bash
npm run build
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use npx next and the correct command and add information about the output folder.

Copy link
Member

@2color 2color left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See coments

@2color 2color changed the title Update static-site-generators.md Add next.js information in static-site-generators Mar 7, 2025
@2color
Copy link
Member

2color commented Mar 7, 2025

Sites generated with next.js also need to set trailingSlash to true in next.config.js to ensure that a folder is created with an index.html instead of path.html in the root folder.

When used with output: "export" configuration, the /about page will output /about/index.html (instead of the default /about.html). https://nextjs.org/docs/app/api-reference/config/next-config-js/trailingSlash

@2color
Copy link
Member

2color commented Apr 1, 2025

Closing as this has been replaced by #1999

@2color 2color closed this Apr 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants