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(Table): validateTableCellData not exported #4940

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

Conversation

Wesley-0808
Copy link
Contributor

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

💡 需求背景和解决方案

#4848中的问题是因为使用了validateRowData方法,该方法只能校验单行的数据,且二次触发会导致上次数据丢失(覆盖)

image

代码中有个validateTableCellData方法用来校验全部可编辑单元格,但该方法未导出。

使用该方法可以满足issue的需求,同时补充了缺失的问题。

image

📝 更新日志

  • fix(table): 修复缺失校验可编辑单元格方法的问题

  • 本条 PR 不需要纳入 Changelog

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

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

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

@Wesley-0808
Copy link
Contributor Author

貌似三个技术栈都有这个问题?
没问题的话approve后我再补充下其他技术栈和文档

Copy link
Contributor

github-actions bot commented Jan 20, 2025

完成

@uyarn
Copy link
Collaborator

uyarn commented Jan 21, 2025

貌似三个技术栈都有这个问题? 没问题的话approve后我再补充下其他技术栈和文档

可以 @Wesley-0808 , 但是需要增加这个对外暴露方法的文档的描述

@Wesley-0808
Copy link
Contributor Author

貌似三个技术栈都有这个问题? 没问题的话approve后我再补充下其他技术栈和文档

可以 @Wesley-0808 , 但是需要增加这个对外暴露方法的文档的描述

@uyarn 我是等api那边merge了再push 还是我先push上来?😃

@uyarn
Copy link
Collaborator

uyarn commented Jan 22, 2025

貌似三个技术栈都有这个问题? 没问题的话approve后我再补充下其他技术栈和文档

可以 @Wesley-0808 , 但是需要增加这个对外暴露方法的文档的描述

@uyarn 我是等api那边merge了再push 还是我先push上来?😃

@chaishi 评估下这个是否export

@Wesley-0808
Copy link
Contributor Author

貌似三个技术栈都有这个问题? 没问题的话approve后我再补充下其他技术栈和文档

可以 @Wesley-0808 , 但是需要增加这个对外暴露方法的文档的描述

@uyarn 我是等api那边merge了再push 还是我先push上来?😃

@chaishi 评估下这个是否export

好的

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.

2 participants