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(vue-app): var names shadowing in chrome debugger #7939
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #7939 +/- ##
=======================================
Coverage 68.92% 68.92%
=======================================
Files 91 91
Lines 3846 3846
Branches 1041 1041
=======================================
Hits 2651 2651
Misses 971 971
Partials 224 224
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
packages/vue-app/template/store.js
Outdated
@@ -94,10 +94,10 @@ function resolveStoreModules (moduleData, filename) { | |||
// If src is a known Vuex property | |||
if (VUEX_PROPERTIES.includes(moduleName)) { | |||
const property = moduleName | |||
const storeModule = getStoreModule(store, namespaces, { isProperty: true }) | |||
const moduleStore = getStoreModule(store, namespaces, { isProperty: 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.
What about propertyStoreModule
?
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.
Why not! It's better actually haha
Gonna change that soon
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.
Corrected :)
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.
LGTM. Thanks @Kapcash 👍
Fixes #7731 :)
Just renamed a variable in a small block so that another variable in the parent block don't shadow eachother in the Chrome debugger.
Types of changes
Checklist: