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

Docker container version #12

Closed
dojoca opened this issue Dec 17, 2023 · 3 comments · Fixed by #39
Closed

Docker container version #12

dojoca opened this issue Dec 17, 2023 · 3 comments · Fixed by #39

Comments

@dojoca
Copy link

dojoca commented Dec 17, 2023

Hello,

I apologise if this is the wrong place to post this, but I saw this announced on Reddit and I'm very excited to use it. Thank you so much for creating it!

I was wondering whether it would be a lot to ask for a docker container version? It might be easier for a lot of people rather than trying to set up a database and app separately.

Thank you so much!

@ridv
Copy link

ridv commented Dec 19, 2023

I wrote a first pass of this:

I'm not too familiar with how NodeJS apps are packaged so there's some opportunities to package this app better.

@scastiel, some questions:

Is there a way to run the postinstall script without having the database running?

Is it possible to run npm run postinstall without needing to do npm run build again afterwards?

Also wanted to say thanks for writing this app!

@LIRIKKER
Copy link

LIRIKKER commented Jan 3, 2024

+1 for the docker image

@ChristopherJohnston
Copy link
Contributor

see: https://github.com/vercel/next.js/tree/canary/examples/with-docker

  1. Copy Dockerfile to root of spliit2 repository - may also need to add lines to this to copy over prisma.schema
  2. add the following to ./next.config.js:
const nextConfig = {
    output: 'standalone'
}
  1. npm run build
  2. docker build -t spliit2-docker .
  3. docker run -p 3000:3000 spliit2-docker

@scastiel scastiel linked a pull request Jan 11, 2024 that will close this issue
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 a pull request may close this issue.

4 participants