Skip to content

Fix bug in dev-server when a proxyTable entry is a string #965

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 11, 2017
Merged

Fix bug in dev-server when a proxyTable entry is a string #965

merged 1 commit into from
Oct 11, 2017

Conversation

schettino72
Copy link
Contributor

options can not be a const because it is modified if value is a string.
@schettino72
Fix bug in dev-server when a proxyTable entry is a string
51f5418 
`options` can not be a `const` because it is modified if value is a string.
@LinusBorg
Copy link
Contributor

Sorry, already fixed this on my own after seeing an issue about it :/
@LinusBorg LinusBorg closed this Oct 11, 2017
@LinusBorg
Copy link
Contributor

Also, it can't be a let either because this code is injected into the client and not transpiled, so we have to use ES5 (var)
@schettino72
Copy link
Contributor Author

dev-server.js file is injected into client? surprising...
@schettino72
Copy link
Contributor Author

I checked the code and saw no change regarding this. I saw a commit on dev-client.js... wondering if you got confused somehow.
@LinusBorg
Copy link
Contributor

Oh, right, I indeed got confused. sorry.
@LinusBorg LinusBorg reopened this Oct 11, 2017
@LinusBorg LinusBorg merged commit 52a069c into vuejs-templates:develop Oct 11, 2017
@LinusBorg
Copy link
Contributor

Thanks!
frandiox pushed a commit to OnsenUI/vue-cordova-webpack that referenced this pull request Oct 13, 2017
@schettino72 @frandiox
Fix bug in dev-server when a proxyTable entry is a string (vuejs-temp…
faca7e5 
…lates#965) `options` can not be a `const` because it is modified if value is a string.
@anubhav7495 anubhav7495 mentioned this pull request Oct 15, 2017
Merged
LinusBorg added a commit that referenced this pull request Oct 16, 2017
@LinusBorg
Bumping Vue+VueRouter versions, some minor fixes. (#986)
a355ee8 
* switch hello-world tag to PascalCase (#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (#984) - vue 2.5 - vue-router 3.0
LinusBorg added a commit that referenced this pull request Oct 16, 2017
@LinusBorg
Fix missing dependency bumps (#987)
6043118 
* switch hello-world tag to PascalCase (#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (#984) - vue 2.5 - vue-router 3.0 * fix missing dependency updates for * vue-template compiler * vue-loader * bump version string
LinusBorg pushed a commit that referenced this pull request Nov 12, 2017
@eddyerburgh @LinusBorg
Add Jest as new unit test runner, keep Karma + Mocha as a fallback (#824
cdb3710 
) * refactor: replace karma and mocha with jest * refactor: remove webpack.test.conf * refactor: prompt jest instead of mocha and karma * feat(jest): add coverage reporting * docs(jest): update unit.md with details of jest * docs(jest): remove karma files from structure.md * docs(jest): replace details on Karma with details on Jest * docs: add missing period * docs(jest): replace karma with Jest in commands.md * docs(jest): update npm run unit bullet points * refactor: use module-resolver in place of jest moduleNameMapper * refactor: remove .vue extension from Hello import * test: include src files in test coverage * test: only ignore router/index in coverage if router option selected * chore: fix merge conflicts * chore: add MIT free Jest * Reference dev script in start script instead of copy pasting (#894) Less duplication FTW. * feat: add karma option * refactor: use const in webpack.test.conf * add version tag to config/index.js * stick to ES5 this file is not transpiled * Bumping Vue+VueRouter versions, some minor fixes. (#986) * switch hello-world tag to PascalCase (#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (#984) - vue 2.5 - vue-router 3.0 * Fix missing dependency bumps (#987) * switch hello-world tag to PascalCase (#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (#984) - vue 2.5 - vue-router 3.0 * fix missing dependency updates for * vue-template compiler * vue-loader * bump version string * refactor: use jest module mapping * feat(jest): filter jest setup file * fix: fix trailing comma * docs: add Jest and Karma to unit section * fix: add Jest options if jest option * test: change env in .eslintrc * fix: remove merge trace * fix: merge package.json with develop * docs: split unit test docs into Jest and Karma * docs: add Karma and Jest to README * docs: reimplement karma files in structure.md
LinusBorg added a commit that referenced this pull request Nov 12, 2017
@LinusBorg
re-merge (#1045)
fa807df 
* Reference dev script in start script instead of copy pasting (#894) Less duplication FTW. * add version tag to config/index.js * stick to ES5 this file is not transpiled * Bumping Vue+VueRouter versions, some minor fixes. (#986) * switch hello-world tag to PascalCase (#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (#984) - vue 2.5 - vue-router 3.0 * Fix missing dependency bumps (#987) * switch hello-world tag to PascalCase (#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (#984) - vue 2.5 - vue-router 3.0 * fix missing dependency updates for * vue-template compiler * vue-loader * bump version string * Fix Unknown plugin istanbul error when no unit test (#785) * Fix Unknown plugin istanbul error when no unit test * simplify handlebars conditionals. * Revert "Fix Unknown plugin istanbul error when no unit test (#785)" (#1035) This reverts commit 14f2a92.
LinusBorg added a commit that referenced this pull request Nov 13, 2017
@LinusBorg
1.2.1 - Bugfix release (#1052)
ad2de3b 
* re-merge (#1045) * Reference dev script in start script instead of copy pasting (#894) Less duplication FTW. * add version tag to config/index.js * stick to ES5 this file is not transpiled * Bumping Vue+VueRouter versions, some minor fixes. (#986) * switch hello-world tag to PascalCase (#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (#984) - vue 2.5 - vue-router 3.0 * Fix missing dependency bumps (#987) * switch hello-world tag to PascalCase (#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (#984) - vue 2.5 - vue-router 3.0 * fix missing dependency updates for * vue-template compiler * vue-loader * bump version string * Fix Unknown plugin istanbul error when no unit test (#785) * Fix Unknown plugin istanbul error when no unit test * simplify handlebars conditionals. * Revert "Fix Unknown plugin istanbul error when no unit test (#785)" (#1035) This reverts commit 14f2a92. * Remove unnecessary whitespaces (#1047) * fix(jest): fix Jest coverage directory (#1046) Explicitly set the coverage directory to match Karma and .gitignore * Fix typos and formatting (#1048) * re-add postcss-loader was mistakenly removed during merge of #824 * bumb version string in config * bumb version string in config in package.json
shenron pushed a commit to shenron/webpack that referenced this pull request Mar 20, 2018
@eddyerburgh @LinusBorg
Add Jest as new unit test runner, keep Karma + Mocha as a fallback (v…
1ce066c 
…uejs-templates#824) * refactor: replace karma and mocha with jest * refactor: remove webpack.test.conf * refactor: prompt jest instead of mocha and karma * feat(jest): add coverage reporting * docs(jest): update unit.md with details of jest * docs(jest): remove karma files from structure.md * docs(jest): replace details on Karma with details on Jest * docs: add missing period * docs(jest): replace karma with Jest in commands.md * docs(jest): update npm run unit bullet points * refactor: use module-resolver in place of jest moduleNameMapper * refactor: remove .vue extension from Hello import * test: include src files in test coverage * test: only ignore router/index in coverage if router option selected * chore: fix merge conflicts * chore: add MIT free Jest * Reference dev script in start script instead of copy pasting (vuejs-templates#894) Less duplication FTW. * feat: add karma option * refactor: use const in webpack.test.conf * add version tag to config/index.js * stick to ES5 this file is not transpiled * Bumping Vue+VueRouter versions, some minor fixes. (vuejs-templates#986) * switch hello-world tag to PascalCase (vuejs-templates#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (vuejs-templates#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (vuejs-templates#984) - vue 2.5 - vue-router 3.0 * Fix missing dependency bumps (vuejs-templates#987) * switch hello-world tag to PascalCase (vuejs-templates#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (vuejs-templates#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (vuejs-templates#984) - vue 2.5 - vue-router 3.0 * fix missing dependency updates for * vue-template compiler * vue-loader * bump version string * refactor: use jest module mapping * feat(jest): filter jest setup file * fix: fix trailing comma * docs: add Jest and Karma to unit section * fix: add Jest options if jest option * test: change env in .eslintrc * fix: remove merge trace * fix: merge package.json with develop * docs: split unit test docs into Jest and Karma * docs: add Karma and Jest to README * docs: reimplement karma files in structure.md
shenron pushed a commit to shenron/webpack that referenced this pull request Mar 20, 2018
@LinusBorg
re-merge (vuejs-templates#1045)
4de8eb7 
* Reference dev script in start script instead of copy pasting (vuejs-templates#894) Less duplication FTW. * add version tag to config/index.js * stick to ES5 this file is not transpiled * Bumping Vue+VueRouter versions, some minor fixes. (vuejs-templates#986) * switch hello-world tag to PascalCase (vuejs-templates#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (vuejs-templates#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (vuejs-templates#984) - vue 2.5 - vue-router 3.0 * Fix missing dependency bumps (vuejs-templates#987) * switch hello-world tag to PascalCase (vuejs-templates#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (vuejs-templates#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (vuejs-templates#984) - vue 2.5 - vue-router 3.0 * fix missing dependency updates for * vue-template compiler * vue-loader * bump version string * Fix Unknown plugin istanbul error when no unit test (vuejs-templates#785) * Fix Unknown plugin istanbul error when no unit test * simplify handlebars conditionals. * Revert "Fix Unknown plugin istanbul error when no unit test (vuejs-templates#785)" (vuejs-templates#1035) This reverts commit 14f2a92.
shenron pushed a commit to shenron/webpack that referenced this pull request Mar 20, 2018
@LinusBorg
1.2.1 - Bugfix release (vuejs-templates#1052)
6700bbc 
* re-merge (vuejs-templates#1045) * Reference dev script in start script instead of copy pasting (vuejs-templates#894) Less duplication FTW. * add version tag to config/index.js * stick to ES5 this file is not transpiled * Bumping Vue+VueRouter versions, some minor fixes. (vuejs-templates#986) * switch hello-world tag to PascalCase (vuejs-templates#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (vuejs-templates#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (vuejs-templates#984) - vue 2.5 - vue-router 3.0 * Fix missing dependency bumps (vuejs-templates#987) * switch hello-world tag to PascalCase (vuejs-templates#951) * fix Es6 code issue. This file is not transpiled, so we should stick to ES5 * Fix bug in dev-server when a proxyTable entry is a string (vuejs-templates#965) `options` can not be a `const` because it is modified if value is a string. * Fix casing * Revert 9befbfc * update to vue and vue-router latest version (vuejs-templates#984) - vue 2.5 - vue-router 3.0 * fix missing dependency updates for * vue-template compiler * vue-loader * bump version string * Fix Unknown plugin istanbul error when no unit test (vuejs-templates#785) * Fix Unknown plugin istanbul error when no unit test * simplify handlebars conditionals. * Revert "Fix Unknown plugin istanbul error when no unit test (vuejs-templates#785)" (vuejs-templates#1035) This reverts commit 14f2a92. * Remove unnecessary whitespaces (vuejs-templates#1047) * fix(jest): fix Jest coverage directory (vuejs-templates#1046) Explicitly set the coverage directory to match Karma and .gitignore * Fix typos and formatting (vuejs-templates#1048) * re-add postcss-loader was mistakenly removed during merge of vuejs-templates#824 * bumb version string in config * bumb version string in config in package.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@schettino72 @LinusBorg @aladdin-add