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

Feature/notion save image #783

Closed

Conversation

zqjimlove
Copy link
Contributor

image

@Kenshin
Copy link
Owner

Kenshin commented Jun 2, 2020

👍👍

谢谢,我还没仔细看代码 只是我记得 Notion 不是没有所谓图床吗?你是怎么实现的?

另外,这些代码估计可能够呛能 Merge 成功,因为很多代码都改版了,下个版本发布前,我试着 Merge 下吧,如果不行的话,到时候再单独加上你的贡献。

@zqjimlove
Copy link
Contributor Author

zqjimlove commented Jun 2, 2020

其实是把图片下载转成Blob然后调了notion的上传接口。和上传md文件差不多。如果merge不成功的话,可以等改版之后我来迁移一下代码。

@Kenshin
Copy link
Owner

Kenshin commented Jun 2, 2020

emmm 思路很清奇,好棒 👍

我来做这个合并吧,如果有需求的话,再联络你。

@Kenshin
Copy link
Owner

Kenshin commented Jun 17, 2020

嗨,我上传了一份 1.1.4.6016

代码没有包含你 Pull 的内容,我在想你提交的功能,能否改为:

类似 Notion 原生剪藏那样,自动把图片更改为 Notion 图床 方案,而非现在 选项方案

#799 提出的建议。


另外,不知道研究过 databases 自带链接的方式?

image

@zqjimlove
Copy link
Contributor Author

可以,我现在刚已经把最新的 dev 分支合并了。我修改一下。
但是默认的话会有个问题就是剪藏的过程由于需要上传图片后才能上传文章。导致整个过程耗时很长,所以才考虑做成开关的。

@zqjimlove
Copy link
Contributor Author

原生的剪藏方案是直接提交URL的方式,处理过程都在Notion的服务端进行的,所以有很大的区别。

@Kenshin
Copy link
Owner

Kenshin commented Jun 17, 2020

@zqjimlove

原来如此... 那还是目前的方式吧。可能下个 1.1.4.xxx 版本时再上这个功能。( 先看下是否好用~

@zqjimlove
Copy link
Contributor Author

我发现 notion 授权有报错,可能我的帐号比较多的 空间和block,其中会有些问题。我尝试修复一下

@Kenshin
Copy link
Owner

Kenshin commented Jun 20, 2020

嗨,我 Merge 了 #800 的代码后,做了以下一些小修改:

  1. 格式化了一些代码,因为没有提供统一的代码规约,并且确实代码量也不大,所以我手动按照我个人喜好修改了一些代码格式。

  2. url_schema_key to schema 前者太长了,不是很美观。 😅

  3. 优化了 this.schema 获取后的保存流程( 这里也有个小问题 https://github.com/Kenshin/simpread/pull/800/files#diff-9793a3498d7ce495b3ead4d1f555a998R1566-R1568

    storage.Safe( callback, new_value ) 有两个参数时,才是保存。 😀

    新的方式

    if ( notion.type == "collection" && notion.schema != storage.secret.notion.schema ) {

  4. https://github.com/Kenshin/simpread/pull/800/files#diff-9793a3498d7ce495b3ead4d1f555a998R1364-R1368 这里也有个小问题,即:我不需要显示 type == space 的 value ,只是把它们展示出来,用户不小心选择时,默认为第一个 block

新的方式

Object.values( spaceMaps ).forEach( space => {

这么做的主要原因是:我自己写的 Dropdown 暂时不支持层级显示方式... 😱


Notion 图床 功能,你可以直接从 release/1.1.4.6022 这个分支修改,到时我直接 Merge 到此分支。

PS:我顺道关闭了这个 PR 😀

@Kenshin Kenshin closed this Jun 20, 2020
@zqjimlove
Copy link
Contributor Author

好的

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