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

主要是升级到了webpack4, 还在准备升级babel7 #11

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

Onesimu
Copy link

@Onesimu Onesimu commented Oct 16, 2018

本来想只把最开始的配置更改提交, 初次使用pr, 可能因为没有新建分支的缘故, 只能把master分支提交上去了. 后期已经加入了外层封装的组件, 使用了https://github.com/gwuhaolin/spring-data-rest-js 请求数据, https://github.com/agershun/alasql 作为前端存储. 作为交流, 欢迎指导.

@sunhaoxiang
Copy link
Member

非常感谢你为D2Crud贡献的代码。
对一部分人来说,将网络请求封装进组件可以让他们更方便的使用,不需要自己再封装网络请求;但是对另一部分用户来说,将是额外的开销。
因此,经过慎重考虑,我还是决定不将其加入D2Crud中。
再次感谢你的代码贡献!

@sunhaoxiang
Copy link
Member

另外,欢迎提一个webpack的配置pr~

@Onesimu
Copy link
Author

Onesimu commented Oct 18, 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