F-loat / mpvue-entry

⛓️ 集中式页面配置，统一构建入口，支持新增页面热更新

MIT License

353 stars 56 forks source link

兼容自定义page入口文件模板 #72

Closed HelloZJW closed 6 years ago

HelloZJW commented 6 years ago

为了解决 mpvue issue 140 我创建了mpvue-page-factory, 修改了page entry的写法。使用mpvue-entry搭配mpvue-page-factory需要兼容自定义page entry模板。具体查看讨论mpvue-page-factory/issues/1

codecov[bot] commented 6 years ago

Codecov Report

Merging #72 into master will increase coverage by 0.15%. The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
+ Coverage   53.62%   53.78%   +0.15%     
==========================================
  Files           8        8              
  Lines         248      251       +3     
  Branches       47       48       +1     
==========================================
+ Hits          133      135       +2     
- Misses        115      116       +1

Impacted Files	Coverage Δ
lib/index.js	`32.3% <66.66%> (+1.66%)`	:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 11eb267...42a8c84. Read the comment docs.

F-loat commented 6 years ago

模板的路径是可以自定义的，你这个判断方式有问题吧，而且建议加个参数来控制是否对模板进行处理，或者直接支持自定义模板解析函数

HelloZJW commented 6 years ago

@F-loat 模板的路径是可以自定义，但是你的模板处理逻辑是写死的。我这个判断方式确实不妥，我从外部传一个模板解析函数

F-loat commented 6 years ago

你直接在原分支继续提交也可以的

F-loat commented 6 years ago

这样可以的，具体代码可能我明天看着改改再合，但是原本的解析函数是哪个地方会有问题？

HelloZJW commented 6 years ago

我发现之前是我自己写的模板有问题，以为是你这个解析代码导致的。你觉得这个外部传解析函数有必要就合，没必要就算了。不好意思，浪费你时间了。

F-loat commented 6 years ago

好的，这个还是可以合的