Skip to content
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

feat: add the configure for declare copyright on posts #1497

Merged
merged 2 commits into from
Mar 13, 2017

Conversation

Jacksgong
Copy link
Contributor

Config on the _config.yml

# Declare license on posts
post_copyright:
  enable: true
  license: CC BY-NC-SA 3.0 CN
  license_url: http://creativecommons.org/licenses/by-nc-sa/3.0/cn/

Result

image

@Jacksgong
Copy link
Contributor Author

You can check out the effect from my blog.

@iissnan iissnan merged commit 8bd0819 into iissnan:master Mar 13, 2017
@iissnan
Copy link
Owner

iissnan commented Mar 13, 2017

Thanks. 👍

@ivan-nginx
Copy link
Collaborator

ivan-nginx commented Mar 13, 2017

There is some questions about this pull.

In _config.yml:

  1. Mb post_copyright in config need to replace into post block?
  2. Why CN is default? EN for defualt must be always.
  3. license_url must be httpS.

In post-copyright.swig:

  1. Why style is not use note tag style? There is no customization.
  2. Why CN langauge again?
  3. And where is customization of language again?

@Jacksgong i undestand what this is your own customization, but need to do for all peoples and all languages, not only for CN.

@iissnan very easy merge.

@iissnan
Copy link
Owner

iissnan commented Mar 13, 2017

@ivan-nginx Good catches. I am working on the improvement to this pr right now.

iissnan added a commit that referenced this pull request Mar 13, 2017
iissnan added a commit that referenced this pull request Mar 15, 2017
habren pushed a commit to habren/hexo-theme-next that referenced this pull request Apr 9, 2017
feat: add the configure for declare copyright on posts
habren pushed a commit to habren/hexo-theme-next that referenced this pull request Apr 9, 2017
habren pushed a commit to habren/hexo-theme-next that referenced this pull request Apr 9, 2017
@xiao-fang
Copy link

xiao-fang commented Apr 25, 2017

how can i disable some posts' copyrights but not all posts? via font-matter?

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

Successfully merging this pull request may close these issues.

4 participants