search

vitodeploy / vito

The Ultimate Server Management Tool
https://vitodeploy.com
GNU Affero General Public License v3.0
1.06k stars 166 forks source link

add nextjs site type #190

Closed mehrdadx10 closed 3 months ago

mehrdadx10 commented 5 months ago

add custom site type for nextjs projects

note: only work for static exported projects export folder is out

suggested deploy script

git pull
npm i
npm run build
npm run export
saeedvaziry commented 5 months ago

In general the PR looks good. The only problem I see here is that Vito doesn't install Node.js on the target servers!

To solve this problem I would recommend we make the Nodejs available to be installed as a service in the Services > Supported Services section.

mehrdadx10 commented 5 months ago

In general the PR looks good. The only problem I see here is that Vito doesn't install Node.js on the target servers!

To solve this problem I would recommend we make the Nodejs available to be installed as a service in the Services > Supported Services section.

In general the PR looks good. The only problem I see here is that Vito doesn't install Node.js on the target servers!

To solve this problem I would recommend we make the Nodejs available to be installed as a service in the Services > Supported Services section.

its seems nodejs installed by app/SSH/OS/scripts/upgrade.sh .

saeedvaziry commented 5 months ago

@mehrdadx10 so I am trying to deploy a nextjs site but seems I am lost. there is no export command of course.

when I build the output is in .next folder.

Have you tried the code locally to see if you can deploy a nextjs site?

saeedvaziry commented 5 months ago

OK. I figured it out! Supporting only the export mode in VitoDeploy doesn't really make sense.

I would go only if we're able to support nodejs server side applications.

However I don't want to close this PR. So lets change the direction to the main branch instead of 1.x and work on it for the next major release.

Thanks for the work

saeedvaziry commented 3 months ago

Closing due to inactivity