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

Set port details on npm run start #118

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

Conversation

Bijit-Mondal
Copy link
Contributor

@Bijit-Mondal Bijit-Mondal commented Oct 1, 2024

Summary of change

We can define the ports that the npm run start command will use, default is 3001 for backend and 3000 for frontend.
We can specify a custom port when running your application:

Related Issue-

#108

Checklist for important updates

  • Had run npm run build-pretty
  • Had installed and ran the pre-commit
  • react custom port PORT=8000 REACT_APP_API_PORT=8080 bun run start
  • vue custom port VITE_APP_PORT=8082 VITE_API_PORT=8000 bun run start
  • angular custom port
  • express custom port
  • nest JS custom port
  • flask custom port
  • drf custom port
  • fastapi custom port
  • go custom port
  • next JS custom port NEXT_PUBLIC_PORT=5167 bun run dev -- --port="5167"
  • remix custom port VITE_PORT=5167 bun run dev
  • sveltekit custom port VITE_PORT=9192 bun run dev
  • astro custom port PUBLIC_PORT=5167 bun run dev

@rishabhpoddar
Copy link
Contributor

@Bijit-Mondal this requires changes to all the boilerplate code and the configs in the appInfo on the FE and BE for them. Unfortunately i can't merge this PR unless it's fully completed.

@Bijit-Mondal
Copy link
Contributor Author

@rishabhpoddar okay making changes on all boilerplate code.

@Bijit-Mondal
Copy link
Contributor Author

Bijit-Mondal commented Oct 7, 2024

@rishabhpoddar please review this PR.

@rishabhpoddar
Copy link
Contributor

hey @Bijit-Mondal thanks for this PR. Given the size of it though, im not sure when we will have the time to review it.

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