Skip to content

Conversation

@cyfung1031
Copy link
Collaborator

@cyfung1031 cyfung1031 commented Nov 15, 2025

概述 Descriptions

Prevent accident deletion. If it is deleted in the cloud and also deleted in local, then no way to go back.

变更内容 Changes

截图 Screenshots

@CodFrm
Copy link
Member

CodFrm commented Nov 18, 2025

可能是bug,我先确定是个什么情况

@cyfung1031
Copy link
Collaborator Author

可能是bug,我先确定是个什么情况

就算不是Bug也应该不是Default吧
同步大多数都是不想掉失脚本
所以"加" 是用户想的,"减"是用户不太想的

当然如果有回收筒功能,保持 default syncDelete 也可以吧
完全没有复原方法就有点可怕了
误删之类

@cyfung1031 cyfung1031 force-pushed the pr-Sync-Delete-disable-by-default branch from 5039028 to 08c5478 Compare December 27, 2025 00:54
@CodFrm
Copy link
Member

CodFrm commented Jan 6, 2026

这个功能对于多设备用户很有用,一台机器上删了,另外一台机器也会删除

如果关闭的话,一台机器删了,另外一台机器不删除又会被同步上来,造成删不掉的情况

感觉主要看用户是用作同步,还是只是一个云备份

@cyfung1031
Copy link
Collaborator Author

一台机器删了,另外一台机器不删除又会被同步上来,造成删不掉的情况

我觉得假如用户有其他的同步工具,也会跟脚本猫的同步做成这样的冲突吧

再退一步讲,即使不使预设,用户打勾了,你说的这个问题还是会有呀
应该要想方法避免这个问题。跟是不是预设是另一个话题

@CodFrm
Copy link
Member

CodFrm commented Jan 7, 2026

一台机器删了,另外一台机器不删除又会被同步上来,造成删不掉的情况

我觉得假如用户有其他的同步工具,也会跟脚本猫的同步做成这样的冲突吧

再退一步讲,即使不使预设,用户打勾了,你说的这个问题还是会有呀 应该要想方法避免这个问题。跟是不是预设是另一个话题

还有会什么同步工具,这肯定是错误的操作

勾选了,就会给这个脚本标记删除,就不会存在这个删不掉的情况

不勾选是删除这个脚本相关的内容,如果有另外一台设备同步的话,获取文件列表的时候本地有这个脚本,但是云端没有,就又会同步本地的文件上去,删除的设备下一次同步时,发现云端有脚本,但是本地没脚本,又会同步下来,造成脚本删不掉的情况

现在用户的问题是,脚本删不掉,而不是脚本被误删

不过你说的也有道理,同步删除的话只有一个不起眼的通知,用户也不知道被删除了,在没有回收站的情况下无法恢复,现在可以默认设置为false,加上一些提示告知用户这个功能

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

本 PR 将 syncDelete 配置的默认值从 true 改为 false,以防止意外删除脚本。当脚本在云端被删除后,如果同步删除被启用,本地脚本也会被删除,这样就无法恢复了。

主要变更:

  • syncDelete 的默认值从 true 改为 false
  • 在设置界面的同步删除选项上添加了 Popover 提示,解释该功能的作用
  • 为所有支持的语言添加了 sync_delete_desc 翻译文本

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/pkg/config/config.ts CloudSyncConfigsyncDelete 默认值从 true 改为 false
src/pages/options/routes/Setting.tsx 为"同步删除"复选框添加 Popover 悬停提示,并添加 eslint-disable 注释以抑制 exhaustive-deps 警告
src/locales/zh-TW/translation.json 添加繁体中文的 sync_delete_desc 说明文本
src/locales/zh-CN/translation.json 添加简体中文的 sync_delete_desc 说明文本
src/locales/vi-VN/translation.json 添加越南语的 sync_delete_desc 说明文本
src/locales/ru-RU/translation.json 添加俄语的 sync_delete_desc 说明文本
src/locales/ja-JP/translation.json 添加日语的 sync_delete_desc 说明文本
src/locales/en-US/translation.json 添加英语的 sync_delete_desc 说明文本
src/locales/de-DE/translation.json 添加德语的 sync_delete_desc 说明文本

@CodFrm CodFrm merged commit 9c4c7dc into scriptscat:main Jan 7, 2026
2 of 3 checks 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.

2 participants