-
Notifications
You must be signed in to change notification settings - Fork 14
Fixing bug in deployment and problem with npm start. #4
Conversation
@@ -15,8 +15,9 @@ deploy: | |||
cache-control: "max-age=31536000" | |||
local_dir: dist | |||
skip_cleanup: true | |||
acl: public_read |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is required to make the uploaded js files available in Digital Ocean spaces
package.json
Outdated
@@ -5,7 +5,7 @@ | |||
"main": "dist/root-config-lib.js", | |||
"scripts": { | |||
"clean": "rimraf dist", | |||
"start": "webpack-dev-server --port", | |||
"start": "webpack-dev-server", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The convention established by fatma and me is that to run the project you should do npm start -- --port 8081
@@ -24,7 +24,8 @@ module.exports = { | |||
devServer: { | |||
headers: { | |||
"Access-Control-Allow-Origin": "*" | |||
} | |||
}, | |||
disableHostCheck: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need this because we're loading these apps dynamically via SystemJS from urls instead of on a hard coded url and domain.
https://webpack.js.org/configuration/dev-server/#devserverdisablehostcheck
Fixed Patient Search For Hackathon
No description provided.