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

Docs: nuxt examples & Vuejs fix #6

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

StefanVDWeide
Copy link

I've added examples for deploy a Nuxt site both with server side rendering and as a static stite. Also, solved an issue with the VueJS docs where the wrong start command was written down.

@StefanVDWeide StefanVDWeide changed the title Docs: nuxt examples Docs: nuxt examples & Vuejs fix Nov 13, 2024
@mezotv
Copy link

mezotv commented Nov 13, 2024

I dont think setting the port to 80 is required for this. Any port should work since it runs in a container anyways.

@StefanVDWeide
Copy link
Author

I dont think setting the port to 80 is required for this. Any port should work since it runs in a container anyways.

80 is the default port for NGINX which serves the files in this case. Setting it to anything else will result in a bad gateway error. I've tested 3000 and 81 just in case.

@mezotv
Copy link

mezotv commented Nov 13, 2024

I dont think setting the port to 80 is required for this. Any port should work since it runs in a container anyways.

80 is the default port for NGINX which serves the files in this case. Setting it to anything else will result in a bad gateway error. I've tested 3000 and 81 just in case.

Oh so thats only an internal port then?

@StefanVDWeide
Copy link
Author

I dont think setting the port to 80 is required for this. Any port should work since it runs in a container anyways.

80 is the default port for NGINX which serves the files in this case. Setting it to anything else will result in a bad gateway error. I've tested 3000 and 81 just in case.

Oh so thats only an internal port then?

Yeah exactly, since the container exposes port 80 the domain needs to be configured to point there

@Siumauricio
Copy link
Contributor

I tested and both works, awesome

Can you change npm to pnpm? is more fast than npm

nuxt/README.md Outdated Show resolved Hide resolved
@StefanVDWeide
Copy link
Author

I tested and both works, awesome

Can you change npm to pnpm? is more fast than npm

Great! Changed the example to pnpm

@mezotv
Copy link

mezotv commented Nov 15, 2024

Great! Changed the example to pnpm

With pnpm you can drop the run so its just pnpm generate

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.

3 participants