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 SSR Recipe #12

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JosephSemrai
Copy link

Description

Dunner recipe for running SSR Next.js projects in a standardized Node environment:

Supports running all of the default Next commands and any custom scripts that the user specifies in their package.json.

Standardizes around the Node v10 environment and takes in a common environment variable for the secret for basic user authentication.

Related Issue

#11

Copy link
Contributor

@agentmilindu agentmilindu left a comment

Choose a reason for hiding this comment

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

How about a task to build the project, which will generate static files under a folder build or public or etc?

( Optional ) And then a deploy command that will push the static files to Firebase or something?

recipes/next-js-ssr/metadata.yml Outdated Show resolved Hide resolved
recipes/next-js-ssr/.dunner.yaml Outdated Show resolved Hide resolved
Copy link
Author

@JosephSemrai JosephSemrai left a comment

Choose a reason for hiding this comment

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

I went ahead and addressed the improvements that you had recommended. Additionally, I have attempted a deploy script that allows users to deploy the built application to Heroku for CSR should they choose to do so.

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