Skip to content

Conversation

@varg1714
Copy link
Contributor

Description / 描述

为移动和复制操作增加跳过已存在文件的选项。

Motivation and Context / 背景

当前移动或复制操作在未勾选覆盖文件的情况下,若存在重名文件则直接失败。在大批量的文件的操作中出现此情况时需要在页面上手动取消勾选已存在的文件,操作不方便。基于此功能可快速跳过已存在的文件。

How Has This Been Tested? / 测试

  • 勾选 overwrite 选项,则 skipExisting 选项默认置灰
  • 未勾选overwrite 选项,可手动勾选 skipExisting,根据勾选结果决定跳过重复文件还是直接报错返回
  • 勾选 skipExisting 的情况下,若再次勾选 overwrite 选项,则 skipExisting 将被取消勾选

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).
    我已相应更新了相关仓库(若适用)。

>
{t("home.conflict_policy.skip_existing")}
</Checkbox>
<Checkbox
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy相比Move多增加了一个merge的选项,因此这块逻辑分开了,麻烦重新review一下

fix(fs): Code smell.

fix(fs): Code smell.
@varg1714 varg1714 force-pushed the feature/varg-1031-fs branch from 896239c to 4cb31af Compare November 2, 2025 05:26
@jenfonro
Copy link
Contributor

推送了个pr给你 varg1714#1

修复了取消与确定的对齐问题,前端这边我好像是没法直接推送至这个pr

fix: align Copy/Move modal actions with new skip options
@jyxjjj jyxjjj merged commit 7c369df into OpenListTeam:main Nov 24, 2025
1 check passed
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.

4 participants