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

add wechat subscriber #788

Merged
merged 1 commit into from Apr 3, 2016

Conversation

Projects
None yet
3 participants
@huiwang
Contributor

huiwang commented Apr 2, 2016

在每一篇文章结尾处添加微信公众号二维码,扫一扫,订阅博客

@huiwang huiwang referenced this pull request Apr 2, 2016

Open

添加个人二维码 #760

@huiwang

This comment has been minimized.

Show comment
Hide comment
@huiwang

huiwang Apr 3, 2016

Contributor

@iissnan 麻烦您看一下,能不能接受这个PR,谢谢。

Contributor

huiwang commented Apr 3, 2016

@iissnan 麻烦您看一下,能不能接受这个PR,谢谢。

@flashlab

This comment has been minimized.

Show comment
Hide comment
@flashlab

flashlab Apr 3, 2016

Contributor

我觉得加配置还是慎重点好,整个模板已经很庞杂了,没解决的和没发现的bug越来越多,如果确实需要可以再开一些分支,把这些小的附加功能归类到一起。纯属个人意见 :)

Contributor

flashlab commented Apr 3, 2016

我觉得加配置还是慎重点好,整个模板已经很庞杂了,没解决的和没发现的bug越来越多,如果确实需要可以再开一些分支,把这些小的附加功能归类到一起。纯属个人意见 :)

@huiwang

This comment has been minimized.

Show comment
Hide comment
@huiwang

huiwang Apr 3, 2016

Contributor

@flashlab 谢谢您的意见。我觉得您的担忧是对的,新的功能确实增加系统的规模。大规模的系统的维护通常是趋于复杂的。

不知道你有没有看这个PR里改变的文件的细节。这个新功能是独立的,封装好的,它的增加会使系统更加庞大,但并不会使它变得更加复杂。

另外,您说的这些话让我想到,我们是不是应该写一个程序员贡献手册,来规范今后新的开发。

Contributor

huiwang commented Apr 3, 2016

@flashlab 谢谢您的意见。我觉得您的担忧是对的,新的功能确实增加系统的规模。大规模的系统的维护通常是趋于复杂的。

不知道你有没有看这个PR里改变的文件的细节。这个新功能是独立的,封装好的,它的增加会使系统更加庞大,但并不会使它变得更加复杂。

另外,您说的这些话让我想到,我们是不是应该写一个程序员贡献手册,来规范今后新的开发。

@flashlab

This comment has been minimized.

Show comment
Hide comment
@flashlab

flashlab Apr 3, 2016

Contributor

恩,Next拓展性很好,所以这个改动应该不会有什么副作用,但是维护不只局限于代码,文档,demo,复杂环境的兼容性等等都是需要耗费精力的。针对你这个pr,如果改动config文件,有可能其他人在pull的时候会和自己修改的版本冲突。至于手册我觉得不必,contribution就是要五花八门才好

Contributor

flashlab commented Apr 3, 2016

恩,Next拓展性很好,所以这个改动应该不会有什么副作用,但是维护不只局限于代码,文档,demo,复杂环境的兼容性等等都是需要耗费精力的。针对你这个pr,如果改动config文件,有可能其他人在pull的时候会和自己修改的版本冲突。至于手册我觉得不必,contribution就是要五花八门才好

@huiwang

This comment has been minimized.

Show comment
Hide comment
@huiwang

huiwang Apr 3, 2016

Contributor

@flashlab, 同意,代码只是一部分,很多时候是最简单的一部分,如果大家接受这个PR的话,我会把接下的工作一并做好。

至于你所提到的冲突的问题,我想是有可能的,毕竟只有一个config文件。解析这个冲突应该会很快,因为它是独立的。根据我个人经验,避免冲突的最有效的方式,就是连续的作小的整合,不要积攒了很多之后再merge。

最后,能不能被接受,当然还是看commiter的想法,看大家的需求。我只是觉得,我做的这个小功能,别人可能会有同样的需求,所以就分享出来了,避免重复性工作。

Contributor

huiwang commented Apr 3, 2016

@flashlab, 同意,代码只是一部分,很多时候是最简单的一部分,如果大家接受这个PR的话,我会把接下的工作一并做好。

至于你所提到的冲突的问题,我想是有可能的,毕竟只有一个config文件。解析这个冲突应该会很快,因为它是独立的。根据我个人经验,避免冲突的最有效的方式,就是连续的作小的整合,不要积攒了很多之后再merge。

最后,能不能被接受,当然还是看commiter的想法,看大家的需求。我只是觉得,我做的这个小功能,别人可能会有同样的需求,所以就分享出来了,避免重复性工作。

@iissnan

This comment has been minimized.

Show comment
Hide comment
@iissnan

iissnan Apr 3, 2016

Owner

@huiwang Thanks 👍

我合并到 v5.0.1 分支中,master 分支目前 freeze for v5.0.0。

Owner

iissnan commented Apr 3, 2016

@huiwang Thanks 👍

我合并到 v5.0.1 分支中,master 分支目前 freeze for v5.0.0。

@iissnan iissnan added this to the v5.0.1 milestone Apr 3, 2016

@iissnan iissnan added the Docs label Apr 3, 2016

@huiwang

This comment has been minimized.

Show comment
Hide comment
@huiwang

huiwang Apr 3, 2016

Contributor

谢谢 @iissnan , 我可以帮着把doc也填上,不知道doc在哪里改?

Contributor

huiwang commented Apr 3, 2016

谢谢 @iissnan , 我可以帮着把doc也填上,不知道doc在哪里改?

@iissnan

This comment has been minimized.

Show comment
Hide comment
Owner

iissnan commented Apr 3, 2016

@iissnan iissnan merged commit b3fa964 into iissnan:master Apr 3, 2016

1 check passed

hound No violations found. Woof!
@@ -0,0 +1,6 @@
{% if theme.wechat_subscriber.enabled %}
<div id="wechat_subscriber" style="display: block; padding: 10px 0; margin: 20px auto; width: 100%; text-align: center">
<img id="wechat_subscriber_qcode" src="{{ theme.wechat_subscriber.qcode }}" alt="{{ theme.author }} wechat" style="width: 200px; max-width: 100%;"/>

This comment has been minimized.

@iissnan

iissnan Apr 10, 2016

Owner

这个地方使用 url_for 会比较好点。

src="{{ theme.wechat_subscriber.qcode }} -> src="{{ url_for(theme.wechat_subscriber.qcode) }}

@iissnan

iissnan Apr 10, 2016

Owner

这个地方使用 url_for 会比较好点。

src="{{ theme.wechat_subscriber.qcode }} -> src="{{ url_for(theme.wechat_subscriber.qcode) }}

@iissnan iissnan removed the Docs label May 9, 2016

@huiwang huiwang deleted the huiwang:wechat-subscriber-final branch Oct 19, 2016

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