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

Fixes #2 #28

Open
wants to merge 7 commits into
base: github_issues
Choose a base branch
from
Open

Fixes #2 #28

wants to merge 7 commits into from

Conversation

samly97
Copy link

@samly97 samly97 commented Sep 1, 2020

"npm run build" works out of box

Fix "npm run start"

  • updated webpack to 4.44.1
  • updated "react-dev-utils" to 6.0.0

Above breaks "npm run build" ...

Following changes are to get both commands to work

  • the other dependency updates were to get "npm run build" working again
  • replace unused UglifyJs webpack plugin with TerserPlugin
  • modified InterpolationHtmlPlugin to reflect how it works with webpack 4.44.1
  • replaced "contentHash" with "hash" as contentHash clashes with webpack definition

DPTeamMember and others added 7 commits April 23, 2020 13:34
…pack-dev-server-3.10.3

Bump webpack-dev-server from 3.0.0 to 3.10.3
- point app build to backend assets folder

config/webpack.config.[].js
- remove static from assets so img, css, js, etc are the children directory
  of the backend assets folder: i.e. assets/react/[child]

package.json
- have assets point to the correct URL --> assets/react
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.

None yet

3 participants