-
Notifications
You must be signed in to change notification settings - Fork 536
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
Conversation
zqjimlove
commented
Jun 2, 2020
👍👍 谢谢,我还没仔细看代码 只是我记得 Notion 不是没有所谓图床吗?你是怎么实现的? 另外,这些代码估计可能够呛能 Merge 成功,因为很多代码都改版了,下个版本发布前,我试着 Merge 下吧,如果不行的话,到时候再单独加上你的贡献。 |
其实是把图片下载转成Blob然后调了notion的上传接口。和上传md文件差不多。如果merge不成功的话,可以等改版之后我来迁移一下代码。 |
emmm 思路很清奇,好棒 👍 我来做这个合并吧,如果有需求的话,再联络你。 |
# Conflicts: # src/module/authorize.jsx
嗨,我上传了一份 1.1.4.6016 代码没有包含你 Pull 的内容,我在想你提交的功能,能否改为:
由 #799 提出的建议。 另外,不知道研究过 databases 自带链接的方式? |
可以,我现在刚已经把最新的 dev 分支合并了。我修改一下。 |
原生的剪藏方案是直接提交URL的方式,处理过程都在Notion的服务端进行的,所以有很大的区别。 |
原来如此... 那还是目前的方式吧。可能下个 1.1.4.xxx 版本时再上这个功能。( 先看下是否好用~ |
我发现 notion 授权有报错,可能我的帐号比较多的 空间和block,其中会有些问题。我尝试修复一下 |
嗨,我 Merge 了 #800 的代码后,做了以下一些小修改:
新的方式 simpread/src/service/export.js Line 1353 in f3c3bdf
这么做的主要原因是:我自己写的 Notion 图床 功能,你可以直接从 PS:我顺道关闭了这个 PR 😀 |
好的 |