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: ensure userRequest
stays unique (module.userRequest
)
#58
fix: ensure userRequest
stays unique (module.userRequest
)
#58
Conversation
userRequest
stays unique (module.userRequest
)
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.
👍
index.js
Outdated
@@ -32,6 +32,7 @@ module.exports.pitch = function(remainingRequest) { | |||
); | |||
this.cacheable && this.cacheable(); | |||
if(!this.query) throw new Error("query parameter is missing"); | |||
this._module.userRequest = this._module.userRequest + ' - exposed'; |
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.
Could you please add a comment marking this is as a workaround until module.libIdent()
gets fixed ? :)
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
Please sign the CLA by closing and reopening the PR to trigger the CLA Bot again :) |
i signed it, see => https://cla.js.foundation/webpack-contrib/expose-loader?pullRequest=58 |
🤔 Is your email used by |
85dc4fc
to
273542c
Compare
that was it :P sorry |
userRequest
stays unique (module.userRequest
)userRequest
stays unique (module.userRequest
)
@timse Thx again 👍 |
np :) can we release that? |
Sry I can't cc @d3viant0ne :) |
@timse - Sorry, missed this one. I'll have it out in 20 minutes. |
no worries @d3viant0ne and thnks for the release! :) |
Notable Changes
There is an issue using expose loader in combination with
webpack.NamedModulesPlugin
as the userRequest that is used by webpack to calculate the named modules id stays the same.See images below for explanation and fix:
Issues
webpack.config.js
file not working with NamedModulesPlugin #55