Skip to content

fix(types): 修复@select-change事件类型#825

Merged
timi137137 merged 4 commits intoTencent:developfrom
calandnong:develop
Mar 16, 2025
Merged

fix(types): 修复@select-change事件类型#825
timi137137 merged 4 commits intoTencent:developfrom
calandnong:develop

Conversation

@calandnong
Copy link
Copy Markdown
Contributor

@calandnong calandnong commented Mar 14, 2025

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

💡 需求背景和解决方案

📝 更新日志

  • chore(deps): 升级组件库版本

  • fix(types): 修复@select-change事件类型

  • 本条 PR 不需要纳入 Changelog

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

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

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

…t/base/index.vue下t-table的类型失效问题

新tdesign-vue-next的版本到@1.11.3&解决src/pages/list/base/index.vue下t-table的类型失效问题
…select-change绑定的事件类型错误问题

修复src/pages/list/base/index.vue下@select-change绑定的事件类型错误问题,问题原因点:@select-change绑定的函数入参应是value:
Array<string | number> 而非  value: number[]
@calandnong calandnong changed the title fix: fix: 修复t-table组件未有类型提示问题&使用案例下t-table的@select-change绑定的事件类型错误问题 Mar 14, 2025
@calandnong
Copy link
Copy Markdown
Contributor Author

calandnong commented Mar 14, 2025

这里主要是在run build时会有类型错误抛出,建议修复下

Copy link
Copy Markdown
Collaborator

@timi137137 timi137137 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

请不要在vue文件中直接编写类型。
如果可以的话请顺手修复MenuRoute的类型问题。
感谢贡献

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 16, 2025

完成

@timi137137 timi137137 changed the title fix: 修复t-table组件未有类型提示问题&使用案例下t-table的@select-change绑定的事件类型错误问题 fix(types): 修复@select-change事件类型 Mar 16, 2025
@timi137137 timi137137 merged commit d956cd2 into Tencent:develop Mar 16, 2025
9 checks passed
@calandnong
Copy link
Copy Markdown
Contributor Author

请不要在vue文件中直接编写类型。 如果可以的话请顺手修复MenuRoute的类型问题。 感谢贡献

https://github.com/Tencent/tdesign-vue-next/pull/5198,这边跟进中,后续提个修改版本,是组件库的问题导致

@github-actions github-actions Bot mentioned this pull request Jun 17, 2025
16 tasks
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