-
Notifications
You must be signed in to change notification settings - Fork 8
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
fix: Replaced FaviconWebpackPlugin to WebappWebpackPlugin #22
Conversation
Update webpack-addons to @webpack-cli/webpack-scaffold
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.
Thanks @rishabh3112
It looks great 👍
generator.js
Outdated
@@ -136,8 +136,8 @@ module.exports = class WebpackGenerator extends Generator { | |||
.then(answer => { | |||
if (answer) { | |||
favPath = answer['favPath']; | |||
this.options.env.configuration.dev.topScope.push('const FaviconsWebpackPlugin = require("favicons-webpack-plugin");'); | |||
this.dependencies.push("favicons-webpack-plugin"); | |||
this.options.env.configuration.dev.topScope.push("const WebappWebpackPlugin = require('webapp-webpack-plugin');"); |
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.
Can you please revert the quotes
here ?
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.
Sure, In next commit
@sendilkumarn can you create a basic scaffold to check whether this fix actually solves the problem or not? |
@sendilkumarn I have cross checked it and I think it's ready to merge. |
I am on it 👍 I will merge this today 😊 |
It will certainly cause a conflict with #24 @sendilkumarn. I have integrated #22 into #24. |
Closing this in favor of #24 |
* fix: Replaced FaviconWebpackPlugin to WebappWebpackPlugin * bugfix(favicon): Revert quotes * bugfix(favicon): Removed unnecessory manifest.json file * feat(swReference): Add html-webpack-plugin and Update Tests * feat(seReference): [Feature Working] Add template for index.js * feat(swReference): Add support for custom entry * feat(swReference): Change copy destination of manifest.json to root * bugfix: fixed some minor issues * feat: All fix and merge in this branch Closes #22
Resolves #11
Requires a check @sendilkumarn