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

fix(reactive): fix wrong reaction._boundary #4255

Open
wants to merge 1 commit into
base: formily_next
Choose a base branch
from

Conversation

meowtec
Copy link

@meowtec meowtec commented Dec 30, 2024

Before submitting a pull request, please make sure the following is done...

  • Ensure the pull request title and commit message follow the Commit Specific in English.
  • Fork the repo and create your branch from master or formily_next.
  • If you've added code that should be tested, add tests!
  • If you've changed APIs, update the documentation.
  • Ensure the test suite passes (npm test).
  • Make sure your code lints (npm run lint) - we've done our best to make sure these rules match our internal linting guidelines.

Please do not delete the above content


What have you changed?

Fixes #3837 #3666 #4212

#3892 解决的是同一个问题,但是实现方式不同。(我的实现逻辑参考了 vue 的 endBatch)

另外, #3892 新增的一些测试用例在这里没有通过(actual 次数比 expected 少一次),需要再研究一下是否符合预期。

@CLAassistant
Copy link

CLAassistant commented Dec 30, 2024

CLA assistant check
All committers have signed the CLA.

@meowtec meowtec changed the title [WIP]fix(reactive): fix wrong reaction._boundary fix(reactive): fix wrong reaction._boundary Dec 30, 2024
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.

[Bug Report] formily react被动联动计算与预期不符合
2 participants