Skip to content

Conversation

@Amaru333
Copy link

  • Fixed hot reload issue.
  • npm run dev now uses hot reload.
  • React app runs in port 8080 (http://localhost:8080) while the nodejs server runs in port 7080.
  • Added a proxy to redirect all the API calls to port 7080 in the dev webpack

Copy link
Member

@Woozl Woozl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a great fix and a huge improvement, thanks for handling this

@Woozl
Copy link
Member

Woozl commented Jun 10, 2025

The only thing I noticed was the localhost:7080 port still responds with the non-reloading version, since the server.js also points to the app:

qgraph/server.js

Lines 32 to 36 in 14db2d9

// Send every request to the React app
// Define any API routes before this runs
app.get('*', (req, res) => {
res.sendFile(path.join(__dirname, 'pack', 'index.html'));
});

Can we put a brief note in the README saying to use localhost:8080? @Amaru333

@Amaru333 Amaru333 merged commit 5866cf3 into staging Jun 10, 2025
2 checks passed
@Amaru333 Amaru333 deleted the bugfix/hot-reload branch June 10, 2025 20:12
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