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

添加评论聚合 - 网易云跟帖 #245

Closed
wants to merge 21 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@Hscpro
Copy link

Hscpro commented Mar 26, 2017

包括首页的评论数文章页内的评论数 #242 #

Hscpro added some commits Mar 26, 2017

添加评论聚合 - 网易云跟帖
包括首页评论数与文章页评论数
@tufu9441

This comment has been minimized.

Copy link
Owner

tufu9441 commented Mar 31, 2017

最近一次更新中已经添加基本功能(不包括评论数)。

Hscpro added some commits Apr 14, 2017

Merge branch 'master' of https://github.com/tufu9441/maupassant-hexo
# Conflicts:
#	LICENSE
#	README.md
#	_config.yml
#	layout/_partial/comments.jade
@Hscpro

This comment has been minimized.

Copy link

Hscpro commented Apr 15, 2017

Hscpro added some commits Apr 15, 2017

Synchronous main branch
Signed-off-by: hscpro <huisc@foxmail.com>
Add index gentie comment
Add page gentie comment

Signed-off-by: hscpro <huisc@foxmail.com>
@tufu9441

This comment has been minimized.

Copy link
Owner

tufu9441 commented Apr 19, 2017

主要是网易云跟帖好像在本地还测试不出效果,只有上传到服务器才能生效,这让我调试起来很蛋疼……

@Hscpro

This comment has been minimized.

Copy link

Hscpro commented Apr 20, 2017

它是判断域名的,这个的确蛋疼。本地测试要改一下电脑的host,指定网易绑定的域名解释到本地就好。还得更改端口hoxe s -p 80

@Hscpro

This comment has been minimized.

Copy link

Hscpro commented Apr 20, 2017

我是随便绑定一个域名,然后在本地也用这个域名测试
上面的地址一个手滑改了https://www.hscbook.com/article/hexo-gentie/

Hscpro added some commits Apr 20, 2017

@tufu9441

This comment has been minimized.

Copy link
Owner

tufu9441 commented Apr 20, 2017

我使用了全站https,hexo站点的url也是带https的,貌似改了hosts也没有用呃?

@tufu9441

This comment has been minimized.

Copy link
Owner

tufu9441 commented Apr 20, 2017

如果你显示评论数量那里“span”开头的代码都按照jade的语法写的话就好了,我就不用自己测试和修改了,可以直接添加上~你可以仿照一下其它地方的写法试一试

@Hscpro

This comment has been minimized.

Copy link

Hscpro commented Apr 21, 2017

他验证应该不判断是不是https,只是评论的时候https和http的评论内容会隔离,要过一段时间才会同步,也可以在后台手动同步

@Hscpro

This comment has been minimized.

Copy link

Hscpro commented Apr 21, 2017

我试试,因为我没接触过jade,现在还在摸索中

Hscpro added some commits Apr 21, 2017

jade.Grammatical correction
gentie.Style correction

Signed-off-by: hscpro <huisc@foxmail.com>
_config.yml.Sample correction
Signed-off-by: hscpro <huisc@foxmail.com>
del.gentie_loader
Signed-off-by: hscpro <huisc@foxmail.com>
@Hscpro

This comment has been minimized.

Copy link

Hscpro commented Apr 21, 2017

趁着午休修改了一下,我这边测试是没问题的。

style样式

style样式是有必要的,因为云跟帖自带了样式,与主题不符合而且后台没办法改样式。

首页

20170421_131207

文章

20170421_131056

评论

20170421_131122

@tufu9441

This comment has been minimized.

Copy link
Owner

tufu9441 commented Apr 21, 2017

我看你把loader删除了,意思是这个是每个用户的loader是相同的吗?

@Hscpro

This comment has been minimized.

Copy link

Hscpro commented Apr 21, 2017

你看一下你的是不是一样,在后台获取代码列表页参与数Tie.loader。如果一样就是每个用户的loader是相同的。我这边有两个域名都是一样的loader。

@tufu9441

This comment has been minimized.

Copy link
Owner

tufu9441 commented Apr 21, 2017

我看了我这边是 aHR0cHM6Ly9hcGkuZ2VudGllLjE2My5jb20vcGMvbGl2ZXNjcmlwdC5odG1s,还是不一样的

@Hscpro

This comment has been minimized.

Copy link

Hscpro commented Apr 21, 2017

我对比了一下我的两个域名后台发现它有两个 Tie.loader一个productKey
跟贴完整模块代码(Web单独版) 处的 Tie.loader 两个域名是不一样的 。
列表页参与数 处的 Tie.loader 两个域名却是一样的。
所以就以为 列表页参与数 处的 Tie.loader 都是是一样的loader。

@tufu9441

This comment has been minimized.

Copy link
Owner

tufu9441 commented Apr 21, 2017

嗯,我看到了,确实是这样

@tufu9441

This comment has been minimized.

Copy link
Owner

tufu9441 commented Apr 21, 2017

我更新了一个版本,显示“Disucss”那里稍微改动了一下,麻烦您测试一下有没有问题。

@Hscpro

This comment has been minimized.

Copy link

Hscpro commented Apr 21, 2017

暂时没有发现问题,浏览器也没有报错。不过似乎把评论改为动态之类的词语更加好些,显示的不是真正的评论数,就算留言没有通过审核或者删除了留言,数值还是会增加或不减。包括赞也在记数内
20170421_142458

@tufu9441

This comment has been minimized.

Copy link
Owner

tufu9441 commented Apr 21, 2017

好的,没有问题就好。翻译我也改了一下。

Hscpro added some commits Apr 22, 2017

Squashed commit of the following:
commit 0ddda1b
Author: tufu9441 <tufu9441@gmail.com>
Date:   Fri Apr 21 21:17:02 2017 +0800

    A bug fixed.

commit fce2e21
Author: tufu9441 <tufu9441@gmail.com>
Date:   Fri Apr 21 18:22:10 2017 +0800

    略微改动

commit 1e71abf
Author: tufu9441 <tufu9441@gmail.com>
Date:   Fri Apr 21 13:56:21 2017 +0800

    更新网易云跟帖

commit 6760b49
Merge: 4ff908f 1b67e9f
Author: tufu9441 <tufu9441@gmail.com>
Date:   Wed Apr 19 22:45:35 2017 +0800

    Merge pull request #261 from yiqiok/master

    Add 'year' after copyright

commit 1b67e9f
Merge: 4ff908f de5b15a
Author: Alex Stone <yiqiok@users.noreply.github.com>
Date:   Wed Apr 19 22:40:44 2017 +0800

    Merge pull request #1 from yiqiok/yiqiok-patch-1

    add 'year' after copyright

commit de5b15a
Author: Alex Stone <yiqiok@users.noreply.github.com>
Date:   Wed Apr 19 22:39:01 2017 +0800

    add 'year' after copyright

    '© title.'     ==>   '© year title' ,'year' will be  auto updated after a year

@tufu9441 tufu9441 closed this Apr 23, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment