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

[Table] validateTrigger: 'change' 校验不正确 #2219

Closed
NiLinli opened this issue May 16, 2023 · 4 comments
Closed

[Table] validateTrigger: 'change' 校验不正确 #2219

NiLinli opened this issue May 16, 2023 · 4 comments
Assignees
Labels
🐞 bug Something isn't working to be published fixed, not be published

Comments

@NiLinli
Copy link

NiLinli commented May 16, 2023

tdesign-react 版本

1.1.4

重现链接

https://stackblitz.com/edit/react-inliuo?file=src%2Fdemo.jsx

重现步骤

官网demo

  1. 申请人 col 添加 validateTrigger: 'change',
  2. 在对应的 input 输入任意一个字

期望结果

没有任何提示

实际结果

会提示 不能为空

bug原因分析:
输入 ‘你' 校验的是 ''
输入 '你好' 校验的是 '你'
bug 在于校验的是 prevValue

框架版本

React

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

No response

@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

♥️ 有劳 @yunfeic @tengcaifeng @Yacent @basecss 尽快确认问题。
确认有效后将下一步计划和可能需要的时间回复给 @NiLinli

@chaishi
Copy link
Collaborator

chaishi commented Jun 18, 2023

validateTrigger: 'change' 原计划是应用在 可编辑单元格场景,对于 可以编辑行 场景,没有特地支持和校验。(可编辑单元格场景,也校验了 preValue,需同步修复)

需要针对 可以编辑行 场景进行开发验证

@chaishi chaishi added the 🐞 bug Something isn't working label Jun 18, 2023
@chaishi chaishi mentioned this issue Jun 18, 2023
16 tasks
@chaishi
Copy link
Collaborator

chaishi commented Jun 18, 2023

resolved in #2297

@chaishi chaishi added the to be published fixed, not be published label Jun 18, 2023
This was referenced Jun 20, 2023
@uyarn uyarn closed this as completed Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working to be published fixed, not be published
Projects
None yet
Development

No branches or pull requests

3 participants