walinejs / waline

💬 A Simple, Safe Comment System
https://waline.js.org/en/
GNU General Public License v2.0
2.19k stars 379 forks source link

[Feature Request] 添加将评论存储为 GitHub Issues 的功能 #413

Closed slblog-github closed 3 years ago

Mister-Hope commented 3 years ago

这不太是waline计划提供的功能,它和现有架构是完全不兼容甚至冲突的。如有需要你可以使用其他项目,如vssue。

Mister-Hope commented 3 years ago

@lizheming 大哥的想法呢?

slblog-github commented 3 years ago

这不太是waline计划提供的功能,它和现有架构是完全不兼容甚至冲突的。如有需要你可以使用其他项目,如vssue。

Thanks for the reply,不过waline+leancloud也挺好用的

Mister-Hope commented 3 years ago

你这个建议的问题

  1. 目前现有结构根本和issue对不上,即使我们的其他字段以注释的方式存在issue里。GitHub存储的是markdown而我们目前是html,这点基本就不兼容
  2. 目前我们不要求用户登录,而GitHub issue必须登(除非你想用你的账户代发一堆issue)
  3. 根本不存在迁移和导入的可能

补充,还有一点是可能存在性能问题,我们提供的方案都是基于数据库的,而GitHub API的逻辑未必满足,所以我们在做中间件兼容现有逻辑的时候可能会被迫发送很多请求。响应速度可能也是一个问题。

Mister-Hope commented 3 years ago

所以我表达的核心观点是这种需求应该直接跟架构是对应上的,想要实现功能架构就得根据这个来做。🌚

lizheming commented 3 years ago

如 @Mister-Hope 由于架构不匹配的原因,官方不会支持 Issue 的这种形式。不过如果感兴趣的话,可以根据这个 https://waline.js.org/guide/server/databases.html#%E6%9B%B4%E5%A4%9A 自己实现一个基于 Github Issue 的阉割版。

另外有兴趣的话也可以试试 https://waline.js.org/guide/server/databases.html#github GitHub 存储方式~