Skip to content

fix(Form): add success validation type to setValidateMessage #6147

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
Merged
uyarn merged 4 commits into from
Nov 14, 2025
Merged

fix(Form): add success validation type to setValidateMessage #6147

uyarn merged 4 commits into from
Nov 14, 2025

Conversation

@RSS1102
Copy link
Collaborator

@RSS1102 RSS1102 commented Nov 10, 2025 •
edited by uyarn
Loading

🤔 这个 PR 的性质是？

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他

🔗 相关 Issue

💡 需求背景和解决方案

这里只设置验证状态为 success，如果需要 Input 也有成功的边框需要设置 success-border

📝 更新日志

  • 本条 PR 不需要纳入 Changelog

tdesign-vue-next

  • fix(form): 在 setValidateMessage 中添加 success 验证类型支持

@tdesign-vue-next/chat

@tdesign-vue-next/auto-import-resolver

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项。⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

@RSS1102 RSS1102 mentioned this pull request Nov 10, 2025
Open
@tdesign-bot
Copy link
Collaborator

tdesign-bot commented Nov 10, 2025 •
edited
Loading

TDesign Component Site Preview Open

Component Preview
tdesign-vue-next 完成
@tdesign-vue-next/chat 完成

@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 10, 2025 •
edited
Loading

tdesign-vue-next-demo

npm i https://pkg.pr.new/Tencent/tdesign-vue-next/@tdesign-vue-next/auto-import-resolver@6147

npm i https://pkg.pr.new/Tencent/tdesign-vue-next@6147

npm i https://pkg.pr.new/Tencent/tdesign-vue-next/@tdesign-vue-next/chat@6147

commit: 380a28f

@RSS1102 RSS1102 force-pushed the rss1102/feat/form/set-validate-message/success branch from 00e2d4c to 5ed75ad Compare November 10, 2025 07:40
@RSS1102 RSS1102 changed the title fix(form): add success validation type to setValidateMessage fix(Form): add success validation type to setValidateMessage Nov 10, 2025
@RSS1102 RSS1102 marked this pull request as draft November 10, 2025 10:52
@RSS1102
Copy link
Collaborator Author

RSS1102 commented Nov 10, 2025

可能要考虑 InputNumber 是否也正常。

@Wesley-0808
Copy link
Collaborator

Wesley-0808 commented Nov 10, 2025

是这个issue嘛～

#6115

@RSS1102
Copy link
Collaborator Author

RSS1102 commented Nov 11, 2025

是这个issue嘛～

#6115

是滴，这里 success 展示绿色需要配合 success-border

@RSS1102
Copy link
Collaborator Author

RSS1102 commented Nov 13, 2025 •
edited
Loading

可能要考虑 InputNumber 是否也正常。

经过测试也是正常的。
350

210

@RSS1102 RSS1102 marked this pull request as ready for review November 13, 2025 09:10
uyarn
uyarn reviewed Nov 13, 2025
View reviewed changes
uyarn
uyarn reviewed Nov 13, 2025
View reviewed changes
uyarn
uyarn reviewed Nov 13, 2025
View reviewed changes
@uyarn uyarn merged commit 765cba1 into develop Nov 14, 2025
1 check passed
@uyarn uyarn deleted the rss1102/feat/form/set-validate-message/success branch November 14, 2025 05:37
@tdesign-bot tdesign-bot mentioned this pull request Nov 14, 2025
Merged
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@uyarn uyarn uyarn approved these changes

@PengYYYYY PengYYYYY Awaiting requested review from PengYYYYY PengYYYYY is a code owner

@chaishi chaishi Awaiting requested review from chaishi chaishi is a code owner

@zhangpaopao0609 zhangpaopao0609 Awaiting requested review from zhangpaopao0609 zhangpaopao0609 is a code owner

@liweijie0812 liweijie0812 Awaiting requested review from liweijie0812 liweijie0812 is a code owner

@Cat1007 Cat1007 Awaiting requested review from Cat1007 Cat1007 is a code owner

@Wesley-0808 Wesley-0808 Awaiting requested review from Wesley-0808 Wesley-0808 is a code owner

@zydemail zydemail Awaiting requested review from zydemail zydemail is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@RSS1102 @tdesign-bot @Wesley-0808 @uyarn