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

feat: perfect migrate to @vue/cli-service, upgrade vue babel version #1232

Closed
wants to merge 7 commits into from
Closed

Conversation

zthxxx
Copy link

@zthxxx zthxxx commented Oct 19, 2018

  1. update to @vue/cli-service@3.0.5, @babel/core@7.0.0
  2. use vue-cli service replace config file in build/ and config/
  3. upgrade vue and babel configuration
  4. solve the svg-sprite config problem 怎么在 vue-cli3.0 中使用 svg-icon 组件? #980

refs: #932 #1087 #980 #1056

1. update to @vue/cli-service@3.0.5, @babel/core@7.0.0
2. use vue-cli service replace config file in build/ and config/
3. upgrade vue and babel configuration
4. solve the svg-sprite config problem #980

refs: #932 #1087  #980 #1056
@zthxxx
Copy link
Author

zthxxx commented Oct 19, 2018

@PanJiaChen
本地构建查看没有问题;
因为没有单元测试,我就手动点开各级页面,确认了对各个地方没有影响。

@zthxxx
Copy link
Author

zthxxx commented Oct 19, 2018

感谢作者大大的工作, vue/cli-service release 出来有一段时间了,很希望这个项目能迁移过去

@PanJiaChen
Copy link
Owner

感谢你的pr,其实之前我升级了一下 vue-cli@3。 但因为一些别的事情耽搁了。我争取最近搞定他吧。

PanJiaChen/vue-admin-template#188

@zthxxx
Copy link
Author

zthxxx commented Oct 21, 2018

4d44f6a 修复了 path-to-regexp 依赖版本不正确导致面包屑导航不能显示的问题

…k server

1. 把 Mockjs 功能移到 server 端中间件,同时也兼容前端直接劫持 XHR
2. dev 环境下默认作为 express 中间件通过 webpack server 提供 mock api
3. prod 构建时,默认在前端用 Mockjs 劫持 XHR

benefits:
  - dev 开发调试时能直接看到 XHR 请求,方便调试网络,能和后端对接联调
  - 避开在开发时因为 Mockjs 引起的网络 bug
  - prod 构建时劫持 XHR,保证本项目的 Github Pages preview 能正常显示 (逻辑和 error-log 一样)
  - 前后台使用的 mock 是同一份代码,不会增加维护负担

ref: [#562](#562 (comment))
@zthxxx
Copy link
Author

zthxxx commented Oct 21, 2018

改动有点多了 😂

第一个 pr 测试不到位,今天修复了面包屑导航、tinymce 静态资源加载问题、driver.js 引入问题;
Mockjs 抽出来了前后端共用,原因见上一个 commit message 330dc7e

vue-admin-template 的 vue-cli@3 我也看了,还不是很完善吧,平时用这个 admin 项目跟多,这次 pr 里的踩坑也希望能帮到作者大大推进 v4.0

作者大大有空 review 一下啦 @PanJiaChen

@PanJiaChen
Copy link
Owner

👍

@PanJiaChen
Copy link
Owner

@zthxxx 你的分支不对,不要基于 deploy分支操作。

你基于 v4.0 分支新开。
之后pr的时候也将代码往4.0分支pr提,不要往master提。

@zthxxx
Copy link
Author

zthxxx commented Oct 24, 2018

OK,我迁到 4.0 分支去

@PanJiaChen
Copy link
Owner

@zthxxx 大概啥时候能提交pr?因为我可能要基于你的代码进行开发。

@zthxxx
Copy link
Author

zthxxx commented Oct 28, 2018

@PanJiaChen 今天!周末没看到😱

@zthxxx
Copy link
Author

zthxxx commented Oct 28, 2018

rebase to branch v4.0
ref: #1267
close #1232

@zthxxx zthxxx closed this Oct 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants