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

Improve copying database block #11460

Closed
TCOTC opened this issue May 19, 2024 · 43 comments
Closed

Improve copying database block #11460

TCOTC opened this issue May 19, 2024 · 43 comments
Assignees
Milestone

Comments

@TCOTC
Copy link
Contributor

TCOTC commented May 19, 2024

  • 改进 复制 文案为 复制镜像/Copy mirror

  • 改进 复制为副本 文案为 复制为镜像副本/Duplicate mirror

  • 增加 复制为完整副本/Duplicate completely 菜单项

    副本数据库块中的数据库名称:<数据库名称> (Duplicated xxxx-xx-xx xx:xx:xx)(中间有个空格)


数据库块复制为副本时创建一个新数据库块

Originally posted by @88250 in #9285 (comment)

深克隆有点问题,Notion 中重复(Duplicate)数据库块时是把视图中的所有 page 都深克隆一次。我们没法这么做,因为块不只是文档块,还有其他块,其他块没有文档容器的话无法重复。

可能还是只能浅克隆 😂 再考虑看看,各位有什么建议欢迎讨论啊。

话说这个问题能有解决方案吗?我有深克隆数据库的需求

我试了一下把数据库的 json 复制一份,手动改文件名和里面的部分 id ,然后在文档里添加了这个数据库块。可以看到这不是镜像数据库,点击主键都会跳转到同一个文档:

image

但我即使是重建索引之后,在这个文档的数据库属性里都只能看到原来的数据库:

image

@88250
Copy link
Member

88250 commented May 19, 2024

完整的深克隆做不了,原因在之前的讨论中已经描述。

如果只是做数据库的克隆(不克隆块),那可以试试导出 .sy.zip 导入。

上面提到手动修改 id 的方式在块中不生效是因为块属性 avs 没有改,我刚刚发现导入 .sy.zip 也没有生效,这个问题稍后会解决。

@TCOTC
Copy link
Contributor Author

TCOTC commented May 19, 2024

我刚刚发现导入 .sy.zip 也没有生效,这个问题稍后会解决。

我这里导入数据库感觉没有问题,绑定块的数据库属性是正常显示的。

@TCOTC
Copy link
Contributor Author

TCOTC commented May 19, 2024

完整的深克隆做不了,原因在之前的讨论中已经描述。

我还是不太理解为什么做不了

手动修改 id 的方式在块中不生效是因为块属性 avs 没有改

意思是改了就行了吗?

@88250
Copy link
Member

88250 commented May 19, 2024

我这里导入数据库感觉没有问题,绑定块的数据库属性是正常显示的。

同一个文档中的块正常,如果块在其他文档中就不正常了。

我还是不太理解为什么做不了

没有地方存放克隆块。

意思是改了就行了吗?

还是不能完整深克隆,只是修复上面说的 bug。

@TCOTC
Copy link
Contributor Author

TCOTC commented May 19, 2024

我还是不太理解为什么做不了

没有地方存放克隆块。

我的想法是仍然绑定原来的块,不需要克隆块。

如果需要克隆块的话只用导出文档再导入就有双份的了

相当于把原来绑定的块批量添加进一个新的数据库(只不过这个数据库不是空的)

@88250
Copy link
Member

88250 commented May 19, 2024

是的,现在只能做到这一步,所以不是完整的深克隆。

@TCOTC
Copy link
Contributor Author

TCOTC commented May 19, 2024

是的,现在只能做到这一步

那这个功能可以做吗?我不需要完整的深克隆,只要数据库是双份的就行了

p.s. 好像其他人也是这个需求:

只要数据库是双份的,不需要把数据库绑定的块也变成双份的。因为是要创建多个独立的数据库去管理同一个块

@88250
Copy link
Member

88250 commented May 19, 2024

这个应该没问题,我们先解决上面提到的导入后绑定已有块的属性不正确问题。

@TCOTC TCOTC changed the title 数据库块深克隆 数据库块复制为副本时创建一个新数据库块 May 19, 2024
@TCOTC
Copy link
Contributor Author

TCOTC commented May 19, 2024

  • 数据库块 复制为副本 时创建一个新数据库块,关联已绑定的块
  • 数据库块 复制粘贴 仍然是创建镜像数据库(保持不变)
  • 仅改变数据库块 复制为副本 的行为,跟其他类型的块 复制为副本 的行为保持一致(创建新块,而不是镜像块)

@TCOTC

This comment was marked as outdated.

@88250 88250 changed the title 数据库块复制为副本时创建一个新数据库块 Improve duplicating database block May 23, 2024
@88250 88250 added this to the 3.0.16 milestone May 23, 2024
@5kyfkr
Copy link

5kyfkr commented May 23, 2024

  • 数据库块 复制为副本 时创建一个新数据库块,关联已绑定的块

副本的数据库标题应该要加上个(独立副本)的标识,不然会以为是镜像数据库

@TCOTC
Copy link
Contributor Author

TCOTC commented May 23, 2024

@5kyfkr 镜像数据库 本来就有单独的标识,不需要这个

@TCOTC
Copy link
Contributor Author

TCOTC commented May 23, 2024

@88250 数据库块 复制为副本 后,数据库名称增加一段 <数据库名称> (Duplicated xxxx-xx-xx xx:xx:xx)(中间有个空格)

@88250
Copy link
Member

88250 commented May 23, 2024

好的

@88250
Copy link
Member

88250 commented May 23, 2024

和 V 讨论了下,我们会多加一个块标菜单 复制数据库块副本 而不是修改 Ctrl+D(复制为副本),因为后者用户可能难以察觉其区别,并且改动以后这个行为和其他块会不一致,所以加一个块标菜单来突出这个功能比较好。

@5kyfkr
Copy link

5kyfkr commented May 23, 2024

那数据库块就没有“修改 Ctrl+D(复制为副本)”这个了吧,相当于被 复制数据库块副本 这个取代

@88250
Copy link
Member

88250 commented May 23, 2024

Ctrl+D 保持不变,依然是复制镜像块。

@5kyfkr
Copy link

5kyfkr commented May 23, 2024

Ctrl+D 保持不变,依然是复制镜像块。

两个选项都叫复制副本。。有点难理解
应该 Ctrl+D叫 复制镜像数据库副本,然后新加的叫 复制独立数据库副本,名字可能直观点,但是ctrl d估计为了保持其它块相同改不了名字,新加的可以突出一下是独立的新副本

@88250 88250 changed the title Improve duplicating database block Add Duplicate database block May 23, 2024
@88250
Copy link
Member

88250 commented May 23, 2024

对于数据库来说,默认的复制就是镜像,这个不用单独强调了吧,Ctrl+D 和 复制+粘贴 一样的。单独强调非镜像复制就可以了。

@5kyfkr
Copy link

5kyfkr commented May 23, 2024

对于数据库来说,默认的复制就是镜像,这个不用单独强调了吧,Ctrl+D 和 复制+粘贴 一样的。单独强调非镜像复制就可以了。

可以,就单独强调非镜像的够了

@88250 88250 changed the title Add Duplicate database block Add Duplicate non-mirror database block May 23, 2024
@88250
Copy link
Member

88250 commented May 23, 2024

我们再考虑下。

@88250 88250 changed the title Add Duplicate non-mirror database block Improve copying database block May 23, 2024
@88250
Copy link
Member

88250 commented May 23, 2024

  • 改进 复制 文案为 复制镜像/Copy mirror

  • 改进 复制为副本 文案为 复制为镜像副本/Duplicate mirror

  • 增加 复制为完整副本/Duplicate completely 菜单项

    副本数据库块中的数据库名称:<数据库名称> (Duplicated xxxx-xx-xx xx:xx:xx)(中间有个空格)

@TCOTC
Copy link
Contributor Author

TCOTC commented May 23, 2024

复制为完整副本复制为镜像副本 最好是放在同一个二级菜单下挨着

@88250
Copy link
Member

88250 commented May 23, 2024

是的,放在 复制 菜单下面

@TCOTC
Copy link
Contributor Author

TCOTC commented May 23, 2024

要不把 复制纯文本 放到 复制 的上面?让 复制复制为副本 挨着

image

@88250
Copy link
Member

88250 commented May 23, 2024

还有个复制文本*呢?

@TCOTC
Copy link
Contributor Author

TCOTC commented May 23, 2024

哦被我隐藏起来了,我就是觉得把复制放下面一点可能好一点

@88250
Copy link
Member

88250 commented May 23, 2024

等做完功能以后看看,我感觉怕是没有必要,这两个已经离得很近了。

@TCOTC
Copy link
Contributor Author

TCOTC commented May 23, 2024

现在这个 复制 不上不下的感觉很奇怪

@88250
Copy link
Member

88250 commented May 23, 2024

和其他块一样嘛

@TCOTC
Copy link
Contributor Author

TCOTC commented May 23, 2024

反正还是这个菜单感觉奇怪的问题,也不知道怎么改善

image

@88250
Copy link
Member

88250 commented May 23, 2024

上次讨论说是把图标都去掉?

@5kyfkr
Copy link

5kyfkr commented May 23, 2024

龙生九子,各有不同,应该把最常用的几个加上图标放最前面,ctrl +c应该加个图标放前面点,复制markdown链接那个也可以加个M图标,就不用费劲看文字了,其他同理,能加图标的都加就好看了,不然感觉不统一

@88250
Copy link
Member

88250 commented May 23, 2024

这个菜单只要少一个图标就会不和谐,所以考虑全部去掉。

Vanessa219 added a commit that referenced this issue May 23, 2024
Vanessa219 added a commit that referenced this issue May 23, 2024
Vanessa219 added a commit that referenced this issue May 23, 2024
@88250 88250 closed this as completed May 23, 2024
@TCOTC
Copy link
Contributor Author

TCOTC commented May 23, 2024

@88250 删图标 和 删「复制」

Screenshot_2024-05-24-00-15-41-09_9d26c6446fd7bb8e41d99b6262b17def.jpg

@5kyfkr
Copy link

5kyfkr commented May 23, 2024

这是什么群,我也想加个(坏笑)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants