-
Notifications
You must be signed in to change notification settings - Fork 0
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
Switch deployed frontend to Next js #30
Conversation
ce25bdf
to
bc37ea0
Compare
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.
cleaned up changes, ready for review 🔥 🧯
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.
lgtm 🚀
24ccdd5
to
ce4e27a
Compare
ce4e27a
to
0c715cb
Compare
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.
Bare noen småting jeg er nysgjerrig på, men alt ser bra ut! GG<3
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.
lgtm🚀
Includes all config and setup for deploying the next-frontend to Heroku through Terraform
Previously we built the frontend with github actions, and heroku served the resulting static assets with nginx. Now the frontend is built by Heroku, and the Nextjs server is run by the NodeJS plugin in heroku.