gitment icon indicating copy to clipboard operation
gitment copied to clipboard

organization下的仓库无法使用gitment

Open cebarobot opened this issue 8 years ago • 25 comments

Gitment 是一个很好用的评论插件! 但是,我在使用中发现 organization下的仓库无法使用gitment。我们有一个团队项目想要在项目主页中使用 Gitment 作为评论系统,但 owner 不能直接写团队名(没法登陆来初始化);若强行修改 js 使得团队成员能够初始化,每次刷新页面都会重新提示要初始化。 我们的网站生成器是自己写的,url 以及标题都不会发生变化,url 中存在中文(但是个人仓库并没有问题)。配置都采用了默认配置。 谢谢你!

cebarobot avatar Jul 22 '17 17:07 cebarobot

同样遇到这个问题

Whimsyduke avatar Jul 29 '17 15:07 Whimsyduke

同样遇到这个问题

hiyangguo avatar Aug 25 '17 06:08 hiyangguo

@hiyangguo @jiangzhonglian @Whimsyduke @cebarobot

Anybody, 能给我一个可以重现这个Bug的organization仓库不?

另外,有两个问题。第一,各位是不是希望organization之外的人就不能添加评论了?(现在是只要是github用户就可以添加,没有organization这样的限制)。我不太清楚这是否是真实的需求,也不清楚是否能实现。

第二,如果你用Owner身份在organization仓库中创建一个issue(添加上post的href作为标签1,再添加Gitment作为标签2),那么这个issue应该就是这个post对应的评论了。这种情况下,organization的成员能继续评论吗?

aimingoo avatar Oct 16 '17 09:10 aimingoo

@aimingoo Hi,这是我们的仓库地址发布地址。 目前我们的解决方案是,把评论都放在了一个私有仓库。 关于权限限制的问题,我们并没有。只要登录了 github 就可以评论。 至于你给出的解决方案 我们试过了,并不可以。 很感谢你的回应,希望能解决这个问题 祝好!

hiyangguo avatar Oct 23 '17 09:10 hiyangguo

Ok.. 谢谢 @hiyangguo

我将尝试一下这个问题。尽量在周五前搞定,如果有需要我再联络你。

应该达到的效果是:

  • 在 https://github.com/hypers 中的团队中的所有人,都有权限在 blog.hypers.io 中初始化评论框,让 hypers/blog 中正确创建某个post对应的issue

对吧。

aimingoo avatar Oct 23 '17 16:10 aimingoo

@hiyangguo

应该已经搞定了。你尝试一下这个线上的Gitmint版本,如果没有问题我就发PR给gitment。

注意使用的是Gitmint

<link rel="stylesheet" href="https://aimingoo.github.io/gitmint/style/default.css">
<script src="https://aimingoo.github.io/gitmint/dist-trial/gitmint.browser.js"></script>

// 注意:这里使用Gitmint来创建
var gitment = new Gitmint({
  owner: 'hypers',
  admin: ['hiyangguo', 'simonguo', 'yelijing18'],
  ...  // 其它与Giment完全一样
})

// 与Gitment一样
gitment.render('comments')

aimingoo avatar Oct 24 '17 18:10 aimingoo

@aimingoo 效果是

在 @hypers 中的团队中的所有人,都有权限在 blog.hypers.io 中初始化评论框,让 hypers/blog 中正确创建某个post对应的issue

不过麻烦问一下 client_idclient_secret填什么?是组织的么? 感谢。

hiyangguo avatar Oct 25 '17 02:10 hiyangguo

是的。准确地说是 github.com/:owner的。因为是owner来验证login user的授权。

在组织中,owner与admin是仓库的所有者与管理者,是分开的;而在一般仓库中,owner就是admin。所以以前的版本中就只需要一个owner参数就可以了。

aimingoo avatar Oct 25 '17 03:10 aimingoo

@hiyangguo ding dong?

求进展。另,你不需要update整个blog来测试,你只需要找一篇已经发布过的改改页面然后push一下,刷新看看效果就好了。另外,如果有bug,留下这个现场,把页面链接扔给我~~~ :)

aimingoo avatar Oct 26 '17 04:10 aimingoo

这份代码是盲写的……没现场没环境没测试……所以心里一直发毛~~

aimingoo avatar Oct 26 '17 04:10 aimingoo

@aimingoo 我今天抽空弄,不好意思

hiyangguo avatar Oct 26 '17 04:10 hiyangguo

@aimingoo 测试可以了。Blog地址 十分感谢 👍👍

hiyangguo avatar Oct 26 '17 05:10 hiyangguo

呵呵,搞定! 我试过Login不成功,但应该是application中的callback设置的问题,与这里的issue就没关系了。 我会尽快提pr到gitment~ :)

aimingoo avatar Oct 26 '17 06:10 aimingoo

@aimingoo admin 这个字段 是什么意思? 指的就是可以初始化评论的人么? 所有 Hypers组织下的人都可以设置在这里么?

hiyangguo avatar Oct 26 '17 06:10 hiyangguo

@aimingoo 现在确实没办法登录 麻烦 解决一下这个问题 thx

hiyangguo avatar Oct 26 '17 06:10 hiyangguo

@hiyangguo

admin是一个字符串数组,指的就是可以初始化评论的人,必须是你的Hypers组织下有创建issues权限的人——好象需要是对这个repo有写权限的用户。只有他们在登录后可以看到“初始化”那个按钮。

好象这个没办法登录的问题是hypers的后台设置的问题,它的错误信息是“The redirect_uri MUST match the registered callback URL for this application.” 你现在用的

client_id=4d35960e25a9ddd509e1&client_secret=67135bfe928907183511ad93d386bf149f34e772

是哪个github仓库?在这个仓库的后台配置一下这个授权的callback url就好。这个应该不是我的update的问题。

aimingoo avatar Oct 26 '17 06:10 aimingoo

这是我们的后台设置, image

hiyangguo avatar Oct 26 '17 06:10 hiyangguo

Oh... 了解了。

简单地处理一下先:

var gitment = new Gitmint({
  owner: 'hypers',
  admin: ['hiyangguo', 'simonguo', 'yelijing18'],
  oauth: {
    redirect_protocol: 'http',
    ...
  ...  // 其它与Giment完全一样
})

先用下redirect_protocol强制重定向协议就好了。这个选项在Gitment中用不到。

aimingoo avatar Oct 26 '17 06:10 aimingoo

@aimingoo 我需要更新 gitmin 文件么

hiyangguo avatar Oct 26 '17 06:10 hiyangguo

不用。

aimingoo avatar Oct 26 '17 06:10 aimingoo

@aimingoo 好了 thx

hiyangguo avatar Oct 26 '17 07:10 hiyangguo

Done. see:

https://github.com/imsun/gitment/pull/87

aimingoo avatar Oct 26 '17 07:10 aimingoo

https://github.com/imsun/gitment/issues/56

sariabiha avatar Feb 11 '22 14:02 sariabiha

https://github.com/imsun/gitment/issues/56

sariabiha avatar Feb 11 '22 14:02 sariabiha

https://advance-esthetic.us/rf-facial-machines

sariabiha avatar Feb 11 '22 14:02 sariabiha