Skip to content

Conversation

@mkitsdts
Copy link

@mkitsdts mkitsdts commented Jan 1, 2026

Description / 描述

添加了转存分享文件的接口

Motivation and Context / 背景

管理界面无法直接存储分享的文件,所以加了一个新的接口转存分享文件。

How Has This Been Tested? / 测试

先测试了分享链接中提取分享码的函数。

然后直接在前端对文件做转存测试,目前天翼云盘和夸克云盘测试没有问题。

Checklist / 检查清单

  • I have read the CONTRIBUTING document.
    我已阅读 CONTRIBUTING 文档。
  • I have formatted my code with go fmt or prettier.
    我已使用 go fmtprettier 格式化提交的代码。
  • I have added appropriate labels to this PR (or mentioned needed labels in the description if lacking permissions).
    我已为此 PR 添加了适当的标签(如无权限或需要的标签不存在,请在描述中说明,管理员将后续处理)。
  • I have requested review from relevant code authors using the "Request review" feature when applicable.
    我已在适当情况下使用"Request review"功能请求相关代码作者进行审查。
  • I have updated the repository accordingly (If it’s needed).
    我已相应更新了相关仓库(若适用)。

@KirCute KirCute changed the title feat(transfer api):add transfer share file api feat(fs): add transfer share file api Jan 1, 2026
@KirCute
Copy link
Member

KirCute commented Jan 1, 2026

MaybeCannotTransfer接口用于在另一个 PR 中为alias驱动实现驱动穿透转存功能

KirCute
KirCute previously approved these changes Jan 1, 2026
@KirCute KirCute requested a review from j2rong4cn January 1, 2026 18:25
@xrgzs
Copy link
Member

xrgzs commented Jan 2, 2026

前端

@mkitsdts 前端麻烦 pr 一下

@mkitsdts
Copy link
Author

mkitsdts commented Jan 2, 2026

我是不是闯祸了😭我修改了一下189的驱动,然后删掉了UA,然后用默认方式合并代码

@xrgzs
Copy link
Member

xrgzs commented Jan 2, 2026

我是不是闯祸了😭我修改了一下189的驱动,然后删掉了UA,然后用默认方式合并代码

没问题,后面重新 approve 一下就好了😁

@xrgzs
Copy link
Member

xrgzs commented Jan 2, 2026

有个小缺陷,不支持配置转存分享链接的目录,如果分享链接里面的文件太多,实际只需要一小部分,可能没办法进行选择

@mkitsdts
Copy link
Author

mkitsdts commented Jan 2, 2026 via email

@KirCute KirCute added the Module: Filesystem File-system operations related issue/PR label Jan 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Module: Filesystem File-system operations related issue/PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants