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

你好, 我 fork 了这个项目, 然后移了更多的 vue-element-admin 页面过来,不知能不能合到你的这个项目中 #11

Closed
mei-rune opened this issue Feb 26, 2019 · 10 comments

Comments

@mei-rune
Copy link

mei-rune commented Feb 26, 2019

你好, 我 fork 了这个项目, 然后移了更多的 vue-element-admin 页面过来,不知能不能合到你的这个项目中

https://github.com/runner-mei/vue-typescript-admin-template

@Armour
Copy link
Owner

Armour commented Feb 26, 2019

看起来很棒 我正好打算以后慢慢把全部的vue-element-admin弄出来
你可以先pull下我最新的改动么 最近有几个push 稍微refactor了下code
另外我看你用了eslint的config 很多改动都是和style有关的 这部分一般editor都可以自己调整出来 而且每个人喜欢用的eslint config也不一样 所以我推荐你先注释掉那几行对code有大改动的config然后在发PR 这样我可以清楚的看到真正的feature改动有哪里 我们先focus功能 之后可以单独弄个code style的改动
等你有时间弄好后发个PR到develop branch吧 我review下 谢谢啦!

@mei-rune
Copy link
Author

mei-rune commented Feb 27, 2019

好的, 我这几天都在改
另外, eslint 配置,你可以发我一个, 我直接用你的吧

@mei-rune
Copy link
Author

mei-rune commented Mar 1, 2019

是不是可建一个任务列表,然后我有空一个个的改过来
每次我改之前先领个任务,然后提交, 省得我们之前做重了

@kevinlvhsl
Copy link

页面一进来就会把所有js都下载,是什么原因?
image

@ZurrTum
Copy link

ZurrTum commented Mar 4, 2019

@kevinlvhsl 你可以了解一下vue-cli:https://cli.vuejs.org/zh/guide/html-and-static-assets.html#prefetch

<link rel="prefetch"> 是一种 resource hint,用来告诉浏览器在页面加载完成后,利用空闲时间提前获取用户未来可能会访问的内容。

@kevinlvhsl
Copy link

@ZurrTum 了解了,谢谢

@Ghostdar
Copy link

@runner-mei @Armour 最近项目有接触到这个,也想参与迁移的工作,可以建个组件迁移的Todo List吗?方便管理。

@Armour
Copy link
Owner

Armour commented Mar 21, 2019

@Ghostdar @runner-mei #39 #40 我创建了一些todo list 然后你们可以从这里pick 然后发PR :)
如果觉得某个任务太大可以在分开成多个小任务

@Armour
Copy link
Owner

Armour commented Mar 21, 2019

@runner-mei 十分感谢你的贡献 你可以看看能不能把你的那个很大的PR分开分别对应上这些任务 这样会好code review很多

@Armour
Copy link
Owner

Armour commented Mar 23, 2019

@runner-mei 我把style方面已经定下来了 然后create了一个full-features分支 以后全功能版本的就在这个上面开发了 这个issue我先close了

@Armour Armour closed this as completed Mar 23, 2019
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

No branches or pull requests

5 participants