tdesign-vue-next icon indicating copy to clipboard operation
tdesign-vue-next copied to clipboard

feat(form): 添加IP地址校验

Open lxKylin opened this issue 1 year ago • 13 comments
trafficstars

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

💡 需求背景和解决方案

📝 更新日志

  • feat(form): 添加IP地址校验

  • [ ] 本条 PR 不需要纳入 Changelog

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

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

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

lxKylin avatar Dec 21 '23 08:12 lxKylin

失败

github-actions[bot] avatar Dec 21 '23 08:12 github-actions[bot]

有API变动的PR嘛

PengYYYYY avatar Dec 21 '23 09:12 PengYYYYY

有API变动的PR嘛

没有变动,加了个校验IP,{ ip: true, message: '请输入正确的IP格式', type: 'warning' },

lxKylin avatar Dec 21 '23 09:12 lxKylin

@lxKylin 麻烦执行下npm run test:update

uyarn avatar Dec 26 '23 10:12 uyarn

@lxKylin 麻烦执行下npm run test:update

image 我的改动不至于影响到这吧🤔

lxKylin avatar Dec 26 '23 11:12 lxKylin

对 你的改动不会增加这个 新增的form-item示例 会新增snapshot 需要更新的是这部分快照 image node版本是多少?node版本可能会影响这块的snapshot 挂载的位置 (原因还没明确 最好用node 18 image

uyarn avatar Dec 26 '23 12:12 uyarn

对 你的改动不会增加这个 新增的form-item示例 会新增snapshot 需要更新的是这部分快照 image node版本是多少?node版本可能会影响这块的snapshot 挂载的位置 (原因还没明确 最好用node 18 image

用的是18

lxKylin avatar Dec 26 '23 12:12 lxKylin

稳定有这个diff吗 guide的

uyarn avatar Dec 26 '23 12:12 uyarn

稳定有这个diff吗 guide的

昂?啥意思,没懂🤦

lxKylin avatar Dec 26 '23 12:12 lxKylin

对 你的改动不会增加这个 新增的form-item示例 会新增snapshot 需要更新的是这部分快照 image node版本是多少?node版本可能会影响这块的snapshot 挂载的位置 (原因还没明确 最好用node 18 image

这个快照更新,是要怎么更新呀,运行那个命令么,还是要写测试用例

lxKylin avatar Dec 26 '23 12:12 lxKylin

@lxKylin 没事 明天我在你分支上改动吧

uyarn avatar Dec 26 '23 15:12 uyarn

这个还是改了API的 需要新增ip的formRules

uyarn avatar Jan 08 '24 07:01 uyarn

这个还是改了API的 需要新增ip的formRules

image 这个是么

lxKylin avatar Jan 08 '24 07:01 lxKylin