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

Updated webpack to v2 #42

Merged
merged 8 commits into from
Jun 17, 2017
Merged

Updated webpack to v2 #42

merged 8 commits into from
Jun 17, 2017

Conversation

ig0rsky
Copy link
Collaborator

@ig0rsky ig0rsky commented Jun 15, 2017

Now using babel-preset-env
Added loaders for images and fonts
We can now use the latest docs for webpack ~ 2.6.1

Now using babel-preset-env
Added loaders for images and fonts
We can now use the latest docs for webpack ~ 2.6.0
Nothing too important
@@ -40,9 +40,9 @@
},
"scripts": {
"test": "echo \"Error: no test specified\" && exit 1",
"build": "webpack --config webpack.config.js",
"watch": "webpack --config webpack.config.js --watch",
"build": "webpack",
Copy link
Owner

Choose a reason for hiding this comment

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

Why did you change this? Is webpack.config.js a config name by default?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, it is. I also added a webpack-dev-server plugin instead of the http-server which works much better with hot-reloading the page. ( I will commit that very soon)

Copy link
Owner

Choose a reason for hiding this comment

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

But do not forget to leave http-server and npm run start for "production" ;)

}),

new HtmlWebpackPlugin({
template: path.resolve(__dirname,'src/index.html')
title: 'Testing',
template: path.resolve(__dirname, 'src/index.html')
}),

new ExtractTextPlugin("styles/[name].css"),
Copy link
Owner

Choose a reason for hiding this comment

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

This is bugged also. Change to new ExtractTextPlugin("[name].css") please. Otherwise, it creates bad img links. webpack-contrib/extract-text-webpack-plugin#27 (comment)
Maybe there is a "wiser" solution but only this helped me.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yeah, will do now.

},
"author": "Adrian Hernandez-Mendez <adrian.hernandez@tum.de>",
"author": "SEBA-TEAM-24",
Copy link
Owner

Choose a reason for hiding this comment

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

I think it would be more correct to say SEBA Team #25 & Adrian Hernandez-Mendez <adrian.hernandez@tum.de> :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Okay, changing... but we're team 24 :D

Copy link
Owner

Choose a reason for hiding this comment

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

yep, 24, my fault xD

wingsofovnia and others added 4 commits June 17, 2017 16:30
* Fix webpack image processing

* Update toolbar style and add landing header

* Add overview and pricing landing sections

* Unstick footer and add icon credits
Now using babel-preset-env
Added loaders for images and fonts
We can now use the latest docs for webpack ~ 2.6.0
"prestart": "npm install",
"start": "http-server -a localhost -p 8000 -c-1 ./dist"
"start": "http-server -a localhost -p 8000 -c-1 ./dist",
"dev-server": "webpack-dev-server --progress --open"
Copy link
Owner

Choose a reason for hiding this comment

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

imho start-dev will be more cohere with previous "prod" one (start).

Copy link
Owner

@wingsofovnia wingsofovnia left a comment

Choose a reason for hiding this comment

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

Please update README.md with info how to start dev server

Added a default port for the node server: 8080
Updated the README.md
@ig0rsky ig0rsky merged commit 9d50846 into master Jun 17, 2017
@ig0rsky ig0rsky deleted the feature/webpack-update-to-v2 branch June 17, 2017 21:08
ig0rsky added a commit that referenced this pull request Jun 18, 2017
* Updated webpack to v2

Now using babel-preset-env
Added loaders for images and fonts
We can now use the latest docs for webpack ~ 2.6.0

* Minor changes to package.json

Nothing too important

* Updated webpack to v2

Now using babel-preset-env
Added loaders for images and fonts
We can now use the latest docs for webpack ~ 2.6.0

* Implement landing page (#43)

* Fix webpack image processing

* Update toolbar style and add landing header

* Add overview and pricing landing sections

* Unstick footer and add icon credits

* Updated package.json and webpack.config.js

* Updated backend package.json

Added a default port for the node server: 8080
Updated the README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants