fix: @logicflow/vue-node-registry 1.1.13版本打包没有js#2391
Conversation
@logicflow/vue-node-registry@1.1.14-alpha.1
|
WXLKaylee
changed the title
There was a problem hiding this comment.
Pull request overview
This PR fixes the build failure in @logicflow/vue-node-registry@1.1.13 by updating Vue Teleport cleanup calls to match the disconnect(id, flowId) function signature.
Changes:
- Pass
flowIdas the required second argument todisconnect(...)inVueNodeViewwhen cleaning up Teleport mounts in early-return branches.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
| // Config exists but has no component; also clean up any existing Teleport mount | ||
| if (isVue3 && isActive()) { | ||
| disconnect(this.targetId()) | ||
| disconnect(this.targetId(), this.props.graphModel.flowId as string) | ||
| } |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Fixes #2386
修复 @logicflow/vue-node-registry 1.1.13版本打包没有js问题；
原因：/LogicFlow/packages/vue-node-registry/src/view.ts 中 disconnect 缺少必要参数。