Skip to content

Commit

Permalink
feat(build): make GUI dev server use port 8000 everywhere
Browse files Browse the repository at this point in the history
  • Loading branch information
ssube committed Feb 18, 2023
1 parent 06681bf commit 763bdcf
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -467,7 +467,7 @@ Still in the `gui/` directory, build the UI bundle and run the dev server with N

### Hosting the client

You should be able to access the web interface at http://127.0.0.1:3000/index.html or your local machine's hostname.
You should be able to access the web interface at http://127.0.0.1:8000/index.html or your local machine's hostname.

- If you get a `Connection Refused` error, make sure you are using the correct address and the dev server is still running.
- If you get a `File not found` error, make sure you have built the UI bundle (`make bundle`) and are using the `/index.html` path
Expand Down
2 changes: 1 addition & 1 deletion docs/user-guide.md
Original file line number Diff line number Diff line change
Expand Up @@ -615,7 +615,7 @@ If you are running the GUI separately, such as when using nginx or for developme

- TCP/80 for the GUI using nginx without a container
- TCP/8000 for the GUI using the nginx container
- TCP/3000 for the GUI dev server
- TCP/8000 for the GUI dev server

#### Error: name 'cmd' is not defined

Expand Down
4 changes: 2 additions & 2 deletions gui/esbuild.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ const config = {
sourcemap: true,
};

if(watch) {
const copyArray = (files) => files.map(file =>
if (watch) {
const copyArray = (files) => files.map(file =>
copy({
resolveFrom: 'cwd',
assets: {
Expand Down
2 changes: 1 addition & 1 deletion gui/serve.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import { join } from 'path';
const { env } = process;

const host = mustDefault(env.ONNX_WEB_DEV_HOST, '127.0.0.1');
const port = mustDefault(env.ONNX_WEB_DEV_PORT, '3000');
const port = mustDefault(env.ONNX_WEB_DEV_PORT, '8000');
const root = process.cwd();

const portNum = parseInt(port, 10);
Expand Down

0 comments on commit 763bdcf

Please sign in to comment.