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

Add serve and dev scripts #1637

Merged
merged 1 commit into from
Sep 30, 2022
Merged

Add serve and dev scripts #1637

merged 1 commit into from
Sep 30, 2022

Conversation

lfdebrux
Copy link
Member

@lfdebrux lfdebrux commented Sep 30, 2022

We want to make sure users have NODE_ENV=production set when their prototypes are hosted online, because that controls a lot of important behaviour including the password control and HTTPS redirects. To make this easier, this commit adds a command to the cli tool that sets NODE_ENV and then does the build and run. We also add these commands to the starter files package scripts.


This is part of resolving #1202, although it keeps the behaviour of npm start the same for now.

@lfdebrux lfdebrux changed the base branch from v13 to ldeb-heroku-tests September 30, 2022 11:19
Base automatically changed from ldeb-heroku-tests to v13 September 30, 2022 11:23
@lfdebrux lfdebrux force-pushed the ldeb-add-serve-script branch 2 times, most recently from 1ddcdae to 0957992 Compare September 30, 2022 11:42
We want to make sure users have NODE_ENV=production set when their
prototypes are hosted online, because that controls a lot of important
behaviour including the password control and HTTPS redirects. To make
this easier, this commit adds a command to the cli tool that sets
NODE_ENV and then does the build and run. We also add these commands to
the starter files package scripts.
@lfdebrux lfdebrux merged commit 0d2c09e into v13 Sep 30, 2022
@lfdebrux lfdebrux deleted the ldeb-add-serve-script branch September 30, 2022 16:55
@lfdebrux lfdebrux mentioned this pull request Nov 17, 2022
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