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

feat: 用于支持 Uploader 批量合并上传功能 #486

Merged
merged 8 commits into from
Mar 4, 2022

Conversation

YikaJ
Copy link
Contributor

@YikaJ YikaJ commented Mar 3, 2022

🤔 这个 PR 的性质是?

  • 新特性提交

🔗 相关 Issue

暂无

💡 需求背景和解决方案

具体场景:
多文件上传时, 可以支持一个请求包含多个文件同时上传的能力。当前 TD 只支持单文件上传的模式。

增加新的API:

  • uploadAllFilesInOneRequest: 上传时是否只使用一个接口上传所有的文件,需配合 multiple 使用。
  • batchUpload: 是否为合并上传模式,配合 multiple 与 uploadAllFilesInOneRequest 一起使用。当处于合并上传模式时,每次选择上传文件都会清空已上传的文件列表,等待再次合并上传。

image

image

注:需要配合 tdesign-common 的 PR 共同使用
#267 feat: 用于支持单个请求可以上传多个文件

📝 更新日志

  • feat(Uploader): 支持一个请求合并上传文件

  • 本条 PR 不需要纳入 Changelog

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

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

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

@YikaJ YikaJ changed the title Feat/batch upload feat: 用于支持单个请求可以上传多个文件 Mar 3, 2022
@YikaJ YikaJ changed the title feat: 用于支持单个请求可以上传多个文件 feat: 用于支持 Uploader 批量合并上传功能 Mar 3, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2022

完成

src/upload/flow-list.tsx Outdated Show resolved Hide resolved
src/upload/flow-list.tsx Outdated Show resolved Hide resolved
src/upload/flow-list.tsx Outdated Show resolved Hide resolved
src/upload/upload.tsx Show resolved Hide resolved
src/upload/upload.tsx Outdated Show resolved Hide resolved
src/upload/upload.tsx Outdated Show resolved Hide resolved
@YikaJ
Copy link
Contributor Author

YikaJ commented Mar 4, 2022

@chaishi
CI 里的 lint 依赖该 PR 的更新。
#267 feat: 用于支持单个请求可以上传多个文件

@uyarn uyarn force-pushed the feat/batch-upload branch from ee3ab0c to 4e7b631 Compare March 4, 2022 07:55
@uyarn uyarn merged commit ec3f97d into Tencent:develop Mar 4, 2022
@YikaJ YikaJ deleted the feat/batch-upload branch March 4, 2022 08:59
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.

3 participants