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

Setup express server #2

Merged
merged 10 commits into from
Nov 3, 2021
Merged

Setup express server #2

merged 10 commits into from
Nov 3, 2021

Conversation

JordT
Copy link
Collaborator

@JordT JordT commented Nov 2, 2021

I've added Chalk and Debug packages, and started using concurrently and nodemon packages.

I've also added a simple Express server, and improved the express servers logging using chalk/debug.

You can test this by doing npm start, then hitting the express server on port 4000.

The time and date are included in the logging, this is kind of annoying but looks like the developers haven't got a fix for this yet. The docs suggest using .env and REACT_APP_DEBUG variable that you'll see in the .env variable.

However this looks to be broken with two fixes being worked on as per here: debug-js/debug#765

Let me know what you think -

@JordT
Copy link
Collaborator Author

JordT commented Nov 2, 2021

package-lock.json throwing conflicts because I installed nodenv as --dev...

@mr-jackpot mr-jackpot self-requested a review November 3, 2021 16:54
Copy link
Owner

@mr-jackpot mr-jackpot left a comment

Choose a reason for hiding this comment

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

Awesome nice work, all looking good to me

@JordT JordT merged commit 55800dd into main Nov 3, 2021
@JordT JordT deleted the setup-express-server branch November 3, 2021 17:03
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