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.

Sign up for GitHub

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

Jump to bottom

[Form.FormItem] shouldUpdate无限触发 #2942

Closed
1379255913 opened this issue Jun 4, 2024 · 1 comment · Fixed by #2948
Closed

[Form.FormItem] shouldUpdate无限触发 #2942

1379255913 opened this issue Jun 4, 2024 · 1 comment · Fixed by #2948
Labels
to be published fixed, not be published

Comments

@1379255913
Copy link

tdesign-react 版本

1.7.5

重现链接

https://stackblitz.com/edit/react-p3bsia-flr9iv?file=src%2Fdemo.jsx,package.json,src%2Findex.css

重现步骤

点击按钮添加项并进行修改时，shouldUpdate会无限触发，使用tdesign版本为1.4.3时未发生此问题
image

期望结果

shouldUpdate只会触发一次

实际结果

无限触发

框架版本

No response

浏览器版本

No response

系统版本

Windows

Node版本

No response

补充说明

No response
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 4, 2024

👋 @1379255913，感谢给 TDesign 提出了 issue。
请根据 issue 模版确保背景信息的完善，我们将调查并尽快回复你。

@1379255913 1379255913 changed the title [Form.FormItem] 描述问题的标题 [Form.FormItem] shouldUpdate无限触发 Jun 4, 2024
moecasts added a commit to moecasts/tdesign-react that referenced this issue Jun 5, 2024
@moecasts
fix(form): resolve shouldUpdate circular trigger problem
367aada 
fix Tencent#2942
@moecasts moecasts mentioned this issue Jun 5, 2024
Merged
16 tasks
@uyarn uyarn added the to be published fixed, not be published label Jun 16, 2024
uyarn pushed a commit that referenced this issue Jun 19, 2024
@moecasts
fix(form): resolve shouldUpdate circular trigger problem (#2948)
53404f6 
fix #2942
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
to be published fixed, not be published
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(form): resolve shouldUpdate circular trigger problem moecasts/tdesign-react
2 participants
@uyarn @1379255913