i5ting / imove

INACTIVE: Move your mouse, generate code from flow chart
https://www.yuque.com/imove/docs/hvu0md
MIT License
3.75k stars 343 forks source link

feat: 移除pipe的时候,同步修改源码模式下的数据 #74

Closed cbbfcd closed 3 years ago

cbbfcd commented 3 years ago

soF5wt.png

操作步骤: 1、可视化模式下,增加 pipe 中的 key-value 2、运行 3、删除 pipe 中的任意一条,发现源码模式的数据没有同步,导致运行出来的结果也不对(依然有删除的那一条) 原因: <MinusCircleOutlined onClick={() => remove(field.name)} /> 这里执行移除的时候并没有触发 formonChange 事件, 看了一下 antd 文档,建议使用 onValuesChange,测试了一下移除的时候可以触发了,只是掉入了另一个坑,就是 add 的时候同样触发了,所以加了一个锁解决。

SmallStoneSK commented 3 years ago

多谢反馈~ 看了下确实某些场景下 onChange 不会触发,而 onValuesChange 则能够捕获 Form 下的所有 field 变化。不过看了下你的修改,应该不用加锁处理那么麻烦。

已经在 0.3.x 上修复,commit: f122e3f2c024e1bea4a33e5e79b86ba94a19e08a

i5ting commented 3 years ago

@cbbfcd 感谢兄弟,在钉钉群里at我一下

cbbfcd commented 3 years ago

@cbbfcd 感谢兄弟,在钉钉群里at我一下

😂,我先下个钉,狼叔