Results 31 comments of aimingoo

这个问题比较复杂,因为使用location.href是Gitment的默认和推荐方式,因此在Gitment中其实不方便直接修改这个约定。 我的建议是处理anchor,放过query。参考:https://github.com/aimingoo/gitmint/blob/master/src/gitment.js#L67 在Gitmint中的处理方式是通过重写'id'的getter/setter的方式,来处理掉了“不同锚点(anchor)” #62 的问题。这是合理的,因为`#`的锚点都是在页内,这意味着有没有锚点都是“同一个页”。然而不应该将这个规则应用到“查询(Query)”,因为没有办法确定不同的查询是否是同一个页面。例如`.../index.do?postId=123`,其实就不能直接将query除去了事。

支持。:) 这里有一个bash版的: https://github.com/aimingoo/monster/blob/master/libexec/updatesite.sh#L173 ```bash > monster update --sync-issue ```

@hiyangguo @jiangzhonglian @Whimsyduke @cebarobot Anybody, 能给我一个可以重现这个Bug的organization仓库不? 另外,有两个问题。第一,各位是不是希望organization之外的人就不能添加评论了?(现在是只要是github用户就可以添加,没有organization这样的限制)。我不太清楚这是否是真实的需求,也不清楚是否能实现。 第二,如果你用Owner身份在organization仓库中创建一个issue(添加上post的href作为标签1,再添加Gitment作为标签2),那么这个issue应该就是这个post对应的评论了。这种情况下,organization的成员能继续评论吗?

Ok.. 谢谢 @hiyangguo 我将尝试一下这个问题。尽量在周五前搞定,如果有需要我再联络你。 应该达到的效果是: * 在 https://github.com/hypers 中的团队中的所有人,都有权限在 blog.hypers.io 中初始化评论框,让 hypers/blog 中正确创建某个post对应的issue 对吧。

@hiyangguo 应该已经搞定了。你尝试一下这个线上的Gitmint版本,如果没有问题我就发PR给gitment。 注意使用的是`Gitmint`: ```html // 注意:这里使用Gitmint来创建 var gitment = new Gitmint({ owner: 'hypers', admin: ['hiyangguo', 'simonguo', 'yelijing18'], ... // 其它与Giment完全一样 }) // 与Gitment一样 gitment.render('comments') ```

是的。准确地说是 github.com/:owner的。因为是owner来验证login user的授权。 在组织中,owner与admin是仓库的所有者与管理者,是分开的;而在一般仓库中,owner就是admin。所以以前的版本中就只需要一个owner参数就可以了。

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

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

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

@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的问题。