-
Notifications
You must be signed in to change notification settings - Fork 51
refactor: 移除timeout魔法 #21
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
Conversation
大佬们倒是review下PR呀 |
这个貌似还是依然未解决hot reload 问题 |
@jyootai 什么情况下不行,别貌似啊... |
@jkzing 感谢大佬的 PR ,上周五由于我们大团队团建,所以就没有合 PR 发版本,这周五测试通过了再合并发版本。 |
@jkzing 还有存在slot 嵌套不会热更新的问题 |
@jyootai 你说的是这种情况吗: <div class="userinfo-nickname">
<card :text="userInfo.nickName">
<hello>
<span>hi</span>
</hello>
</card>
</div> 我这边修改 (如果不是这个情况能不能麻烦贴下代码....) |
这种: // home
<div class="home-page">
<A></A>
</div>
// A
<div class="a-comp">
<B>
<div>修改此处</div>
</B>
</div>
// B
<div class="b-comp">
<slot></slot>
</div> |
@jyootai 没问题啊...同学你是不是没 帮你建了个repo,你可以用这个测试: https://github.com/jkzing/mpvue-slots-demo PS:有已知问题是如果新增组件的话不能正确 reload,不过这也不是这个 PR 要解决的事 |
用你的测试是可以的, 说明不是我机子问题. 我这也安装你的提交测试了, 也确实是不能刷新. 这周看看其他测试, 没有问题就合入发版. 之后在看看有没其他同学在复现这个问题. 感谢大佬 PR 👍 |
如果确认修复了能发版么,我每次黑魔法真的超累 |
@mdsb100 正式版发布前可以先用 |
嗯 好的。谢谢大佬! |
升级了以后发现props都绑不上去了 改回1.0.13就好了,还是自己记count的方式 |
@mdsb100 能否给个示例,不太明白你的意思,我本地测试觉得没问题 |
这是什么意思...能不能去 https://github.com/jkzing/mpvue-slots-demo 提个PR重现下你的问题 |
@jkzing 我去试试
buttonText根本没绑上去 |
@mdsb100 我试了下输出是这样的,是正常的 <button class="_button data-v-21261b62 btn-middle-normal timelineSaveButton" disabled="{{shareButtonDisabled}}" open-type="share">{{buttonText}}</button> |
我在尝试在demo上重现,稍等 |
@jkzing 我在demo上把我们项目中的代码组件放了进去,发现是好的。 我对一下package.json的依赖。 现在我能想到的区别是: 不知道有没有关系。 |
@jkzing 能不能提供一个联系方式 微信orQQ
|
@mdsb100 你得想办法在我给你的 demo repo 里重现,这样我才能帮你找问题。 |
确定我每次重现了。
删了package-lock.json就有问题了。 我不知道这算不算bug?我不记得quick-start那个template有没有默认提供lock文件,如果是npm install后生成的,那大家都可能遇到问题。 |
@mdsb100 |
我确定了 |
其他版本,我没来得及试。如果最新mpvue是好的,应该就是好的吧。 还有一点,我估计我们内部cnpm的镜像没有同步到最新的库吧。 |
我看了下npm。mpvue最新是1.0.12。所以应该是不好的。 |
Meituan-Dianping/mpvue#427 这个问题?当时我有测过,mpvue@1.0.11 在某些情况下也会出错,那个 pr 的改法看来还是有问题。。 |
soga。谢谢楼上各位大佬了。 |
OK,既然是 mpvue 的问题,就在那边继续讨论吧😅 |
* refactor: use async strategy intead of timeout * refactor: remove timeout in createSlotWxml * refactor: use compilation hook to create slots file
做了两件事:
不知道有没有其他情况没有考虑到
close Meituan-Dianping/mpvue#245
close Meituan-Dianping/mpvue#365