4.x #686
Open
4.x #686
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* 升级项目基础依赖，优化构建流程、开发体验 (#590) * merge qiniu/master * 细节调整 * 将大量私有的方法标记为 deprecated & 更新文档 * 调试 workflow actions * 为 upload 的上传结果添加泛型 * 基本完成 * 移除一些非必要的文件及改动 * build 不应该使用 parallel * 添加 vscode workspace 配置文件 * update dev-test package.json * 针对 iife 输出 umd 格式 * update * just move files * init common package * 公共逻辑基本完成 * 微信 SDK 基本完成 * 优化错误和最终结果处理 * 添加微信小程序 example * 更好的类型名称以及移除 UploadBlob 的导出 * 移除微信小程序 SDK 直传 base64 的支持 * 调整示例目录结构和位置 * 错误处理等细节调整 * 添加鸿蒙基本实现（仅支持分片上传） * 为 queue 添加基本的单元测试 * UploadFile 设计内敛 * update lock * fix bug * update actions * fix typo * update package info * 移除与本次无关的浏览器端代码 * 补充上传时指定 key 和 mimetype 的能力 * 补充配置文件 * 添加自定义变量&metadata支持 * 更新配置文件 * 添加浏览器和 react-native 支持 * 添加 rn examples * 暂时移除 rn * 鸿蒙细节调整 * 添加鸿蒙系统兼容性检查 * 添加 browser example * 细节调整 * browser 添加 webWorker 演示 * 浏览器端文件名处理优化 * 优化鸿蒙的文件类型处理 * 优化鸿蒙底层文件处理 * 添加 uploadHosts 支持 * 更新说明文档 * update readme & changelog * 添加 v1 分片上传支持 * 优化类型信息 * add Progress export * fix ohpm install command in radme.md
* 升级鸿蒙 SDK 项目版本 * 优化 monorepo 结构 * update package script * update examples
* 更新 browser 端版本号 * fix typo
* 更新 browser 端版本号 * fix typo * update github actions
* 更新 browser 端版本号 * fix typo * update github actions * auto build common * update examples * fix typo
* 更新 browser 端版本号 * fix typo * update github actions * auto build common * update examples * fix typo * remove publish dry-run params
* 新增 key 参数，更明确指定上传文件名 * MockProgress 兼容多次调用 start * update version
* 使用 copy 共享 common 代码 * 调整引用 path * update tsconfig * update lock * update browser readme.md
* 修复发布文件缺失问题
* 临时修复任务进度回退问题 * 更新版本号
* 适当降低虚拟进度条的输出速度 * 细节优化
|
[Git-flow] Hi @yinxulai, There are some suggestions for your information: Rebase suggestions
Which seems insignificant, recommend to use For other If you have any questions about this comment, feel free to raise an issue here: |
YangSen-qn
reviewed
May 19, 2025
| "postinstall": "run-p build:*", | ||
| "build:browser": "cpx \"src/**/*\" ../browser/src/@internal", | ||
| "build:wechat-miniprogram": "cpx \"src/**/*\" ../wechat-miniprogram/src/@internal", | ||
| "build:harmony": "cpx \"src/**/*\" ../harmony/library/src/main/ets/components/@internal" |
YangSen-qn
reviewed
May 20, 2025
| const logLevel = config.logLevel || 'NONE' | ||
| const protocol = config.protocol || 'HTTPS' | ||
| const uploadHosts = config.uploadHosts || [] | ||
| const apiServerUrl = config.apiServerUrl || 'https://api.qiniu.com' |
There was a problem hiding this comment.
api.qiniu.com 为转发域名有性能问题，调整为：uc.qiniuapi.com
YangSen-qn
reviewed
May 20, 2025
| error: new common.UploadError('FileSystemError', 'Invalid file data') | ||
| } | ||
| } | ||
| if (typeof this.data.data !== 'string') { |
YangSen-qn
reviewed
May 20, 2025
| } | ||
|
|
||
| // 标准 file uri | ||
| if (this.data.data.startsWith('file://')) { |
There was a problem hiding this comment.
这里不做判断？如果 uri 不符合鸿蒙标准在 initFile 时依靠鸿蒙 API 处理，我们不增加这个处理负担（后续更新我们还需要适配影响客户使用）
YangSen-qn
reviewed
May 20, 2025
| } | ||
|
|
||
| // 一般是来自其他应用的文件 | ||
| if (this.data.data.startsWith('datashare://')) { |
There was a problem hiding this comment.
同上：这里不做判断？如果 uri 不符合鸿蒙标准在 initFile 时依靠鸿蒙 API 处理，我们不增加这个处理负担（后续更新我们还需要适配影响客户使用）
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.